Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-22 Thread Li Bin
On 2015/1/22 17:15, Miroslav Benes wrote: > On Thu, 22 Jan 2015, Li Bin wrote: > >> On 2015/1/21 17:07, Li Bin wrote: >>> This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. >>> >>> The method that only allowing the topmost patch on the stack to be >>> enabled or disabled is

Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-22 Thread Miroslav Benes
On Thu, 22 Jan 2015, Li Bin wrote: > On 2015/1/21 17:07, Li Bin wrote: > > This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. > > > > The method that only allowing the topmost patch on the stack to be > > enabled or disabled is unreasonable. Such as the following case: > > > > -

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-22 Thread Miroslav Benes
On Thu, 22 Jan 2015, Li Bin wrote: On 2015/1/21 17:07, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following case: - do live

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-22 Thread Li Bin
On 2015/1/22 17:15, Miroslav Benes wrote: On Thu, 22 Jan 2015, Li Bin wrote: On 2015/1/21 17:07, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the

Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-21 Thread Li Bin
On 2015/1/21 17:07, Li Bin wrote: > This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. > > The method that only allowing the topmost patch on the stack to be > enabled or disabled is unreasonable. Such as the following case: > > - do live patch1 > - disable patch1 >

Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-21 Thread Li Bin
On 2015/1/21 22:36, Seth Jennings wrote: > On Wed, Jan 21, 2015 at 03:06:38PM +0100, Jiri Kosina wrote: >> On Wed, 21 Jan 2015, Li Bin wrote: >> >>> This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. >>> >>> The method that only allowing the topmost patch on the stack to be >>> enabled

Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-21 Thread Seth Jennings
On Wed, Jan 21, 2015 at 03:06:38PM +0100, Jiri Kosina wrote: > On Wed, 21 Jan 2015, Li Bin wrote: > > > This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. > > > > The method that only allowing the topmost patch on the stack to be > > enabled or disabled is unreasonable. Such as the

Re: [PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-21 Thread Jiri Kosina
On Wed, 21 Jan 2015, Li Bin wrote: > This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. > > The method that only allowing the topmost patch on the stack to be > enabled or disabled is unreasonable. Such as the following case: > > - do live patch1 > - disable patch1 >

[PATCH 1/2] livepatch: Revert "livepatch: enforce patch stacking semantics"

2015-01-21 Thread Li Bin
This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following case: - do live patch1 - disable patch1 - do live patch2 //error Now, we will never

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-21 Thread Li Bin
On 2015/1/21 17:07, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following case: - do live patch1 - disable patch1 - do

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-21 Thread Li Bin
On 2015/1/21 22:36, Seth Jennings wrote: On Wed, Jan 21, 2015 at 03:06:38PM +0100, Jiri Kosina wrote: On Wed, 21 Jan 2015, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is

[PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-21 Thread Li Bin
This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following case: - do live patch1 - disable patch1 - do live patch2 //error Now, we will never

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-21 Thread Seth Jennings
On Wed, Jan 21, 2015 at 03:06:38PM +0100, Jiri Kosina wrote: On Wed, 21 Jan 2015, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following

Re: [PATCH 1/2] livepatch: Revert livepatch: enforce patch stacking semantics

2015-01-21 Thread Jiri Kosina
On Wed, 21 Jan 2015, Li Bin wrote: This reverts commit 83a90bb1345767f0cb96d242fd8b9db44b2b0e17. The method that only allowing the topmost patch on the stack to be enabled or disabled is unreasonable. Such as the following case: - do live patch1 - disable patch1 - do