Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2019-01-03 Thread Petr Mladek
On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > On Thu, Nov 29, 2018 at 10:44:25AM +0100, Petr Mladek wrote: > > The possibility to re-enable a registered patch was useful for immediate > > patches where the livepatch module had to stay until the system reboot. > > The improved consistency model

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-14 Thread Josh Poimboeuf
On Fri, Dec 14, 2018 at 11:02:32AM +0100, Petr Mladek wrote: > On Thu 2018-12-13 16:46:25, Josh Poimboeuf wrote: > > Now that we can't re-enable a patch, I wonder if we really need both the > > 'patch->enabled' and 'klp_target_state' variables? > > > > A patch is now always enabled, unless it's

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-14 Thread Josh Poimboeuf
On Fri, Dec 14, 2018 at 10:40:16AM +0100, Petr Mladek wrote: > On Thu 2018-12-13 16:29:59, Josh Poimboeuf wrote: > > On Thu, Dec 06, 2018 at 09:36:06AM -0500, Joe Lawrence wrote: > > > On 12/06/2018 05:14 AM, Petr Mladek wrote: > > > > On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: > > > >> On

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-14 Thread Petr Mladek
On Thu 2018-12-13 16:46:25, Josh Poimboeuf wrote: > Now that we can't re-enable a patch, I wonder if we really need both the > 'patch->enabled' and 'klp_target_state' variables? > > A patch is now always enabled, unless it's in transition, in which case > its 'enabled' state is the same as

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-14 Thread Petr Mladek
On Thu 2018-12-13 16:29:59, Josh Poimboeuf wrote: > On Thu, Dec 06, 2018 at 09:36:06AM -0500, Joe Lawrence wrote: > > On 12/06/2018 05:14 AM, Petr Mladek wrote: > > > On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: > > >> On Thu, 6 Dec 2018, Petr Mladek wrote: > > >> > > >>> On Wed 2018-12-05

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-13 Thread Josh Poimboeuf
On Thu, Nov 29, 2018 at 10:44:25AM +0100, Petr Mladek wrote: > @@ -309,40 +297,33 @@ static ssize_t enabled_store(struct kobject *kobj, > struct kobj_attribute *attr, > > mutex_lock(_mutex); > > - if (!klp_is_patch_registered(patch)) { > - /* > - * Module

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-13 Thread Josh Poimboeuf
On Thu, Dec 06, 2018 at 09:36:06AM -0500, Joe Lawrence wrote: > On 12/06/2018 05:14 AM, Petr Mladek wrote: > > On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: > >> On Thu, 6 Dec 2018, Petr Mladek wrote: > >> > >>> On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > diff --git

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Joe Lawrence
On 12/06/2018 05:14 AM, Petr Mladek wrote: > On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: >> On Thu, 6 Dec 2018, Petr Mladek wrote: >> >>> On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > index

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Joe Lawrence
On 12/06/2018 05:14 AM, Petr Mladek wrote: > On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: >> On Thu, 6 Dec 2018, Petr Mladek wrote: >> >>> On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > index

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Petr Mladek
On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: > On Thu, 6 Dec 2018, Petr Mladek wrote: > > > On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > > > index 972520144713..e01dfa3b58d2 100644 > > > > ---

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Petr Mladek
On Thu 2018-12-06 10:23:40, Miroslav Benes wrote: > On Thu, 6 Dec 2018, Petr Mladek wrote: > > > On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > > > index 972520144713..e01dfa3b58d2 100644 > > > > ---

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Miroslav Benes
On Thu, 6 Dec 2018, Petr Mladek wrote: > On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > > index 972520144713..e01dfa3b58d2 100644 > > > --- a/kernel/livepatch/core.c > > > +++ b/kernel/livepatch/core.c > > > @@ -45,7 +45,7

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Miroslav Benes
On Thu, 6 Dec 2018, Petr Mladek wrote: > On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > > index 972520144713..e01dfa3b58d2 100644 > > > --- a/kernel/livepatch/core.c > > > +++ b/kernel/livepatch/core.c > > > @@ -45,7 +45,7

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Petr Mladek
On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > index 972520144713..e01dfa3b58d2 100644 > > --- a/kernel/livepatch/core.c > > +++ b/kernel/livepatch/core.c > > @@ -45,7 +45,7 @@ > > */ > > DEFINE_MUTEX(klp_mutex); > > > >

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-06 Thread Petr Mladek
On Wed 2018-12-05 14:32:53, Joe Lawrence wrote: > > diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c > > index 972520144713..e01dfa3b58d2 100644 > > --- a/kernel/livepatch/core.c > > +++ b/kernel/livepatch/core.c > > @@ -45,7 +45,7 @@ > > */ > > DEFINE_MUTEX(klp_mutex); > > > >

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-05 Thread Joe Lawrence
On Thu, Nov 29, 2018 at 10:44:25AM +0100, Petr Mladek wrote: > The possibility to re-enable a registered patch was useful for immediate > patches where the livepatch module had to stay until the system reboot. > The improved consistency model allows to achieve the same result by > unloading and

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-05 Thread Joe Lawrence
On Thu, Nov 29, 2018 at 10:44:25AM +0100, Petr Mladek wrote: > The possibility to re-enable a registered patch was useful for immediate > patches where the livepatch module had to stay until the system reboot. > The improved consistency model allows to achieve the same result by > unloading and

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Miroslav Benes
On Tue, 4 Dec 2018, Petr Mladek wrote: > On Tue 2018-12-04 13:54:55, Miroslav Benes wrote: > > > diff --git a/Documentation/livepatch/livepatch.txt > > > b/Documentation/livepatch/livepatch.txt > > > index 2d7ed09dbd59..d849af312576 100644 > > > --- a/Documentation/livepatch/livepatch.txt > > >

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Miroslav Benes
On Tue, 4 Dec 2018, Petr Mladek wrote: > On Tue 2018-12-04 13:54:55, Miroslav Benes wrote: > > > diff --git a/Documentation/livepatch/livepatch.txt > > > b/Documentation/livepatch/livepatch.txt > > > index 2d7ed09dbd59..d849af312576 100644 > > > --- a/Documentation/livepatch/livepatch.txt > > >

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Petr Mladek
On Tue 2018-12-04 13:54:55, Miroslav Benes wrote: > > diff --git a/Documentation/livepatch/livepatch.txt > > b/Documentation/livepatch/livepatch.txt > > index 2d7ed09dbd59..d849af312576 100644 > > --- a/Documentation/livepatch/livepatch.txt > > +++ b/Documentation/livepatch/livepatch.txt > > @@

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Petr Mladek
On Tue 2018-12-04 13:54:55, Miroslav Benes wrote: > > diff --git a/Documentation/livepatch/livepatch.txt > > b/Documentation/livepatch/livepatch.txt > > index 2d7ed09dbd59..d849af312576 100644 > > --- a/Documentation/livepatch/livepatch.txt > > +++ b/Documentation/livepatch/livepatch.txt > > @@

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Miroslav Benes
Mainly nits below. Overall it looks good. > The possibility to re-enable a registered patch was useful for immediate > patches where the livepatch module had to stay until the system reboot. > The improved consistency model allows to achieve the same result by > unloading and loading the

Re: [PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-12-04 Thread Miroslav Benes
Mainly nits below. Overall it looks good. > The possibility to re-enable a registered patch was useful for immediate > patches where the livepatch module had to stay until the system reboot. > The improved consistency model allows to achieve the same result by > unloading and loading the

[PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-11-29 Thread Petr Mladek
The possibility to re-enable a registered patch was useful for immediate patches where the livepatch module had to stay until the system reboot. The improved consistency model allows to achieve the same result by unloading and loading the livepatch module again. Also we are going to add a feature

[PATCH v14 05/11] livepatch: Simplify API by removing registration step

2018-11-29 Thread Petr Mladek
The possibility to re-enable a registered patch was useful for immediate patches where the livepatch module had to stay until the system reboot. The improved consistency model allows to achieve the same result by unloading and loading the livepatch module again. Also we are going to add a feature