On Mon, Nov 02, 2015 at 03:12:49PM +0530, Vineet Gupta wrote: > > @@ -664,16 +670,24 @@ static inline void arch_write_lock(arch_rwlock_t *rw) > > > > static inline void arch_read_unlock(arch_rwlock_t *rw) > > { > > + unsigned long flags; > > + > > + local_irq_save(flags); > > arch_spin_lock(&(rw->lock_mutex)); > > > Isn't raw_spin_lock_irqsave() equivalent and more concise ?
You might run into header file recursion issues; also raw_spinlock has lockdep annotations added, whereas the above does not. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/