This is v3 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 [1].
Per Miroslav's review, the changelog and title have been reworded to avoid implying this is livepatch-specific validation: the check runs during object initialization, which klp_module_coming() also triggers for a module still being loaded. Petr found that the error path in klp_init_object_loaded() -- taken by this new check, among others -- leaves stale object state behind on failure. His two patches (2/3, 3/3) fix that by routing all of klp_init_object_loaded()'s error returns through proper cleanup, and are included in this series since they touch the same code and would otherwise conflict with patch 1/3. [1] https://lore.kernel.org/all/[email protected]/ Harry Hsu (1): livepatch: Fail object initialization on duplicate patched function Petr Mladek (2): livepatch: Move code for updating livepatch object relocations livepatch: Clean up klp_init_object_loaded() when fails kernel/livepatch/core.c | 117 ++++++++++++++++++++++++---------------- 1 file changed, 71 insertions(+), 46 deletions(-) -- 2.43.0
