Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1543565 > unrolled thread

Re: [PATCH v3 08/15] livepatch: separate enabled and patched states

Started byPetr Mladek <pmladek@suse.com>
First post2016-12-16 17:30 +0100
Last post2016-12-16 17:30 +0100
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 08/15] livepatch: separate enabled and patched states Petr Mladek <pmladek@suse.com> - 2016-12-16 17:30 +0100

#1543565 — Re: [PATCH v3 08/15] livepatch: separate enabled and patched states

FromPetr Mladek <pmladek@suse.com>
Date2016-12-16 17:30 +0100
SubjectRe: [PATCH v3 08/15] livepatch: separate enabled and patched states
Message-ID<sP3yq-61M-3@gated-at.bofh.it>
On Thu 2016-12-08 12:08:33, Josh Poimboeuf wrote:
> Once we have a consistency model, patches and their objects will be
> enabled and disabled at different times.  For example, when a patch is
> disabled, its loaded objects' funcs can remain registered with ftrace
> indefinitely until the unpatching operation is complete and they're no
> longer in use.
> 
> It's less confusing if we give them different names: patches can be
> enabled or disabled; objects (and their funcs) can be patched or
> unpatched:
> 
> - Enabled means that a patch is logically enabled (but not necessarily
>   fully applied).
> 
> - Patched means that an object's funcs are registered with ftrace and
>   added to the klp_ops func stack.
> 
> Also, since these states are binary, represent them with booleans
> instead of ints.
> 
> Signed-off-by: Josh Poimboeuf <jpoimboe@redhat.com>

Makes sense. The patch is pretty straightforward.

Reviewed-by: Petr Mladek <pmladek@suse.com>

Best Regards,
Petr

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web