Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 4 Apr 2008 09:04:37 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Jeff Roberson <jeff@freebsd.org>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/kern sched_ule.c
Message-ID:  <200804040904.37819.jhb@freebsd.org>
In-Reply-To: <200804040104.m3414hJj074163@repoman.freebsd.org>
References:  <200804040104.m3414hJj074163@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday 03 April 2008 09:04:43 pm Jeff Roberson wrote:
> jeff        2008-04-04 01:04:43 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/kern             sched_ule.c
>   Log:
>    - Don't check for the ITHD pri class in tdq_load_add and rem.  4BSD
> doesn't do this either.  Simply check P_NOLOAD.  It'd be nice if this was
> in a thread flag so we didn't have an extra cache miss every time we add
> and remove a thread from the run-queue.

Actually, now that all interrupt threads are in a single process, it should be 
a per-thread flag.  It was only ever intended to mask softclock since that is 
what did the load calculation on 4BSD so that 4BSD wouldn't always return a 
load of 1 even on an idle box (since softclock was always running when the 
load was checked).

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200804040904.37819.jhb>