On Thu, Jun 19, 2025 at 02:00:29PM +0200, Sergio González Collado wrote:

Hello,

Please consider applying the following commits for 6.12.y:

c104c16073b7 ("Kunit to check the longest symbol length")
f710202b2a45 ("x86/tools: Drop duplicate unlikely() definition in

insn_decoder_test.c")

They should apply cleanly.

Those two commits implement a kunit test to verify that a symbol with KSYM_NAME_LEN of 512 can be read.

The first commit implements the test. This commit also includes a fix for the test x86/insn_decoder_test. In the case a symbol exceeds the symbol length limit, an error will happen:

arch/x86/tools/insn_decoder_test: error: malformed line 1152000:
tBb_+0xf2>

..which overflowed by 10 characters reading this line:

ffffffff81458193:   74 3d                   je

ffffffff814581d2 <_RNvXse_NtNtNtCshGpAVYOtgW1_4core4iter8adapters7flattenINtB5_13FlattenCompatINtNtB7_3map3MapNtNtNtBb_3str4iter5CharsNtB1v_17CharEscapeDefaultENtNtBb_4char13EscapeDefaultENtNtBb_3fmt5Debug3fmtBb_+0xf2>

The fix was proposed in [1] and initially mentioned at [2].

The second commit fixes a warning when building with clang because there was a definition of unlikely from compiler.h in tools/include/linux, which conflicted with the one in the instruction decoder selftest.

[1] https://lore.kernel.org/lkml/Y9ES4UKl%2F+DtvAVS@gmail.com/ [2] https://lore.kernel.org/lkml/320c4dba-9919-404b-8a26-a8af16be1845@app.fastma...

Both now queued up, thanks.

greg k-h