MAINTAINERS: drop entry to removed file in EXEC & BINFMT API

[PATCH] MAINTAINERS: drop entry to removed file in EXEC & BINFMT API

Posted by Lukas Bulwahn 4 years, 3 months ago

Commit f9444ea5e208 ("x86: Remove a.out support") removes
arch/x86/ia32/ia32_aout.c, but misses a reference to that file in
MAINTAINERS.

Hence, ./scripts/get_maintainer.pl --self-test=patterns complains about a
broken reference.

Drop the file entry to the removed file in EXEC & BINFMT API.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
Borislav, please pick this minor clean-up patch on your -next tree on top
of the commit above.

 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8f5d5fb7cdde..faa5e53db1dd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7365,7 +7365,6 @@ L:	linux-mm@kvack.org
 S:	Supported
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/execve
 F:	arch/alpha/kernel/binfmt_loader.c
-F:	arch/x86/ia32/ia32_aout.c
 F:	fs/*binfmt_*.c
 F:	fs/exec.c
 F:	include/linux/binfmts.h
-- 
2.17.1

Re: [PATCH] MAINTAINERS: drop entry to removed file in EXEC & BINFMT API

Posted by Borislav Petkov 4 years, 3 months ago

On Wed, Mar 16, 2022 at 06:08:28AM +0100, Lukas Bulwahn wrote:
> Commit f9444ea5e208 ("x86: Remove a.out support") removes
> arch/x86/ia32/ia32_aout.c, but misses a reference to that file in
> MAINTAINERS.

It seems we're keeping a.out after all:

https://lore.kernel.org/r/4c449fab-8135-5057-7d2c-7b948ce130cc@theinnocuous.com

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH] MAINTAINERS: drop entry to removed file in EXEC & BINFMT API

Posted by Lukas Bulwahn 4 years, 3 months ago

On Wed, Mar 16, 2022 at 8:50 AM Borislav Petkov <bp@alien8.de> wrote:
>
> On Wed, Mar 16, 2022 at 06:08:28AM +0100, Lukas Bulwahn wrote:
> > Commit f9444ea5e208 ("x86: Remove a.out support") removes
> > arch/x86/ia32/ia32_aout.c, but misses a reference to that file in
> > MAINTAINERS.
>
> It seems we're keeping a.out after all:
>
> https://lore.kernel.org/r/4c449fab-8135-5057-7d2c-7b948ce130cc@theinnocuous.com
>

I guess then you can squash my one-line patch into your patch, and
then we put all this into the closet and wait for another day where
this feature can be removed.

Thanks, Boris.

Lukas

> --
> Regards/Gruss,
>     Boris.
>
> https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH] MAINTAINERS: drop entry to removed file in EXEC & BINFMT API

Posted by Borislav Petkov 4 years, 3 months ago

On Wed, Mar 16, 2022 at 09:12:54AM +0100, Lukas Bulwahn wrote:
> I guess then you can squash my one-line patch into your patch, and

Right.

> then we put all this into the closet and wait for another day where
> this feature can be removed.

Yeah, I wouldn't hold my breath. Everytime I attempt this, someone
crawls out of the woodwork with a use case.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette