d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

- Don't check for the ITHD pri class in tdq_load_add and rem. 4BSD doesn't

Browse Source 
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.
This commit is contained in:
Jeff Roberson 2008-04-04 01:04:43 +00:00
parent e4b1aa6210
commit 03d17db7d5
1 changed files with 6 additions and 12 deletions

18
sys/kern/sched_ule.c

@ -460,17 +460,14 @@ tdq_runq_rem(struct tdq *tdq, struct thread *td)
static void
tdq_load_add(struct tdq *tdq, struct thread *td)
{
struct td_sched *ts;
int class;
ts = td->td_sched;
TDQ_LOCK_ASSERT(tdq, MA_OWNED);
THREAD_LOCK_ASSERT(td, MA_OWNED);
class = PRI_BASE(td->td_pri_class);
tdq->tdq_load++;
CTR2(KTR_SCHED, "cpu %d load: %d", TDQ_ID(tdq), tdq->tdq_load);
if (class != PRI_ITHD && (td->td_proc->p_flag & P_NOLOAD) == 0)
if ((td->td_proc->p_flag & P_NOLOAD) == 0)
tdq->tdq_sysload++;
CTR2(KTR_SCHED, "cpu %d load: %d", TDQ_ID(tdq), tdq->tdq_load);
}
/*
@ -480,18 +477,15 @@ tdq_load_add(struct tdq *tdq, struct thread *td)
static void
tdq_load_rem(struct tdq *tdq, struct thread *td)
{
struct td_sched *ts;
int class;
ts = td->td_sched;
THREAD_LOCK_ASSERT(td, MA_OWNED);
TDQ_LOCK_ASSERT(tdq, MA_OWNED);
class = PRI_BASE(td->td_pri_class);
if (class != PRI_ITHD && (td->td_proc->p_flag & P_NOLOAD) == 0)
tdq->tdq_sysload--;
KASSERT(tdq->tdq_load != 0,
("tdq_load_rem: Removing with 0 load on queue %d", TDQ_ID(tdq)));
tdq->tdq_load--;
if ((td->td_proc->p_flag & P_NOLOAD) == 0)
tdq->tdq_sysload--;
CTR1(KTR_SCHED, "load: %d", tdq->tdq_load);
}