Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-15 Thread Greg KH
On Fri, Sep 15, 2017 at 05:46:58PM +0200, Petr Mladek wrote: > On Thu 2017-09-14 08:31:24, Jason Baron wrote: > > > > > > On 09/14/2017 06:32 AM, Petr Mladek wrote: > > > On Tue 2017-09-12 23:47:32, Jason Baron wrote: > > >> > > >> > > >> On 09/12/2017 01:35 AM, Petr Mladek wrote: > > >>> On Mon

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-15 Thread Petr Mladek
On Thu 2017-09-14 08:31:24, Jason Baron wrote: > > > On 09/14/2017 06:32 AM, Petr Mladek wrote: > > On Tue 2017-09-12 23:47:32, Jason Baron wrote: > >> > >> > >> On 09/12/2017 01:35 AM, Petr Mladek wrote: > >>> On Mon 2017-09-11 23:46:28, Jason Baron wrote: > On 09/11/2017 09:53 AM, Petr Mla

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-14 Thread Jason Baron
On 09/14/2017 06:32 AM, Petr Mladek wrote: > On Tue 2017-09-12 23:47:32, Jason Baron wrote: >> >> >> On 09/12/2017 01:35 AM, Petr Mladek wrote: >>> On Mon 2017-09-11 23:46:28, Jason Baron wrote: On 09/11/2017 09:53 AM, Petr Mladek wrote: > On Wed 2017-08-30 17:38:44, Jason Baron wrote: >

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-14 Thread Petr Mladek
On Tue 2017-09-12 23:47:32, Jason Baron wrote: > > > On 09/12/2017 01:35 AM, Petr Mladek wrote: > > On Mon 2017-09-11 23:46:28, Jason Baron wrote: > >> On 09/11/2017 09:53 AM, Petr Mladek wrote: > >>> On Wed 2017-08-30 17:38:44, Jason Baron wrote: > diff --git a/include/linux/livepatch.h b/i

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-12 Thread Jason Baron
On 09/12/2017 01:35 AM, Petr Mladek wrote: > On Mon 2017-09-11 23:46:28, Jason Baron wrote: >> On 09/11/2017 09:53 AM, Petr Mladek wrote: >>> On Wed 2017-08-30 17:38:44, Jason Baron wrote: diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h index 8d3df55..ee6d18b 100644 >

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-12 Thread Petr Mladek
On Mon 2017-09-11 23:46:28, Jason Baron wrote: > On 09/11/2017 09:53 AM, Petr Mladek wrote: > > On Wed 2017-08-30 17:38:44, Jason Baron wrote: > >> diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h > >> index 8d3df55..ee6d18b 100644 > >> --- a/include/linux/livepatch.h > >> +++ b/i

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-11 Thread Jason Baron
On 09/11/2017 09:53 AM, Petr Mladek wrote: > On Wed 2017-08-30 17:38:44, Jason Baron wrote: >> When doing cumulative patches, if patch A introduces a change to function 1, >> and patch B reverts the change to function 1 and introduces changes to say >> function 2 and 3 as well, the change that pa

Re: [PATCH v2 2/3] livepatch: add atomic replace

2017-09-11 Thread Petr Mladek
On Wed 2017-08-30 17:38:44, Jason Baron wrote: > When doing cumulative patches, if patch A introduces a change to function 1, > and patch B reverts the change to function 1 and introduces changes to say > function 2 and 3 as well, the change that patch A introduced to function 1 > is still present.

[PATCH v2 2/3] livepatch: add atomic replace

2017-08-30 Thread Jason Baron
When doing cumulative patches, if patch A introduces a change to function 1, and patch B reverts the change to function 1 and introduces changes to say function 2 and 3 as well, the change that patch A introduced to function 1 is still present. Introduce atomic replace, by first creating a 'no_op'