Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2017-01-06 Thread Josh Poimboeuf
On Wed, Jan 04, 2017 at 03:57:09PM +0100, Miroslav Benes wrote: > > > diff --git a/samples/livepatch/livepatch-sample.c > > b/samples/livepatch/livepatch-sample.c > > index bb61c65..0625f38 100644 > > --- a/samples/livepatch/livepatch-sample.c > > +++ b/samples/livepatch/livepatch-sample.c > >

Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2017-01-06 Thread Josh Poimboeuf
On Wed, Jan 04, 2017 at 03:57:09PM +0100, Miroslav Benes wrote: > > > diff --git a/samples/livepatch/livepatch-sample.c > > b/samples/livepatch/livepatch-sample.c > > index bb61c65..0625f38 100644 > > --- a/samples/livepatch/livepatch-sample.c > > +++ b/samples/livepatch/livepatch-sample.c > >

Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2017-01-04 Thread Miroslav Benes
> diff --git a/samples/livepatch/livepatch-sample.c > b/samples/livepatch/livepatch-sample.c > index bb61c65..0625f38 100644 > --- a/samples/livepatch/livepatch-sample.c > +++ b/samples/livepatch/livepatch-sample.c > @@ -89,7 +89,6 @@ static int livepatch_init(void) > > static void

Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2017-01-04 Thread Miroslav Benes
> diff --git a/samples/livepatch/livepatch-sample.c > b/samples/livepatch/livepatch-sample.c > index bb61c65..0625f38 100644 > --- a/samples/livepatch/livepatch-sample.c > +++ b/samples/livepatch/livepatch-sample.c > @@ -89,7 +89,6 @@ static int livepatch_init(void) > > static void

Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2016-12-21 Thread Petr Mladek
On Thu 2016-12-08 12:08:40, Josh Poimboeuf wrote: > From: Miroslav Benes > > Currently we do not allow patch module to unload since there is no > method to determine if a task is still running in the patched code. > > The consistency model gives us the way because when the

Re: [PATCH v3 15/15] livepatch: allow removal of a disabled patch

2016-12-21 Thread Petr Mladek
On Thu 2016-12-08 12:08:40, Josh Poimboeuf wrote: > From: Miroslav Benes > > Currently we do not allow patch module to unload since there is no > method to determine if a task is still running in the patched code. > > The consistency model gives us the way because when the unpatching > finishes

[PATCH v3 15/15] livepatch: allow removal of a disabled patch

2016-12-08 Thread Josh Poimboeuf
From: Miroslav Benes Currently we do not allow patch module to unload since there is no method to determine if a task is still running in the patched code. The consistency model gives us the way because when the unpatching finishes we know that all tasks were marked as safe to

[PATCH v3 15/15] livepatch: allow removal of a disabled patch

2016-12-08 Thread Josh Poimboeuf
From: Miroslav Benes Currently we do not allow patch module to unload since there is no method to determine if a task is still running in the patched code. The consistency model gives us the way because when the unpatching finishes we know that all tasks were marked as safe to call an original