Re: [PATCH 6/7] autofs4 - use struct qstr in waitq.c

From: Ian Kent
Date: Wed Jul 23 2008 - 01:12:50 EST



On Tue, 2008-07-22 at 16:13 -0700, Andrew Morton wrote:
> On Tue, 17 Jun 2008 20:24:06 +0800
> Ian Kent <raven@xxxxxxxxxx> wrote:
>
> > From: Jeff Moyer <jmoyer@xxxxxxxxxx>
> >
> > The autofs_wait_queue already contains all of the fields of the
> > struct qstr, so change it into a qstr.
> >
> > This patch, from Jeff Moyer, has been modified a liitle by myself.
> >
> > Signed-off-by: Jeff Moyer <jmoyer@xxxxxxxxxx>
> > Signed-off-by: Ian Kent <raven@xxxxxxxxxx>
>
> So this patch which had been happily sitting in -mm for a month has
> suddenly broken because linux-next's three-day-old
> 4bce7ce7c7d0d57b78dacc3a2bd87ec63b2d9b4c has removed LOOKUP_ACCESS.
>
> This is suboptimal.
>
> Now what do I do?

Ummm .. I'm confused.

Your patch autofs4-use-lookup-intent-flags-to-trigger-mounts-fix.patch
allows the linux-next kernel to build with all the autofs4 patches
currently posted for inclusion in mm but the patch you mention here
isn't concerned with the lookup flags?

The removal of LOOKUP_ACCESS is quite interesting. AFAIKS it effectively
prevents the patch
autofs4-use-lookup-intent-flags-to-trigger-mounts.patch from also
resolving an issue with recursive autofs mounts while still resolving
the issue that the patch was actually meant to address.

It's hard to get exited about the former issue as Al Viro has NACKed a
previous patch that added the LOOKUP_ACCESS check, indicating the
availability of the lookup flags will be changing. Also there is a
question as to whether autofs will support the use mount points in
automount maps that themselves refer to an automount path (the recursive
bit).

Ian

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