On Sat, Nov 23, 2024 at 5:33 AM Thomas Weißschuh <linux@xxxxxxxxxxxxxx> wrote:
>
> Use CONFIG_WERROR to also fail on warnings emitted by resolve_btfids.
> Allow the CI bots to prevent the introduction of new warnings.
>
> Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
> ---
> scripts/link-vmlinux.sh | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/link-vmlinux.sh b/scripts/link-vmlinux.sh
> index a9b3f34a78d2cd4514e73a728f1a784eee891768..61f1f670291351a276221153146d66001eca556c 100755
> --- a/scripts/link-vmlinux.sh
> +++ b/scripts/link-vmlinux.sh
> @@ -274,7 +274,11 @@ vmlinux_link vmlinux
> # fill in BTF IDs
> if is_enabled CONFIG_DEBUG_INFO_BTF; then
> info BTFIDS vmlinux
> - ${RESOLVE_BTFIDS} vmlinux
> + RESOLVE_BTFIDS_ARGS=""
> + if is_enabled CONFIG_WERROR; then
> + RESOLVE_BTFIDS_ARGS=" --fatal-warnings "
> + fi
> + ${RESOLVE_BTFIDS} ${RESOLVE_BTFIDS_ARGS} vmlinux
I'm not convinced we need to fail the build when functions are renamed.
These warns are eventually found and fixed.
