lkml.org 
[lkml]   [2018]   [Feb]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: PATCH v6 0/6] livepatch: Atomic replace feature
    On Thu, 1 Feb 2018, Joe Lawrence wrote:

    > On 02/01/2018 08:49 AM, Miroslav Benes wrote:
    > >
    > > Well, one more thing. I think there is a problem with shadow variables.
    > > Similar to callbacks situation. Shadow variables cannot be destroyed the
    > > way it is shown in our samples. Cumulative patches want to preserve
    > > everything as much as possible. If I'm right, it should be mentioned in
    > > the documentation.
    >
    > Are you talking about using klp_shadow_free_all() call in a module_exit
    > routine? Yeah, I think in this case, that responsibility would be
    > passed to the newly loaded cumulative patch, right?

    Yes, but we haven't got an option not to call it here (as with callbacks,
    where we can omit callbacks completely with atomic replace patches). A
    live patch author must be aware of this and use shadow variables
    appropriately.

    Miroslav

    \
     
     \ /
      Last update: 2018-02-01 16:08    [W:3.878 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site