lkml.org 
[lkml]   [2019]   [Feb]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCH 0/4] livepatch: Followup changes for the atomic replace patchset
    On Wed, Jan 16, 2019 at 05:17:16PM +0100, Petr Mladek wrote:
    > This patchset implements ideas that were mentioned and postponed during
    > the review of the atomic replace patchset. I hope that I did not miss
    > anything.
    >
    > Well, I did not add __used attribute to avoid non-static warnings
    > in modules for the selftest. The work on the sample modules somehow
    > stalled.
    >
    > BTW: Does it make sense to maintain the sample modules any longer?
    > We could point people to the modules used by the selftest instead.
    >
    >
    > The patches apply on top of livepatching.git, branch
    > origin/for-5.1/atomic-replace.
    >
    >
    > Petr Mladek (4):
    > livepatch: Introduce klp_for_each_patch macro
    > livepatch: Handle failing allocation of shadow variables in the
    > selftest
    > livepatch: Module coming and going callbacks can proceed all listed
    > patches
    > livepatch: Remove the redundant enabled flag in struct klp_patch
    >
    > include/linux/livepatch.h | 2 --
    > kernel/livepatch/core.c | 57 ++++++++++++++++--------------------
    > kernel/livepatch/core.h | 6 ++++
    > kernel/livepatch/transition.c | 9 +++---
    > kernel/livepatch/transition.h | 1 +
    > lib/livepatch/test_klp_shadow_vars.c | 8 ++---
    > 6 files changed, 40 insertions(+), 43 deletions(-)
    >
    > --
    > 2.13.7
    >

    Hi Petr,

    This change is trivial, but since folks are letting loose various static
    code analysers on the livepatch samples and selftests, could you add
    this to your patchset. The shadow variable selftest is happy with this
    change since it expects to see specific (non-negative) ptr_id values.

    Thanks,

    -- Joe

    -->8-- -->8-- -->8-- -->8-- -->8-- -->8-- -->8-- -->8-- -->8-- -->8--

    From e4c1e95a2145405115a984c6567740d12f20ccb7 Mon Sep 17 00:00:00 2001
    From: Joe Lawrence <joe.lawrence@redhat.com>
    Date: Fri, 1 Feb 2019 10:53:25 -0500
    Subject: [PATCH] livepatch: return -ENOMEM on ptr_id() allocation failure

    Fixes the following smatch warning:

    lib/livepatch/test_klp_shadow_vars.c:47 ptr_id() warn: returning -1 instead of -ENOMEM is sloppy

    Signed-off-by: Joe Lawrence <joe.lawrence@redhat.com>
    ---
    lib/livepatch/test_klp_shadow_vars.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/lib/livepatch/test_klp_shadow_vars.c b/lib/livepatch/test_klp_shadow_vars.c
    index 02f892f941dc..f5441c193166 100644
    --- a/lib/livepatch/test_klp_shadow_vars.c
    +++ b/lib/livepatch/test_klp_shadow_vars.c
    @@ -44,7 +44,7 @@ static int ptr_id(void *ptr)

    sp = kmalloc(sizeof(*sp), GFP_ATOMIC);
    if (!sp)
    - return -1;
    + return -ENOMEM;
    sp->ptr = ptr;
    sp->id = count++;

    --
    2.20.1
    \
     
     \ /
      Last update: 2019-02-01 17:03    [W:2.586 / U:0.160 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site