Re: [PATCH v2 0/5] livepatch: new API to track system state changes

From: Miroslav Benes
Date: Tue Aug 13 2019 - 09:47:07 EST


On Fri, 19 Jul 2019, Petr Mladek wrote:

> Hi,
>
> this is another piece in the puzzle that helps to maintain more
> livepatches.
>
> Especially pre/post (un)patch callbacks might change a system state.
> Any newly installed livepatch has to somehow deal with system state
> modifications done be already installed livepatches.
>
> This patchset provides a simple and generic API that
> helps to keep and pass information between the livepatches.
> It is also usable to prevent loading incompatible livepatches.
>
>
> Changes since v1:
>
> + Use "unsigned long" instead of "int" for "state.id" [Nicolai]
> + Use "unsigned int" instead of "int" for "state.version [Petr]
> + Include "state.h" to avoid warning about non-static func [Miroslav]
> + Simplify logic in klp_is_state_compatible() [Miroslav]
> + Document how livepatches should handle the state [Nicolai]
> + Fix some typos, formulation, module metadata [Joe, Miroslav]

I noticed couple of typos along the way but apart from that

Acked-by: Miroslav Benes <mbenes@xxxxxxx>

Miroslav