Re: [PATCH 17/19] VFS: set PF_FSTRANS while namespace_sem is held.

From: Al Viro
Date: Wed Apr 16 2014 - 12:37:57 EST


On Wed, Apr 16, 2014 at 03:52:30PM +1000, NeilBrown wrote:

> So something like this? I put that in to my testing instead.

Something like this, yes... And TBH I would prefer the same approach
elsewhere - this kind of hidden state makes it hard to do any kind of
analysis.

Put it that way - the simplest situation is when the allocation flags
depend only on the call site. Next one is when it's a function of
call chain - somewhat harder to review. And the worst is when it's
a function of previous history of execution - not just the call chain,
but the things that had been called (and returned) prior to that one.

How many of those locations need to be of the third kind? All fs/namespace.c
ones are of the first one...
--
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/