On Wed, Sep 29, 2010 at 09:52:20PM -0700, Andrew Morton wrote: > On Wed, 29 Sep 2010 22:18:35 +1000 Dave Chinner <david@xxxxxxxxxxxxx> wrote: > > > DEFINE_SPINLOCK(inode_lock); > > DEFINE_SPINLOCK(sb_inode_list_lock); > > +DEFINE_SPINLOCK(inode_hash_lock); > > I assume these all go away later on. If not, they'll probably all land > in the same cacheline! indeed, the hash lock goes away in this series, as does the inode_lock. I'll check the other new locks don't land in the same cacheline. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html