On Wed, Mar 23, 2005 at 08:40:58PM +1100, Herbert Xu wrote:
> Paul E. McKenney <[EMAIL PROTECTED]> wrote:
> >
> > +void rcu_read_unlock(void)
> > +{
> > +       int cpu;
> > +
> > +       if (--current->rcu_read_lock_nesting == 0) {
> > +               atomic_dec(&current->rcu_data->active_readers);
> > +               /*
> > +                * Check whether we have reached quiescent state.
> > +                * Note! This is only for the local CPU, not for
> > +                * current->rcu_data's CPU [which typically is the
> > +                * current CPU, but may also be another CPU].
> > +                */
> > +               cpu = get_cpu();
> > 
> > And need an smp_mb() here, again for non-x86 CPUs.
> 
> The atomic_dec is already a barrier.

Sorry, I was confused.  atomic_dec is not a barrier of course.
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to