diff mbox series

[SRU,JAMMY] (upstream) scripts/pahole-flags.sh: Add --skip_encoding_btf_enum64 for pahole versions >= 124

Message ID 20240125221311.912250-1-mitchell.augustin@canonical.com
State New
Headers show
Series [SRU,JAMMY] (upstream) scripts/pahole-flags.sh: Add --skip_encoding_btf_enum64 for pahole versions >= 124 | expand

Commit Message

mitchell.augustin@canonical.com Jan. 25, 2024, 10:13 p.m. UTC
From: Mitchell Augustin <mitchell.augustin@canonical.com>

[ Upstream commit b775fbf532dc01ae53a6fc56168fd30cb4b0c658 ]

New pahole (version 1.24) generates by default new BTF_KIND_ENUM64 BTF tag,
which is not supported by stable kernel.

As a result the kernel with CONFIG_DEBUG_INFO_BTF option will fail to
compile with following error:

  BTFIDS  vmlinux
FAILED: load BTF from vmlinux: Invalid argument

New pahole provides --skip_encoding_btf_enum64 option to skip BTF_KIND_ENUM64
generation and produce BTF supported by stable kernel.

Signed-off-by: Mitchell Augustin <mitchell.augustin@canonical.com>
---
 scripts/pahole-flags.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Stefan Bader Jan. 26, 2024, 8:17 a.m. UTC | #1
On 25.01.24 23:13, mitchell.augustin@canonical.com wrote:
> From: Mitchell Augustin <mitchell.augustin@canonical.com>
> 
> [ Upstream commit b775fbf532dc01ae53a6fc56168fd30cb4b0c658 ]

Upstream reference to be made by cherry picked / backported line (see 
below). We need a launchpad bug report for SRU though:

BugLink: https://bugs.launchpad.net/bugs/???

The subject (commit title) would just have "[SRU Jammy]" added to it 
(git format-patch).

> 
> New pahole (version 1.24) generates by default new BTF_KIND_ENUM64 BTF tag,
> which is not supported by stable kernel.
> 
> As a result the kernel with CONFIG_DEBUG_INFO_BTF option will fail to
> compile with following error:
> 
>    BTFIDS  vmlinux
> FAILED: load BTF from vmlinux: Invalid argument
> 
> New pahole provides --skip_encoding_btf_enum64 option to skip BTF_KIND_ENUM64
> generation and produce BTF supported by stable kernel.
> 
(cherry picked from commit b775fbf532dc01ae53a6fc56168fd30cb4b0c658)
> Signed-off-by: Mitchell Augustin <mitchell.augustin@canonical.com>

Rejected for the following reasons:
- see above

-Stefan

PS: also subscribe to the mailing list. If you tried it was not 
successful. There should be an automatic reply with a link to click. 
Check spam folder if it does not arrive.

> ---
>   scripts/pahole-flags.sh | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/pahole-flags.sh b/scripts/pahole-flags.sh
> index ac8b23787427..d38fa6d84d62 100755
> --- a/scripts/pahole-flags.sh
> +++ b/scripts/pahole-flags.sh
> @@ -4,7 +4,7 @@
>   extra_paholeopt=
>   
>   if ! [ -x "$(command -v ${PAHOLE})" ]; then
> -	return
> +	exit 0
>   fi
>   
>   pahole_ver=$($(dirname $0)/pahole-version.sh ${PAHOLE})
> @@ -17,4 +17,8 @@ if [ "${pahole_ver}" -ge "121" ]; then
>   	extra_paholeopt="${extra_paholeopt} --btf_gen_floats"
>   fi
>   
> +if [ "${pahole_ver}" -ge "124" ]; then
> +	extra_paholeopt="${extra_paholeopt} --skip_encoding_btf_enum64"
> +fi
> +
>   echo ${extra_paholeopt}
diff mbox series

Patch

diff --git a/scripts/pahole-flags.sh b/scripts/pahole-flags.sh
index ac8b23787427..d38fa6d84d62 100755
--- a/scripts/pahole-flags.sh
+++ b/scripts/pahole-flags.sh
@@ -4,7 +4,7 @@ 
 extra_paholeopt=
 
 if ! [ -x "$(command -v ${PAHOLE})" ]; then
-	return
+	exit 0
 fi
 
 pahole_ver=$($(dirname $0)/pahole-version.sh ${PAHOLE})
@@ -17,4 +17,8 @@  if [ "${pahole_ver}" -ge "121" ]; then
 	extra_paholeopt="${extra_paholeopt} --btf_gen_floats"
 fi
 
+if [ "${pahole_ver}" -ge "124" ]; then
+	extra_paholeopt="${extra_paholeopt} --skip_encoding_btf_enum64"
+fi
+
 echo ${extra_paholeopt}