lkml.org 
[lkml]   [2020]   [Jan]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[POC 14/23] livepatch: Never block livepatch modules when the related module is being removed
    Date
    Originally, it was not possible to remove the single livepatch module when
    the code was used during a forced transition. There were no guarantees that
    the code was no longer used.

    Even the split livepatch modules have to stay when the entire livepatch
    gets disabled/replaced and the livepatch modules were used during
    a forced transition.

    On the other hand, klp_module_going() callback is called when the patched
    module is about to be removed. It's code should no longer be used.
    The same should be true also for the related livepatch module. Therefore
    the livepatch module could always get removed as well.

    It allows to load the patched module again in the future. Otherwise,
    it would get blocked because the related livepatch module could not
    get loaded more times.

    Signed-off-by: Petr Mladek <pmladek@suse.com>
    ---
    kernel/livepatch/core.c | 7 +++++++
    1 file changed, 7 insertions(+)

    diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c
    index 73462b66f63f..4b55d805f3ec 100644
    --- a/kernel/livepatch/core.c
    +++ b/kernel/livepatch/core.c
    @@ -1391,6 +1391,13 @@ void klp_module_going(struct module *mod)
    klp_for_each_patch(patch) {
    klp_for_each_object_safe(patch, obj, tmp_obj) {
    if (obj->name && !strcmp(obj->name, mod->name)) {
    + /*
    + * The livepatched module is about to be
    + * destroyed. It's code is no longer used.
    + * Same is true for the livepatch even when
    + * it was part of forced transition.
    + */
    + obj->forced = false;
    klp_remove_object(obj);
    }
    }
    --
    2.16.4
    \
     
     \ /
      Last update: 2020-01-17 16:05    [W:4.240 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site