mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/5] livepatch: Fail object initialization on duplicate patched function
@ 2026-09-08 12:01 Petr Mladek
  2026-09-08 12:05 ` Petr Mladek
  0 siblings, 1 reply; 2+ messages in thread
From: Petr Mladek @ 2026-09-08 12:01 UTC (permalink / raw)
  To: Harry Hsu, jpoimboe, mbenes, joe.lawrence
  Cc: jikos, live-patching, shuah, song, linux-kernel, Petr Mladek

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


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 0/5] livepatch: Fail object initialization on duplicate patched function
  2026-09-08 12:01 [PATCH 0/5] livepatch: Fail object initialization on duplicate patched function Petr Mladek
@ 2026-09-08 12:05 ` Petr Mladek
  0 siblings, 0 replies; 2+ messages in thread
From: Petr Mladek @ 2026-09-08 12:05 UTC (permalink / raw)
  To: Harry Hsu, jpoimboe, mbenes, joe.lawrence
  Cc: jikos, live-patching, shuah, song, linux-kernel

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-09-08 12:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-08 12:01 [PATCH 0/5] livepatch: Fail object initialization on duplicate patched function Petr Mladek
2026-09-08 12:05 ` Petr Mladek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®