Re: [PATCH v4 1/3] kbuild: allow symbol whitelisting with TRIM_UNUSED_KSYMS

From: Matthias Maennich
Date: Mon Feb 17 2020 - 11:35:47 EST


On Mon, Feb 17, 2020 at 11:00:39AM -0500, Nicolas Pitre wrote:
On Mon, 17 Feb 2020, Quentin Perret wrote:

On Monday 17 Feb 2020 at 15:22:01 (+0000), Matthias Maennich wrote:
> In case the whitelist file can't be found, the error message is
>
> cat: path/to/file: file not found
>
> I wonder whether we can make this error message a bit more specific by
> telling the user that the KSYMS_WHITELIST is missing.

+1, that'd be really useful. I'll check the file existence in v5 (in a
POSIX-compliant way, I promise).

In fact, if you explicitly provide a file that is not there, then this
is arguably a good reason to even fail the build.

I agree, I would expect the build to fail in that case.

Cheers,
Matthias



Nicolas