[PATCH v3 0/9] objtool/klp: sympos/module/alternative/etc fixes

Josh Poimboeuf posted 9 patches 1 month, 3 weeks ago
tools/objtool/arch/x86/special.c        |  27 ++++++
tools/objtool/include/objtool/klp.h     |  10 ++-
tools/objtool/include/objtool/special.h |   7 ++
tools/objtool/klp-diff.c                | 106 ++++++++++++++++++++----
tools/objtool/klp-post-link.c           |  53 +++++++-----
tools/objtool/klp-symid.c               |   1 +
tools/objtool/klp-sympos.c              |  10 +++
7 files changed, 175 insertions(+), 39 deletions(-)
[PATCH v3 0/9] objtool/klp: sympos/module/alternative/etc fixes
Posted by Josh Poimboeuf 1 month, 3 weeks ago
v3:
- remove "size = 0" and add comments to create_fake_symbols()
- add patches 8 & 9

v2: https://lore.kernel.org/cover.1785939903.git.jpoimboe@kernel.org
- rebased on tip/master (first 6 patches were merged)
- dropped original patches 7-8 (will post followup)
- added .klp.symid fix

v1: https://lore.kernel.org/cover.1785727106.git.jpoimboe@kernel.org

This consolidates fixes for the klp-build issues reported by Joe over
the last several weeks, plus some more things I found while
testing/reviewing.

Joe Lawrence (1):
  objtool/klp: Allow new references to module exports

Josh Poimboeuf (8):
  objtool/klp: Fix vmlinux .klp.symid link error for .no_trim_symbol
    symbols
  objtool/klp: Fix size of empty special section entries
  objtool/klp: Ignore replacement offset of empty x86 alternatives
  objtool/klp: Explicitly disallow patching or referencing init
    code/data
  objtool/klp: Fix cross-module klp relocation section naming
  objtool/klp: Don't match local symbols against exports
  objtool/klp: Fix relocations for EXPORT_SYMBOL_FOR_MODULES() symbols
  objtool/klp: Fix line numbers in Module.symvers parse errors

 tools/objtool/arch/x86/special.c        |  27 ++++++
 tools/objtool/include/objtool/klp.h     |  10 ++-
 tools/objtool/include/objtool/special.h |   7 ++
 tools/objtool/klp-diff.c                | 106 ++++++++++++++++++++----
 tools/objtool/klp-post-link.c           |  53 +++++++-----
 tools/objtool/klp-symid.c               |   1 +
 tools/objtool/klp-sympos.c              |  10 +++
 7 files changed, 175 insertions(+), 39 deletions(-)

-- 
2.54.0
Re: [PATCH v3 0/9] objtool/klp: sympos/module/alternative/etc fixes
Posted by Joe Lawrence 1 month, 2 weeks ago
On 8/7/26 5:37 PM, Josh Poimboeuf wrote:
> v3:
> - remove "size = 0" and add comments to create_fake_symbols()
> - add patches 8 & 9
> 
> v2: https://lore.kernel.org/cover.1785939903.git.jpoimboe@kernel.org
> - rebased on tip/master (first 6 patches were merged)
> - dropped original patches 7-8 (will post followup)
> - added .klp.symid fix
> 
> v1: https://lore.kernel.org/cover.1785727106.git.jpoimboe@kernel.org
> 
> This consolidates fixes for the klp-build issues reported by Joe over
> the last several weeks, plus some more things I found while
> testing/reviewing.
> 
> Joe Lawrence (1):
>   objtool/klp: Allow new references to module exports
> 
> Josh Poimboeuf (8):
>   objtool/klp: Fix vmlinux .klp.symid link error for .no_trim_symbol
>     symbols
>   objtool/klp: Fix size of empty special section entries
>   objtool/klp: Ignore replacement offset of empty x86 alternatives
>   objtool/klp: Explicitly disallow patching or referencing init
>     code/data
>   objtool/klp: Fix cross-module klp relocation section naming
>   objtool/klp: Don't match local symbols against exports
>   objtool/klp: Fix relocations for EXPORT_SYMBOL_FOR_MODULES() symbols
>   objtool/klp: Fix line numbers in Module.symvers parse errors
> 
>  tools/objtool/arch/x86/special.c        |  27 ++++++
>  tools/objtool/include/objtool/klp.h     |  10 ++-
>  tools/objtool/include/objtool/special.h |   7 ++
>  tools/objtool/klp-diff.c                | 106 ++++++++++++++++++++----
>  tools/objtool/klp-post-link.c           |  53 +++++++-----
>  tools/objtool/klp-symid.c               |   1 +
>  tools/objtool/klp-sympos.c              |  10 +++
>  7 files changed, 175 insertions(+), 39 deletions(-)
> 

For the series:
Acked-by: Joe Lawrence <joe.lawrence@redhat.com>

Previous test cases from the earlier parent patchsets and reports run
positive now.

Thanks,
-- 
Joe