Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-19 Thread Jan Beulich
>>> On 19.08.16 at 10:37, wrote: > On 08/15/2016 04:10 PM, Jan Beulich wrote: > On 15.08.16 at 16:29, wrote: >>> On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: >>> On 14.08.16 at 23:52, wrote: >

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-19 Thread Ross Lagerwall
On 08/15/2016 04:10 PM, Jan Beulich wrote: On 15.08.16 at 16:29, wrote: On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: On 14.08.16 at 23:52, wrote: @@ -1034,6 +1047,9 @@ static int revert_payload(struct payload *data)

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:29, wrote: > On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: >> >>> On 14.08.16 at 23:52, wrote: >> > @@ -1034,6 +1047,9 @@ static int revert_payload(struct payload *data) >> > list_del_rcu(>applied_list); >>

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: > >>> On 14.08.16 at 23:52, wrote: > > @@ -374,7 +376,18 @@ static int move_payload(struct payload *payload, > > struct livepatch_elf *elf) > > elf->name, elf->sec[i].name, > >

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > @@ -374,7 +376,18 @@ static int move_payload(struct payload *payload, struct > livepatch_elf *elf) > elf->name, elf->sec[i].name, elf->sec[i].load_addr); > } > else > -

[Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-14 Thread Konrad Rzeszutek Wilk
So that when we apply the patch again the .bss is cleared. Otherwise we may find some variables containing old values. Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall Cc: Jan Beulich v3: Initial submission ---