Re: [RESEND PATCH] module: show disallowed symbol name for inherit_taint()

From: Luis Chamberlain
Date: Wed May 04 2022 - 16:48:16 EST


On Wed, Apr 27, 2022 at 03:36:06PM +0800, Lecopzer Chen wrote:
> The error log for inherit_taint() doesn't really help to find the
> symbol which violates GPL rules.
>
> For example,
> if a module has 300 symbol and includes 50 disallowed symbols,
> the log only shows the content below and we have no idea what symbol is.
> AAA: module using GPL-only symbols uses symbols from proprietary module BBB.
>
> It's hard for user who doesn't really know how the symbol was parsing.
>
> This patch add symbol name to tell the offending symbols explicitly.
> AAA: module using GPL-only symbols uses symbols SSS from proprietary module BBB.
>
> Signed-off-by: Lecopzer Chen <lecopzer.chen@xxxxxxxxxxxx>

Thanks, pushed onto modules-testing. If there are no issues I'll
eventually push this onto modules-next.

Luis