Re: [PATCH] remove volatile from x86 cmos_lock

[Dave Jones]

On Fri, Jul 14, 2006 at 11:09:25AM -0400, Steven Rostedt wrote:
 > On Fri, 2006-07-14 at 16:53 +0200, Oliver Neukum wrote:
 > > Am Freitag, 14. Juli 2006 16:48 schrieb Steven Rostedt:
 > > > @@ -52,14 +54,16 @@ static inline void lock_cmos(unsigned ch
 > > >  
 > > >  static inline void unlock_cmos(void)
 > > >  {
 > > > -       cmos_lock = 0;
 > > > +       set_wmb(cmos_lock, 0);
 > > >  }
 > > >  static inline int do_i_have_lock_cmos(void)
 > > >  {
 > > > +       barrier();
 > > 
 > > Shouldn't these be rmb() ?
 > 
 > I was thinking that too, but I'm still not sure when to use rmb or
 > barrier.  wmb seems pretty straight forward though.  hmm, maybe this
 > really should be a smb_rmb since I believe a barrier would be ok for UP.

I'm more puzzled why it's inventing its own locking primitives instead of
using one of the many the kernel provides.  This stuff is prehistoric though.
Hangover from the really early days ?

		Dave

-- 
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/