El Mon, Apr 23, 2007 at 01:25:58AM +0200 Andi Kleen ha dit: > Matthias Kaehlcke <[EMAIL PROTECTED]> writes: > > > -static DECLARE_MUTEX(sony_sem); /* Semaphore for drive hardware > > access */ > > +static DEFINE_MUTEX(sony_mtx); /* Mutex for drive hardware > > access */ > > That's not a spinlock. Also normally some rationale is added to the > description for a change?
sorry i messed up the description of the change, i meant mutex instead of spinlock (in the last days i reported some spinlock related bugs ...). the rationale is that according to http://lwn.net/Articles/167034/ binary semaphores that aren't given in interrupt context or locked and unlocked by different processes should be replaced by mutexes thanks for your comments -- Matthias Kaehlcke Linux Application Developer Barcelona La posibilidad de realizar un suenyo es lo que hace que la vida sea interesante .''`. using free software / Debian GNU/Linux | http://debian.org : :' : `. `'` gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4 `- - 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/
