Re: [PATCH 07/18] fs: rework icount to be a locked variable

From: Christoph Hellwig
Date: Fri Oct 15 2010 - 20:23:15 EST


On Sat, Oct 16, 2010 at 11:20:43AM +1100, Dave Chinner wrote:
> Oh, NAK that - hit send too soon. I forgot - they're are done that
> way because they are under the inode_lock, and iref(), at this point
> in the series, takes the inode_lock. So while it looks weird, it has
> to stay that way otherwise it deadlocks.....

Sounds fine anyway. I was just wondering why it was done, and
the later extension of the i_lock coverage is reason enough anyway.

--
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/