Re: [PATCH 1/2] autofs: Only declare function when CONFIG_COMPATis defined

From: Frederic Weisbecker
Date: Sun Sep 26 2010 - 09:02:42 EST


On Sat, Sep 25, 2010 at 05:55:57PM +0200, Arnd Bergmann wrote:
> On Friday 24 September 2010 15:22:22 Frederic Weisbecker wrote:
> > From: MÃrton NÃmeth <nm127@xxxxxxxxxxx>
> >
> > The patch solves the following warnings message when CONFIG_COMPAT
> > is not defined:
> >
> > fs/autofs/root.c:30: warning: âautofs_root_compat_ioctlâ declared âstaticâ but never defined
> >
> > Signed-off-by: MÃrton NÃmeth <nm127@xxxxxxxxxxx>
> > Cc: "H. Peter Anvin" <hpa@xxxxxxxxx>
> > Cc: Arnd Bergmann <arnd@xxxxxxxx>
> > Signed-off-by: Frederic Weisbecker <fweisbec@xxxxxxxxx>
>
> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
>
> My initial reaction was to put these into my bkl/vfs tree, but then I noticed
> that they are against a 2.6.36 bug. Who's taking care of getting them upstream?
>
> - Ian (maintainer)
> - Arnd (who wrote the broken patch to start with)
> - Frederic (who pushed the broken patch to Linus)
> - Andrew (took them into -mm)
>
> I don't care either way, just trying to make sure it gets there and we don't
> all submit the same patch simultaneously.
>
> The current state is annoying but harmless, so I could also just add the
> patches to my 2.6.37 queue, which already has autofs patches for the BKL.
>
> Arnd



I think Andrew has taken the autofs one, as I've received an mmotm notification.
I don't know about the autofs4 one though.

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