Re: [PATCH] VFS: Fix automount for negative autofs dentries

From: Christoph Hellwig
Date: Mon Jul 11 2011 - 11:25:01 EST


On Mon, Jul 11, 2011 at 03:13:41PM +0100, David Howells wrote:
> Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
>
> > would do the same.
>
> But is much less obvious. The LOOKUP_FOLLOW flag is the primary reason for
> this statement. The rest are subordinate and would be wholly irrelevant if
> LOOKUP_FOLLOW was to be removed from the list.

Then keep the LOOKUP_FOLLOW semi-separate as in the original code.
But there's absolutely no reason for the goto spaghetti.

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