From: Peter Zijlstra <peterz@infradead.org>
To: Josh Poimboeuf <jpoimboe@kernel.org>
Cc: x86@kernel.org, linux-kernel@vger.kernel.org,
live-patching@vger.kernel.org,
Joe Lawrence <joe.lawrence@redhat.com>,
Song Liu <song@kernel.org>, Miroslav Benes <mbenes@suse.cz>,
Petr Mladek <pmladek@suse.com>
Subject: Re: [PATCH 45/48] x86/Kconfig: Enable CONFIG_PREFIX_SYMBOLS for FineIBT
Date: Thu, 23 Apr 2026 10:47:58 +0200 [thread overview]
Message-ID: <20260423084758.GY3126523@noisy.programming.kicks-ass.net> (raw)
In-Reply-To: <70107aab81b01f8a2360f052ff550a9e97c30f79.1776916871.git.jpoimboe@kernel.org>
On Wed, Apr 22, 2026 at 09:04:13PM -0700, Josh Poimboeuf wrote:
> PREFIX_SYMBOLS has a !CFI dependency because the compiler already
> generates __cfi_ prefix symbols for kCFI builds, so objtool-generated
> __pfx_ symbols were considered redundant.
>
> However, the __cfi_ symbols only cover the 5-byte kCFI type hash. With
> FUNCTION_CALL_PADDING, there are also 11 bytes of NOP padding between
> the hash and the function entry which have no symbol to claim them.
If you force the function alignment to 64 bytes, the prefix will also be
64bytes, rather than the normal 16.
> The NOPs can be rewritten with call depth tracking thunks at runtime.
> Without a symbol, unwinders and other tools that symbolize code
> locations misattribute those bytes.
>
> Remove the !CFI guard so objtool creates __pfx_ symbols for all
> CALL_PADDING configs, covering the full padding area regardless of
> whether there's also a __cfi_ symbol.
Egads, that a ton of symbols :/ Does it not make sense to 'fix' up the
__cfi_ symbols to cover the whole prefix?
next prev parent reply other threads:[~2026-04-23 8:48 UTC|newest]
Thread overview: 126+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-23 4:03 [PATCH 00/48] objtool/klp: Some klp-build fixes and improvements Josh Poimboeuf
2026-04-23 4:03 ` [PATCH 01/48] objtool/klp: Fix is_uncorrelated_static_local() for Clang Josh Poimboeuf
2026-04-23 18:45 ` Song Liu
2026-04-24 9:17 ` Miroslav Benes
2026-04-23 4:03 ` [PATCH 02/48] objtool/klp: Fix .data..once static local non-correlation Josh Poimboeuf
2026-04-23 18:54 ` Song Liu
2026-04-23 23:34 ` Josh Poimboeuf
2026-04-23 23:54 ` Song Liu
2026-04-24 9:21 ` Miroslav Benes
2026-04-23 4:03 ` [PATCH 03/48] objtool/klp: Don't correlate __ADDRESSABLE() symbols Josh Poimboeuf
2026-04-24 9:23 ` Miroslav Benes
2026-04-24 9:34 ` Miroslav Benes
2026-04-23 4:03 ` [PATCH 04/48] objtool/klp: Ignore __UNIQUE_ID_*() PCI stub functions Josh Poimboeuf
2026-04-23 19:05 ` Song Liu
2026-04-23 19:31 ` Josh Poimboeuf
2026-04-23 21:33 ` Song Liu
2026-04-23 23:50 ` Josh Poimboeuf
2026-04-23 23:54 ` Song Liu
2026-04-24 11:26 ` Miroslav Benes
2026-04-23 4:03 ` [PATCH 05/48] objtool: Move mark_rodata() to elf.c Josh Poimboeuf
2026-04-23 8:19 ` Peter Zijlstra
2026-04-24 11:36 ` Miroslav Benes
2026-04-24 20:41 ` Song Liu
2026-04-23 4:03 ` [PATCH 06/48] objtool/klp: Don't correlate rodata symbols Josh Poimboeuf
2026-04-24 11:54 ` Miroslav Benes
2026-04-23 4:03 ` [PATCH 07/48] objtool/klp: Don't correlate absolute symbols Josh Poimboeuf
2026-04-24 12:04 ` Miroslav Benes
2026-04-24 20:56 ` Song Liu
2026-04-23 4:03 ` [PATCH 08/48] objtool/klp: Don't correlate __initstub__ symbols Josh Poimboeuf
2026-04-24 12:17 ` Miroslav Benes
2026-04-24 20:57 ` Song Liu
2026-04-23 4:03 ` [PATCH 09/48] objtool/klp: Fix create_fake_symbols() skipping entsize-based sections Josh Poimboeuf
2026-04-24 21:00 ` Song Liu
2026-04-23 4:03 ` [PATCH 10/48] objtool/klp: Fix --debug-checksum for duplicate symbol names Josh Poimboeuf
2026-04-24 21:06 ` Song Liu
2026-04-23 4:03 ` [PATCH 11/48] objtool/klp: Fix handling of zero-length .altinstr_replacement sections Josh Poimboeuf
2026-04-24 21:19 ` Song Liu
2026-04-23 4:03 ` [PATCH 12/48] objtool/klp: Fix cloning of zero-length section symbols Josh Poimboeuf
2026-04-24 21:24 ` Song Liu
2026-04-23 4:03 ` [PATCH 13/48] objtool/klp: Fix XXH3 state memory leak Josh Poimboeuf
2026-04-24 21:28 ` Song Liu
2026-04-23 4:03 ` [PATCH 14/48] objtool/klp: Fix extraction of text annotations for alternatives Josh Poimboeuf
2026-04-24 21:37 ` Song Liu
2026-04-23 4:03 ` [PATCH 15/48] objtool/klp: Fix kCFI trap handling Josh Poimboeuf
2026-04-24 21:38 ` Song Liu
2026-04-23 4:03 ` [PATCH 16/48] objtool/klp: Fix relocation conversion failures for R_X86_64_NONE Josh Poimboeuf
2026-04-24 21:35 ` Song Liu
2026-04-23 4:03 ` [PATCH 17/48] objtool: Fix reloc hash collision in find_reloc_by_dest_range() Josh Poimboeuf
2026-04-23 8:32 ` Peter Zijlstra
2026-04-23 16:34 ` Josh Poimboeuf
2026-04-24 21:47 ` Song Liu
2026-04-23 4:03 ` [PATCH 18/48] klp-build: Fix hang on out-of-date .config Josh Poimboeuf
2026-04-24 21:51 ` Song Liu
2026-04-23 4:03 ` [PATCH 19/48] klp-build: Fix checksum comparison for changed offsets Josh Poimboeuf
2026-04-23 4:03 ` [PATCH 20/48] klp-build: Don't use errexit Josh Poimboeuf
2026-04-24 22:07 ` Song Liu
2026-04-23 4:03 ` [PATCH 21/48] klp-build: Validate patch file existence Josh Poimboeuf
2026-04-24 21:53 ` Song Liu
2026-04-23 4:03 ` [PATCH 22/48] klp-build: Suppress excessive fuzz output by default Josh Poimboeuf
2026-04-24 21:54 ` Song Liu
2026-04-23 4:03 ` [PATCH 23/48] klp-build: Fix patch cleanup on interrupt Josh Poimboeuf
2026-04-24 21:56 ` Song Liu
2026-04-23 4:03 ` [PATCH 24/48] klp-build: Reject patches to vDSO Josh Poimboeuf
2026-04-24 21:57 ` Song Liu
2026-04-23 4:03 ` [PATCH 25/48] klp-build: Reject patches to realmode Josh Poimboeuf
2026-04-24 21:57 ` Song Liu
2026-04-23 4:03 ` [PATCH 26/48] objtool/klp: Don't set sym->file for section symbols Josh Poimboeuf
2026-04-23 8:34 ` Peter Zijlstra
2026-04-24 21:58 ` Song Liu
2026-04-23 4:03 ` [PATCH 27/48] objtool: Include libsubcmd headers directly from source tree Josh Poimboeuf
2026-04-24 21:59 ` Song Liu
2026-04-23 4:03 ` [PATCH 28/48] objtool/klp: Create empty checksum sections for function-less object files Josh Poimboeuf
2026-04-24 22:08 ` Song Liu
2026-04-23 4:03 ` [PATCH 29/48] klp-build: Print "objtool klp diff" command in verbose mode Josh Poimboeuf
2026-04-24 22:02 ` Song Liu
2026-04-23 4:03 ` [PATCH 30/48] objtool/klp: Handle Clang .data..Lanon anonymous data sections Josh Poimboeuf
2026-04-24 22:09 ` Song Liu
2026-04-23 4:03 ` [PATCH 31/48] objtool: Add is_alias_sym() helper Josh Poimboeuf
2026-04-23 8:35 ` Peter Zijlstra
2026-04-24 22:10 ` Song Liu
2026-04-23 4:04 ` [PATCH 32/48] objtool: Add is_cold_func() helper Josh Poimboeuf
2026-04-23 8:38 ` Peter Zijlstra
2026-04-23 15:12 ` Josh Poimboeuf
2026-04-23 15:14 ` Peter Zijlstra
2026-04-23 19:23 ` Josh Poimboeuf
2026-04-23 4:04 ` [PATCH 33/48] objtool/klp: Extricate checksum calculation from validate_branch() Josh Poimboeuf
2026-04-24 22:14 ` Song Liu
2026-04-23 4:04 ` [PATCH 34/48] objtool: Consolidate file decoding into decode_file() Josh Poimboeuf
2026-04-23 8:41 ` Peter Zijlstra
2026-04-24 22:16 ` Song Liu
2026-04-23 4:04 ` [PATCH 35/48] objtool/klp: Add "objtool klp checksum" subcommand Josh Poimboeuf
2026-04-24 22:18 ` Song Liu
2026-04-23 4:04 ` [PATCH 36/48] klp-build: Use " Josh Poimboeuf
2026-04-24 22:24 ` Song Liu
2026-04-23 4:04 ` [PATCH 37/48] objtool/klp: Remove "objtool --checksum" Josh Poimboeuf
2026-04-24 22:25 ` Song Liu
2026-04-23 4:04 ` [PATCH 38/48] klp-build: Validate short-circuit prerequisites Josh Poimboeuf
2026-04-25 0:06 ` Song Liu
2026-04-23 4:04 ` [PATCH 39/48] objtool: Replace iterator callbacks with for_each_sym_by_*() Josh Poimboeuf
2026-04-25 0:04 ` Song Liu
2026-04-23 4:04 ` [PATCH 40/48] objtool/klp: Calculate object checksums Josh Poimboeuf
2026-04-23 4:04 ` [PATCH 41/48] objtool/klp: Rewrite symbol correlation algorithm Josh Poimboeuf
2026-04-25 0:53 ` Song Liu
2026-04-23 4:04 ` [PATCH 42/48] objtool/klp: Add correlation debugging output Josh Poimboeuf
2026-04-25 0:30 ` Song Liu
2026-04-23 4:04 ` [PATCH 43/48] objtool: Add insn_sym() helper Josh Poimboeuf
2026-04-23 8:45 ` Peter Zijlstra
2026-04-23 15:14 ` Josh Poimboeuf
2026-04-23 4:04 ` [PATCH 44/48] objtool/klp: Fix position-dependent checksums for non-relocated jumps/calls Josh Poimboeuf
2026-04-23 4:04 ` [PATCH 45/48] x86/Kconfig: Enable CONFIG_PREFIX_SYMBOLS for FineIBT Josh Poimboeuf
2026-04-23 8:47 ` Peter Zijlstra [this message]
2026-04-23 15:16 ` Josh Poimboeuf
2026-04-23 15:19 ` Peter Zijlstra
2026-04-23 16:23 ` Josh Poimboeuf
2026-04-23 16:29 ` Peter Zijlstra
2026-04-24 9:02 ` David Laight
2026-04-23 23:30 ` Josh Poimboeuf
2026-04-24 3:38 ` Josh Poimboeuf
2026-04-24 9:45 ` Peter Zijlstra
2026-04-23 4:04 ` [PATCH 46/48] objtool/klp: Make function prefix handling more generic Josh Poimboeuf
2026-04-23 4:04 ` [PATCH 47/48] objtool: Improve and simplify prefix symbol detection Josh Poimboeuf
2026-04-23 8:55 ` Peter Zijlstra
2026-04-23 15:19 ` Josh Poimboeuf
2026-04-23 15:21 ` Peter Zijlstra
2026-04-23 4:04 ` [PATCH 48/48] objtool/klp: Cache dont_correlate() result Josh Poimboeuf
2026-04-25 1:19 ` Song Liu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260423084758.GY3126523@noisy.programming.kicks-ass.net \
--to=peterz@infradead.org \
--cc=joe.lawrence@redhat.com \
--cc=jpoimboe@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=live-patching@vger.kernel.org \
--cc=mbenes@suse.cz \
--cc=pmladek@suse.com \
--cc=song@kernel.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox