Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3;
From:   "Eric W. Biederman" <ebiederm@xmission.com>
To:     Dimitri John Ledkov <dimitri.ledkov@canonical.com>
Cc:     Richard Henderson <richard.henderson@linaro.org>,
        Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
        Matt Turner <mattst88@gmail.com>,
        Geert Uytterhoeven <geert@linux-m68k.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "H . Peter Anvin" <hpa@zytor.com>, linux-alpha@vger.kernel.org,
        linux-m68k@lists.linux-m68k.org, x86@kernel.org,
        linux-kernel@vger.kernel.org
References: <20231123180246.750674-1-dimitri.ledkov@canonical.com>
Date:   Fri, 24 Nov 2023 00:00:15 -0600
In-Reply-To: <20231123180246.750674-1-dimitri.ledkov@canonical.com> (Dimitri
        John Ledkov's message of "Thu, 23 Nov 2023 18:02:40 +0000")
Message-ID: <87plzzu1w0.fsf@email.froward.int.ebiederm.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [PATCH 0/5] remove the last bits of a.out support
Precedence: bulk

Dimitri John Ledkov <dimitri.ledkov@canonical.com> writes:

> I was working on how linux-libc-dev headers are shipped in Ubuntu and
> stumbled upon seemingly unused and useless linux/a.out.h header. It
> seems like it is an accidental leftover at this point.

How do you see that they are unused?

Are they never exported to userspace?

Are there any userspace programs that care?

Performing a quick debian code search I see chromium, qt6, ruby-rogue, hurd,
bazel_bootstrap, aboot, cde.

I can imagine all kinds of reasons old code could be using headers for a
historical format.  Some of them are quite legitimate, and some of them
are quite silly.  If it is old code like aboot it may be that it is
difficult to test any changes.  If memory serves you have to flash your
firmware to change/test aboot.

Because showing userspace does not care about the definitions in a file
is a completely different problem then showing the kernel does not care
about the definitions I left them, last time I was working in this area.
Keeping headers that will never change is not cost to the kernel so it
doesn't hurt us to be nice to historical userspace.

My quick debian code search suggests that there are pieces of userspace
that still use linux/a.out.h.  Are you seeing something I am not?
Do all of those pieces of code compile just fine with a.out.h missing?

Eric


> Dimitri John Ledkov (5):
>   alpha: remove a.out support from tools/objstrip
>   alpha: stop shipping a.out.h uapi headers
>   m68k: stop shipping a.out.h uapi headers
>   x86: stop shipping a.out.h uapi headers
>   uapi: remove a.out.h uapi header
>
>  arch/alpha/boot/tools/objstrip.c    |  52 +-----
>  arch/alpha/include/uapi/asm/a.out.h |  92 ----------
>  arch/m68k/include/uapi/asm/a.out.h  |  21 ---
>  arch/x86/include/uapi/asm/a.out.h   |  21 ---
>  include/uapi/Kbuild                 |   4 -
>  include/uapi/linux/a.out.h          | 251 ----------------------------
>  6 files changed, 6 insertions(+), 435 deletions(-)
>  delete mode 100644 arch/alpha/include/uapi/asm/a.out.h
>  delete mode 100644 arch/m68k/include/uapi/asm/a.out.h
>  delete mode 100644 arch/x86/include/uapi/asm/a.out.h
>  delete mode 100644 include/uapi/linux/a.out.h