Linus,
Please pull the latest objtool/core git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git objtool-core-2022-05-23
# HEAD: 22682a07acc308ef78681572e19502ce8893c4d4 objtool: Fix objtool regression on x32 systems
Objtool changes for this cycle were:
- Comprehensive interface overhaul:
=================================
Objtool's interface has some issues:
- Several features are done unconditionally, without any way to turn
them off. Some of them might be surprising. This makes objtool
tricky to use, and prevents porting individual features to other
arches.
- The config dependencies are too coarse-grained. Objtool enablement is
tied to CONFIG_STACK_VALIDATION, but it has several other features
independent of that.
- The objtool subcmds ("check" and "orc") are clumsy: "check" is really
a subset of "orc", so it has all the same options. The subcmd model
has never really worked for objtool, as it only has a single purpose:
"do some combination of things on an object file".
- The '--lto' and '--vmlinux' options are nonsensical and have
surprising behavior.
Overhaul the interface:
- get rid of subcmds
- make all features individually selectable
- remove and/or clarify confusing/obsolete options
- update the documentation
- fix some bugs found along the way
- Fix x32 regression
- Fix Kbuild cleanup bugs
- Add scripts/objdump-func helper script to disassemble a single function from an object file.
- Rewrite scripts/faddr2line to be section-aware, by basing it on 'readelf',
moving it away from 'nm', which doesn't handle multiple sections well,
which can result in decoding failure.
- Rewrite & fix symbol handling - which had a number of bugs wrt. object files
that don't have global symbols - which is rare but possible. Also fix a
bunch of symbol handling bugs found along the way.
Signed-off-by: Ingo Molnar <[email protected]>
Thanks,
Ingo
------------------>
Borislav Petkov (1):
x86/cpu: Load microcode during restore_processor_state()
Dmitry Monakhov (1):
x86/unwind/orc: Recheck address range after stack info was updated
Josh Poimboeuf (27):
MAINTAINERS: Add x86 unwinding entry
objtool: Enable unreachable warnings for CLANG LTO
x86/static_call: Add ANNOTATE_NOENDBR to static call trampoline
x86/retpoline: Add ANNOTATE_NOENDBR for retpolines
x86/uaccess: Add ENDBR to __put_user_nocheck*()
x86/xen: Add ANNOTATE_NOENDBR to startup_xen()
objtool: Print data address for "!ENDBR" data warnings
objtool: Use offstr() to print address of missing ENDBR
libsubcmd: Fix OPTION_GROUP sorting
objtool: Reorganize cmdline options
objtool: Ditch subcommands
objtool: Don't print parentheses in function addresses
objtool: Add option to print section addresses
objtool: Make stack validation optional
objtool: Rework ibt and extricate from stack validation
objtool: Extricate sls from stack validation
objtool: Add CONFIG_OBJTOOL
objtool: Make stack validation frame-pointer-specific
objtool: Make static call annotation optional
objtool: Make jump label hack optional
objtool: Make noinstr hacks optional
objtool: Rename "VMLINUX_VALIDATION" -> "NOINSTR_VALIDATION"
objtool: Add HAVE_NOINSTR_VALIDATION
objtool: Remove --lto and --vmlinux in favor of --link
objtool: Update documentation
scripts: Create objdump-func helper script
scripts/faddr2line: Fix overlapping text section failures
Mikulas Patocka (1):
objtool: Fix objtool regression on x32 systems
Nur Hussein (1):
x86/Kconfig: fix the spelling of 'becoming' in X86_KERNEL_IBT config
Peter Zijlstra (5):
lib/strn*,objtool: Enforce user_access_begin() rules
x86,xen,objtool: Add UNWIND hint
x86,objtool: Mark cpu_startup_entry() __noreturn
x86,objtool: Explicitly mark idtentry_body()s tail REACHABLE
objtool: Fix symbol creation
Tiezhu Yang (2):
objtool: Remove inat-tables.c when make clean
objtool: Remove libsubcmd.a when make clean
Tony Luck (1):
x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
MAINTAINERS | 9 +
Makefile | 2 +-
arch/Kconfig | 18 +-
arch/x86/Kconfig | 23 +-
arch/x86/Kconfig.debug | 2 +-
arch/x86/entry/entry_64.S | 3 +
arch/x86/include/asm/intel-family.h | 3 +
arch/x86/include/asm/jump_label.h | 6 +-
arch/x86/include/asm/microcode.h | 2 +
arch/x86/include/asm/static_call.h | 1 +
arch/x86/kernel/alternative.c | 6 +-
arch/x86/kernel/cpu/microcode/core.c | 6 +-
arch/x86/kernel/unwind_orc.c | 8 +-
arch/x86/lib/putuser.S | 4 +
arch/x86/lib/retpoline.S | 2 +-
arch/x86/platform/pvh/head.S | 1 +
arch/x86/power/cpu.c | 10 +-
arch/x86/xen/xen-head.S | 1 +
include/linux/compiler.h | 6 +-
include/linux/cpu.h | 2 +-
include/linux/instrumentation.h | 6 +-
include/linux/objtool.h | 6 +-
kernel/trace/Kconfig | 1 +
lib/Kconfig.debug | 22 +-
lib/Kconfig.kcsan | 4 +-
lib/Kconfig.ubsan | 2 +-
lib/strncpy_from_user.c | 2 +-
lib/strnlen_user.c | 2 +-
scripts/Makefile.build | 23 +-
scripts/faddr2line | 150 ++++---
scripts/link-vmlinux.sh | 66 +--
scripts/objdump-func | 29 ++
scripts/package/builddeb | 2 +-
tools/include/linux/objtool.h | 6 +-
tools/lib/subcmd/parse-options.c | 17 +-
tools/objtool/Build | 12 +-
.../{stack-validation.txt => objtool.txt} | 122 +++++-
tools/objtool/Makefile | 10 +-
tools/objtool/arch/x86/decode.c | 2 +-
tools/objtool/arch/x86/special.c | 2 +-
tools/objtool/builtin-check.c | 156 +++++--
tools/objtool/builtin-orc.c | 73 ----
tools/objtool/check.c | 466 +++++++++++----------
tools/objtool/elf.c | 211 ++++++----
tools/objtool/include/objtool/builtin.h | 34 +-
tools/objtool/include/objtool/elf.h | 16 +-
tools/objtool/include/objtool/warn.h | 35 +-
tools/objtool/objtool.c | 103 +----
tools/objtool/weak.c | 9 +-
49 files changed, 1004 insertions(+), 700 deletions(-)
create mode 100755 scripts/objdump-func
rename tools/objtool/Documentation/{stack-validation.txt => objtool.txt} (80%)
delete mode 100644 tools/objtool/builtin-orc.c
* Ingo Molnar <[email protected]> wrote:
> Linus,
>
> Please pull the latest objtool/core git tree from:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git objtool-core-2022-05-23
>
> # HEAD: 22682a07acc308ef78681572e19502ce8893c4d4 objtool: Fix objtool regression on x32 systems
>
> Objtool changes for this cycle were:
>
> - Comprehensive interface overhaul:
> =================================
>
> Objtool's interface has some issues:
>
> - Several features are done unconditionally, without any way to turn
> them off. Some of them might be surprising. This makes objtool
> tricky to use, and prevents porting individual features to other
> arches.
>
> - The config dependencies are too coarse-grained. Objtool enablement is
> tied to CONFIG_STACK_VALIDATION, but it has several other features
> independent of that.
>
> - The objtool subcmds ("check" and "orc") are clumsy: "check" is really
> a subset of "orc", so it has all the same options. The subcmd model
> has never really worked for objtool, as it only has a single purpose:
> "do some combination of things on an object file".
>
> - The '--lto' and '--vmlinux' options are nonsensical and have
> surprising behavior.
>
> Overhaul the interface:
>
> - get rid of subcmds
>
> - make all features individually selectable
>
> - remove and/or clarify confusing/obsolete options
>
> - update the documentation
>
> - fix some bugs found along the way
>
> - Fix x32 regression
>
> - Fix Kbuild cleanup bugs
>
> - Add scripts/objdump-func helper script to disassemble a single function from an object file.
>
> - Rewrite scripts/faddr2line to be section-aware, by basing it on 'readelf',
> moving it away from 'nm', which doesn't handle multiple sections well,
> which can result in decoding failure.
>
> - Rewrite & fix symbol handling - which had a number of bugs wrt. object files
> that don't have global symbols - which is rare but possible. Also fix a
> bunch of symbol handling bugs found along the way.
>
> Signed-off-by: Ingo Molnar <[email protected]>
> Thanks,
>
> Ingo
>
> ------------------>
> Borislav Petkov (1):
> x86/cpu: Load microcode during restore_processor_state()
>
> Dmitry Monakhov (1):
> x86/unwind/orc: Recheck address range after stack info was updated
>
> Josh Poimboeuf (27):
> MAINTAINERS: Add x86 unwinding entry
> objtool: Enable unreachable warnings for CLANG LTO
> x86/static_call: Add ANNOTATE_NOENDBR to static call trampoline
> x86/retpoline: Add ANNOTATE_NOENDBR for retpolines
> x86/uaccess: Add ENDBR to __put_user_nocheck*()
> x86/xen: Add ANNOTATE_NOENDBR to startup_xen()
> objtool: Print data address for "!ENDBR" data warnings
> objtool: Use offstr() to print address of missing ENDBR
> libsubcmd: Fix OPTION_GROUP sorting
> objtool: Reorganize cmdline options
> objtool: Ditch subcommands
> objtool: Don't print parentheses in function addresses
> objtool: Add option to print section addresses
> objtool: Make stack validation optional
> objtool: Rework ibt and extricate from stack validation
> objtool: Extricate sls from stack validation
> objtool: Add CONFIG_OBJTOOL
> objtool: Make stack validation frame-pointer-specific
> objtool: Make static call annotation optional
> objtool: Make jump label hack optional
> objtool: Make noinstr hacks optional
> objtool: Rename "VMLINUX_VALIDATION" -> "NOINSTR_VALIDATION"
> objtool: Add HAVE_NOINSTR_VALIDATION
> objtool: Remove --lto and --vmlinux in favor of --link
> objtool: Update documentation
> scripts: Create objdump-func helper script
> scripts/faddr2line: Fix overlapping text section failures
>
> Mikulas Patocka (1):
> objtool: Fix objtool regression on x32 systems
>
> Nur Hussein (1):
> x86/Kconfig: fix the spelling of 'becoming' in X86_KERNEL_IBT config
>
> Peter Zijlstra (5):
> lib/strn*,objtool: Enforce user_access_begin() rules
> x86,xen,objtool: Add UNWIND hint
> x86,objtool: Mark cpu_startup_entry() __noreturn
> x86,objtool: Explicitly mark idtentry_body()s tail REACHABLE
> objtool: Fix symbol creation
>
> Tiezhu Yang (2):
> objtool: Remove inat-tables.c when make clean
> objtool: Remove libsubcmd.a when make clean
>
> Tony Luck (1):
> x86/cpu: Add new Alderlake and Raptorlake CPU model numbers
>
>
> MAINTAINERS | 9 +
> Makefile | 2 +-
> arch/Kconfig | 18 +-
> arch/x86/Kconfig | 23 +-
> arch/x86/Kconfig.debug | 2 +-
> arch/x86/entry/entry_64.S | 3 +
> arch/x86/include/asm/intel-family.h | 3 +
> arch/x86/include/asm/jump_label.h | 6 +-
> arch/x86/include/asm/microcode.h | 2 +
> arch/x86/include/asm/static_call.h | 1 +
> arch/x86/kernel/alternative.c | 6 +-
> arch/x86/kernel/cpu/microcode/core.c | 6 +-
> arch/x86/kernel/unwind_orc.c | 8 +-
> arch/x86/lib/putuser.S | 4 +
> arch/x86/lib/retpoline.S | 2 +-
> arch/x86/platform/pvh/head.S | 1 +
> arch/x86/power/cpu.c | 10 +-
> arch/x86/xen/xen-head.S | 1 +
> include/linux/compiler.h | 6 +-
> include/linux/cpu.h | 2 +-
> include/linux/instrumentation.h | 6 +-
> include/linux/objtool.h | 6 +-
> kernel/trace/Kconfig | 1 +
> lib/Kconfig.debug | 22 +-
> lib/Kconfig.kcsan | 4 +-
> lib/Kconfig.ubsan | 2 +-
> lib/strncpy_from_user.c | 2 +-
> lib/strnlen_user.c | 2 +-
> scripts/Makefile.build | 23 +-
> scripts/faddr2line | 150 ++++---
> scripts/link-vmlinux.sh | 66 +--
> scripts/objdump-func | 29 ++
> scripts/package/builddeb | 2 +-
> tools/include/linux/objtool.h | 6 +-
> tools/lib/subcmd/parse-options.c | 17 +-
> tools/objtool/Build | 12 +-
> .../{stack-validation.txt => objtool.txt} | 122 +++++-
> tools/objtool/Makefile | 10 +-
> tools/objtool/arch/x86/decode.c | 2 +-
> tools/objtool/arch/x86/special.c | 2 +-
> tools/objtool/builtin-check.c | 156 +++++--
> tools/objtool/builtin-orc.c | 73 ----
> tools/objtool/check.c | 466 +++++++++++----------
> tools/objtool/elf.c | 211 ++++++----
> tools/objtool/include/objtool/builtin.h | 34 +-
> tools/objtool/include/objtool/elf.h | 16 +-
> tools/objtool/include/objtool/warn.h | 35 +-
> tools/objtool/objtool.c | 103 +----
> tools/objtool/weak.c | 9 +-
> 49 files changed, 1004 insertions(+), 700 deletions(-)
> create mode 100755 scripts/objdump-func
> rename tools/objtool/Documentation/{stack-validation.txt => objtool.txt} (80%)
> delete mode 100644 tools/objtool/builtin-orc.c
Not sure what happened there, a previous x86/urgent got mixed into the
shortlog & diffstat - this is the correct shortlog & diffstat you should
get when pulling objtool-core-2022-05-23:
Josh Poimboeuf (19):
libsubcmd: Fix OPTION_GROUP sorting
objtool: Reorganize cmdline options
objtool: Ditch subcommands
objtool: Don't print parentheses in function addresses
objtool: Add option to print section addresses
objtool: Make stack validation optional
objtool: Rework ibt and extricate from stack validation
objtool: Extricate sls from stack validation
objtool: Add CONFIG_OBJTOOL
objtool: Make stack validation frame-pointer-specific
objtool: Make static call annotation optional
objtool: Make jump label hack optional
objtool: Make noinstr hacks optional
objtool: Rename "VMLINUX_VALIDATION" -> "NOINSTR_VALIDATION"
objtool: Add HAVE_NOINSTR_VALIDATION
objtool: Remove --lto and --vmlinux in favor of --link
objtool: Update documentation
scripts: Create objdump-func helper script
scripts/faddr2line: Fix overlapping text section failures
Mikulas Patocka (1):
objtool: Fix objtool regression on x32 systems
Peter Zijlstra (2):
Merge branch 'tip/x86/urgent'
objtool: Fix symbol creation
Tiezhu Yang (2):
objtool: Remove inat-tables.c when make clean
objtool: Remove libsubcmd.a when make clean
================>
Makefile | 2 +-
arch/Kconfig | 18 ++++++-
arch/x86/Kconfig | 21 +++++---
arch/x86/Kconfig.debug | 2 +-
arch/x86/include/asm/jump_label.h | 6 +--
arch/x86/kernel/alternative.c | 6 +--
include/linux/compiler.h | 6 +--
include/linux/instrumentation.h | 6 +--
include/linux/objtool.h | 6 +--
kernel/trace/Kconfig | 1 +
lib/Kconfig.debug | 22 ++++----
lib/Kconfig.kcsan | 4 +-
lib/Kconfig.ubsan | 2 +-
scripts/Makefile.build | 23 +++++----
scripts/faddr2line | 150 +++++++++++++++++++++++++++++++++++-------------------
scripts/link-vmlinux.sh | 66 +++++++++++++++---------
scripts/objdump-func | 29 +++++++++++
scripts/package/builddeb | 2 +-
tools/include/linux/objtool.h | 6 +--
tools/lib/subcmd/parse-options.c | 17 +++++--
tools/objtool/Build | 12 ++---
tools/objtool/Documentation/{stack-validation.txt => objtool.txt} | 122 ++++++++++++++++++++++++++++++++++++--------
tools/objtool/Makefile | 10 ++--
tools/objtool/arch/x86/decode.c | 2 +-
tools/objtool/arch/x86/special.c | 2 +-
tools/objtool/builtin-check.c | 156 +++++++++++++++++++++++++++++++++++++++++++++-----------
tools/objtool/builtin-orc.c | 73 --------------------------
tools/objtool/check.c | 463 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------------------------------------
tools/objtool/elf.c | 211 ++++++++++++++++++++++++++++++++++++++++++++++++---------------------------
tools/objtool/include/objtool/builtin.h | 34 +++++++++++--
tools/objtool/include/objtool/elf.h | 16 ++++--
tools/objtool/include/objtool/warn.h | 35 +++++++------
tools/objtool/objtool.c | 103 ++-----------------------------------
tools/objtool/weak.c | 9 +---
34 files changed, 959 insertions(+), 684 deletions(-)
create mode 100755 scripts/objdump-func
rename tools/objtool/Documentation/{stack-validation.txt => objtool.txt} (80%)
delete mode 100644 tools/objtool/builtin-orc.c
Thanks,
Ingo
* Ingo Molnar <[email protected]> wrote:
> 34 files changed, 959 insertions(+), 684 deletions(-)
Note that with your latest tree you'll get 3 new conflicts:
CONFLICT (content): Merge conflict in scripts/link-vmlinux.sh
CONFLICT (content): Merge conflict in scripts/Makefile.build
CONFLICT (content): Merge conflict in arch/x86/Kconfig
If you want to double check, you can find our resolution in tip:x86/merge:
a1df7cc57786 Merge branch 'objtool/core' into x86/merge, to resolve conflicts
At:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/merge # INTERNAL MERGE TREE, DO NOT PULL
Here's the merge description:
===================
The following commits between objtool/core and various x86 branches conflict:
968b493173ac x86/mm: Make DMA memory shared for TD guest
77a512e35db7 x86/boot: Avoid #VE during boot for TDX platforms
65fab5bc033a x86/tdx: Exclude shared bit from __PHYSICAL_MASK
41394e33f3a0 x86/tdx: Extend the confidential computing API to support TDX guests
59bd54a84d15 x86/tdx: Detect running as a TDX guest in early boot
dbae0a934f09 x86/cpu: Remove CONFIG_X86_SMAP and "nosmap"
and:
753da4179d08 objtool: Remove --lto and --vmlinux in favor of --link
489e355b4225 objtool: Add HAVE_NOINSTR_VALIDATION
0f620cefd775 objtool: Rename "VMLINUX_VALIDATION" -> "NOINSTR_VALIDATION"
22102f4559be objtool: Make noinstr hacks optional
4ab7674f5951 objtool: Make jump label hack optional
26e176896a5b objtool: Make static call annotation optional
720644749647 objtool: Make stack validation frame-pointer-specific
03f16cd020eb objtool: Add CONFIG_OBJTOOL
7dce62041ac3 objtool: Make stack validation optional
b51277eb9775 objtool: Ditch subcommands
2daf7faba7de objtool: Reorganize cmdline options
4cdfc11b2836 x86/Kconfig: fix the spelling of 'becoming' in X86_KERNEL_IBT config
4a5de9b76fcb objtool: Enable unreachable warnings for CLANG LTO
Resolve them:
- overlapping modifications
- modifying removed functionality
Conflicts:
arch/x86/Kconfig
scripts/Makefile.build
scripts/link-vmlinux.sh
===================
Thanks,
Ingo
On Tue, May 24, 2022 at 10:53:40AM -0700, Linus Torvalds wrote:
> On Mon, May 23, 2022 at 11:02 PM Ingo Molnar <[email protected]> wrote:
> >
> > Note that with your latest tree you'll get 3 new conflicts:
>
> No problem, but the conflicts did make something clear: the objtool
> code should just get rid of the "--uaccess" flag that is now
> unconditional when CONFIG_X86_SMAP has been removed.
>
> I didn't actually do that, and instead just did the mindless merge
> conflict resolution, but it might be a good idea.
Yes, I think there is patch pending for that, but given the total merge
pain we already had, I was waiting for all the dust to settle before
taking it further.
On Mon, May 23, 2022 at 11:02 PM Ingo Molnar <[email protected]> wrote:
>
> Note that with your latest tree you'll get 3 new conflicts:
No problem, but the conflicts did make something clear: the objtool
code should just get rid of the "--uaccess" flag that is now
unconditional when CONFIG_X86_SMAP has been removed.
I didn't actually do that, and instead just did the mindless merge
conflict resolution, but it might be a good idea.
See commit dbae0a934f09 ("x86/cpu: Remove CONFIG_X86_SMAP and
'nosmap'") for when it was removed.
Josh? The patch *might* be something like the attached, but this is
(a) untested and (b) that 'opts.noinstr' part of the patch is a bit
dodgy (ie I made the previous 'if' unconditional, but then changed the
follow 'else if ()' to just 'if ()' instead of deleting it, which is
what "uaccess is always set" would technically have done.
Again: I did *not* do this as part of the merge, the attached patch is
just a suggestion of something that I think should now be done after
the merge.
Hmm?
Linus
On Tue, May 24, 2022 at 08:08:05PM +0200, Peter Zijlstra wrote:
> On Tue, May 24, 2022 at 10:53:40AM -0700, Linus Torvalds wrote:
> > On Mon, May 23, 2022 at 11:02 PM Ingo Molnar <[email protected]> wrote:
> > >
> > > Note that with your latest tree you'll get 3 new conflicts:
> >
> > No problem, but the conflicts did make something clear: the objtool
> > code should just get rid of the "--uaccess" flag that is now
> > unconditional when CONFIG_X86_SMAP has been removed.
> >
> > I didn't actually do that, and instead just did the mindless merge
> > conflict resolution, but it might be a good idea.
>
> Yes, I think there is patch pending for that, but given the total merge
> pain we already had, I was waiting for all the dust to settle before
> taking it further.
Actually I think we should go the other direction by keeping '--uaccess'
and adding CONFIG_HAVE_UACCESS_VALIDATION or so. The main point of
that objtool rewrite was to make all the features modular so other
arches could port them piecemeal.
--
Josh
The pull request you sent on Mon, 23 May 2022 18:32:36 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git objtool-core-2022-05-23
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/22922deae13fc8d3769790c2eb388e9afce9771d
Thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html