[PATCH 0/5] livepatch: Fail object initialization on duplicate patched function

Petr Mladek posted 5 patches 2 weeks, 3 days ago
Only 0 patches received!
There is a newer version of this series
kernel/livepatch/core.c                       | 137 ++++++++++++------
tools/testing/selftests/livepatch/Makefile    |   3 +-
.../testing/selftests/livepatch/test-alias.sh |  81 +++++++++++
.../selftests/livepatch/test_modules/Makefile |   4 +-
.../test_modules/test_klp_alias_patch.c       |  62 ++++++++
.../test_modules/test_klp_alias_target.c      |  48 ++++++
6 files changed, 285 insertions(+), 50 deletions(-)
create mode 100755 tools/testing/selftests/livepatch/test-alias.sh
create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_patch.c
create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_target.c
[PATCH 0/5] livepatch: Fail object initialization on duplicate patched function
Posted by Petr Mladek 2 weeks, 3 days ago
This is v4 of the patch failing object initialization when it patches
the same underlying function twice through two aliased symbols
(functions sharing one address) in the same object.

Changes against [v3]:

  + Add selftest for livepatching aliased symbols [Song, Harry]

  + Only partially revert relocations when failing inside
    klp_apply_object_relocs() [Sashiko]

  + Cleanup changes made by klp_init_object_loaded() for failing
    patch on another failures in klp_module_coming() [Sashiko]

Changes against [v2]:

  + Clarify comments and the commit message [Miroslav]

  + Cleanup when klp_init_object_loaded() fails [Sashiko]


[v2] https://lore.kernel.org/r/20260823060734.58443-1-x90613@gmail.com
[v3] https://lore.kernel.org/r/20260830173343.52759-1-x90613@gmail.com


Harry Hsu (2):
  livepatch: Fail object initialization on duplicate patched function
  selftests/livepatch: Test rejection of aliased symbols in one object

Petr Mladek (3):
  livepatch: Move code for updating livepatch object relocations
  livepatch: Clear relocations when klp_apply_object_relocs() fails
  livepatch: Clean up klp_init_object_loaded() when fails

 kernel/livepatch/core.c                       | 137 ++++++++++++------
 tools/testing/selftests/livepatch/Makefile    |   3 +-
 .../testing/selftests/livepatch/test-alias.sh |  81 +++++++++++
 .../selftests/livepatch/test_modules/Makefile |   4 +-
 .../test_modules/test_klp_alias_patch.c       |  62 ++++++++
 .../test_modules/test_klp_alias_target.c      |  48 ++++++
 6 files changed, 285 insertions(+), 50 deletions(-)
 create mode 100755 tools/testing/selftests/livepatch/test-alias.sh
 create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_patch.c
 create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_target.c

-- 
2.55.0
Re: [PATCH 0/5] livepatch: Fail object initialization on duplicate patched function
Posted by Petr Mladek 2 weeks, 3 days ago
Hi there,

please, ignore this mail. I am re-sending the series with v4 in the
subject. I am sorry for rumor.

Best Regards,
Petr

On Tue 2026-09-08 14:01:44, Petr Mladek wrote:
> This is v4 of the patch failing object initialization when it patches
> the same underlying function twice through two aliased symbols
> (functions sharing one address) in the same object.
> 
> Changes against [v3]:
> 
>   + Add selftest for livepatching aliased symbols [Song, Harry]
> 
>   + Only partially revert relocations when failing inside
>     klp_apply_object_relocs() [Sashiko]
> 
>   + Cleanup changes made by klp_init_object_loaded() for failing
>     patch on another failures in klp_module_coming() [Sashiko]
> 
> Changes against [v2]:
> 
>   + Clarify comments and the commit message [Miroslav]
> 
>   + Cleanup when klp_init_object_loaded() fails [Sashiko]
> 
> 
> [v2] https://lore.kernel.org/r/20260823060734.58443-1-x90613@gmail.com
> [v3] https://lore.kernel.org/r/20260830173343.52759-1-x90613@gmail.com
> 
> 
> Harry Hsu (2):
>   livepatch: Fail object initialization on duplicate patched function
>   selftests/livepatch: Test rejection of aliased symbols in one object
> 
> Petr Mladek (3):
>   livepatch: Move code for updating livepatch object relocations
>   livepatch: Clear relocations when klp_apply_object_relocs() fails
>   livepatch: Clean up klp_init_object_loaded() when fails
> 
>  kernel/livepatch/core.c                       | 137 ++++++++++++------
>  tools/testing/selftests/livepatch/Makefile    |   3 +-
>  .../testing/selftests/livepatch/test-alias.sh |  81 +++++++++++
>  .../selftests/livepatch/test_modules/Makefile |   4 +-
>  .../test_modules/test_klp_alias_patch.c       |  62 ++++++++
>  .../test_modules/test_klp_alias_target.c      |  48 ++++++
>  6 files changed, 285 insertions(+), 50 deletions(-)
>  create mode 100755 tools/testing/selftests/livepatch/test-alias.sh
>  create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_patch.c
>  create mode 100644 tools/testing/selftests/livepatch/test_modules/test_klp_alias_target.c
> 
> -- 
> 2.55.0