Date: Sat, 01 Sep 2001 15:21:51 -0700 From: Peter Wemm <peter@wemm.org> To: John Baldwin <jhb@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm vm_zeroidle.c Message-ID: <20010901222151.315853807@overcee.netplex.com.au> In-Reply-To: <200109012016.f81KGU977142@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
John Baldwin wrote:
> jhb 2001/09/01 13:16:30 PDT
>
> Modified files:
> sys/vm vm_zeroidle.c
> Log:
> Process priority is locked by the sched_lock, not the proc lock.
>
> Revision Changes Path
> 1.5 +3 -3 src/sys/vm/vm_zeroidle.c
Then you probably want to tweak rtprio(2) as well, where this was copied
from..
rtprio(curp, uap)
{
...
PROC_LOCK(p);
...
error = rtp_to_pri(&rtp, &p->p_pri);
break;
...
PROC_UNLOCK(p);
return (error);
}
Or perhaps better: put sched_lock calls into rtp_to_pri() function since
that would save all the callers of it from grabbing / releasing it themselves.
(see posix4/ksched.c kern/kern_resource.c vm/vm_zeroidle.c)
Cheers,
-Peter
--
Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au
"All of this is for nothing if we don't go to the stars" - JMS/B5
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010901222151.315853807>