Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-15 Thread Miroslav Benes
On Thu, 10 Aug 2017, Jason Baron wrote: > > > On 08/10/2017 07:12 AM, Miroslav Benes wrote: > > > > > Ok - associating the "atomic replace" with the patch itself makes sense to > > > me. > > > It would also basically work, I think with the patch I proposed except for > > > the > > > case where

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-15 Thread Miroslav Benes
On Thu, 10 Aug 2017, Jason Baron wrote: > > > On 08/10/2017 07:12 AM, Miroslav Benes wrote: > > > > > Ok - associating the "atomic replace" with the patch itself makes sense to > > > me. > > > It would also basically work, I think with the patch I proposed except for > > > the > > > case where

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-10 Thread Jason Baron
On 08/10/2017 07:12 AM, Miroslav Benes wrote: Ok - associating the "atomic replace" with the patch itself makes sense to me. It would also basically work, I think with the patch I proposed except for the case where the the "atomic replace" was on top of several non-"atomic replace" patches.

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-10 Thread Jason Baron
On 08/10/2017 07:12 AM, Miroslav Benes wrote: Ok - associating the "atomic replace" with the patch itself makes sense to me. It would also basically work, I think with the patch I proposed except for the case where the the "atomic replace" was on top of several non-"atomic replace" patches.

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-10 Thread Miroslav Benes
> Ok - associating the "atomic replace" with the patch itself makes sense to me. > It would also basically work, I think with the patch I proposed except for the > case where the the "atomic replace" was on top of several non-"atomic replace" > patches. The reason is that the "atomic replace" I

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-08-10 Thread Miroslav Benes
> Ok - associating the "atomic replace" with the patch itself makes sense to me. > It would also basically work, I think with the patch I proposed except for the > case where the the "atomic replace" was on top of several non-"atomic replace" > patches. The reason is that the "atomic replace" I

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-07-21 Thread Jason Baron
On 07/21/2017 09:06 AM, Miroslav Benes wrote: On Wed, 19 Jul 2017, Jason Baron wrote: Hi, In testing livepatch, I found that when doing cumulative patches, if a patched function is completed reverted by a subsequent patch (back to its original state) livepatch does not revert the funtion

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-07-21 Thread Jason Baron
On 07/21/2017 09:06 AM, Miroslav Benes wrote: On Wed, 19 Jul 2017, Jason Baron wrote: Hi, In testing livepatch, I found that when doing cumulative patches, if a patched function is completed reverted by a subsequent patch (back to its original state) livepatch does not revert the funtion

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-07-21 Thread Miroslav Benes
On Wed, 19 Jul 2017, Jason Baron wrote: > Hi, > > In testing livepatch, I found that when doing cumulative patches, if a patched > function is completed reverted by a subsequent patch (back to its original > state) > livepatch does not revert the funtion to its original state. Specifically, if

Re: [PATCH 0/3] livepatch: introduce atomic replace

2017-07-21 Thread Miroslav Benes
On Wed, 19 Jul 2017, Jason Baron wrote: > Hi, > > In testing livepatch, I found that when doing cumulative patches, if a patched > function is completed reverted by a subsequent patch (back to its original > state) > livepatch does not revert the funtion to its original state. Specifically, if