td->td_critnest manipulations do not use atomic_add_int ?

[John Baldwin]

On Tuesday 20 December 2005 11:16 am, Luigi Rizzo wrote:
> On Tue, Dec 20, 2005 at 06:34:20AM -0800, Suleiman Souhlal wrote:
> > Hello Luigi,
> >
> > Luigi Rizzo wrote:
> > > as in the subject... i see that td->td_critnest (used to determine
> > > whether a thread can be preempted or not) is manipulated using
> > > plain ++ or -- instruction instead of the atomic_add_int().
> >
> > This should be fine as it only gets modified by the current thread. If
> > an interrupt comes while we are decreasing td_critnest back to 0, then
> > we just don't get preempted immediately, but at the end of our quantum,
> > or when someone else tries to preempt us, whichever comes first, which
> > should be totally harmless.
>
> i think that there are still some potential race conditions if
> the variable is read from another processor to make a decision
> based on its value.

It's not, that's the key.  It's only read by the current thread.  Because of 
sched_lock being held when a thread context switches (and thus anytime it 
migrates) and the membars it contains, no other locking is needed for data 
that only curthread accesses.

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org