Re: [bde@FreeBSD.org: cvs commit: src/sys/kern kern_resource.c]

[Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>]

:FYI,
:
:...
:bde         2004-06-22 04:38:08 UTC
:
:  FreeBSD src repository
:
:  Modified files:        (Branch: RELENG_4)
:    sys/kern             kern_resource.c=20
:  Log:
:  Raise the ipl to splstatclock() in donice() before calling resetpriority(=
:).
:  This fixes races with schedcpu() and schedclock().  It corresponds to
:..

    Interesting.  As far as I can tell, this one does not apply to us
    because we use a critical section in our resetpriority() function
    and all operations in curproc on a particular cpu are always forwarded
    to that cpu.  

    Also, the scheduler, stat, and hard clocks on DragonFly are distributed
    to all cpus, not run on just one cpu, so a critical section ought to be
    sufficient for us.

						-Matt