[Xen-devel] [PATCH v3] libelf: allow having HYPERCALL_PAGE entry before VIRT_BASE in __xen_guest section.

2017-10-11 Thread gregory . herrero
From: Gregory Herrero When filling __xen_guest section of a guest, user may define HYPERCALL_PAGE earlier than VIRT_BASE in the section leading to an incorrect hypercall page address since an undefined virt_base could be used to compute hypercall page address. If there is no VIRT_BASE entry in

[Xen-devel] [PATCH v2] libelf: allow having HYPERCALL_PAGE entry before VIRT_BASE in __xen_guest section.

2017-10-11 Thread gregory . herrero
From: Gregory Herrero When filling __xen_guest section of a guest, user may define HYPERCALL_PAGE earlier than VIRT_BASE in the section leading to an incorrect hypercall page address since an undefined virt_base could be used to compute hypercall page address. If there is no VIRT_BASE entry in

Re: [Xen-devel] [PATCH] libelf: allow having HYPERCALL_PAGE entry before VIRT_BASE in __xen_guest section.

2017-10-11 Thread Gregory Herrero
On Wed, Oct 11, 2017 at 06:40:14AM -0600, Jan Beulich wrote: > >>> On 11.10.17 at 14:03, wrote: > > When filling __xen_guest section of a guest, user may define > > HYPERCALL_PAGE earlier than VIRT_BASE in the section leading to an > > incorrect hypercall page address since an undefined virt_base

[Xen-devel] [PATCH] libelf: allow having HYPERCALL_PAGE entry before VIRT_BASE in __xen_guest section.

2017-10-11 Thread gregory . herrero
From: Gregory Herrero When filling __xen_guest section of a guest, user may define HYPERCALL_PAGE earlier than VIRT_BASE in the section leading to an incorrect hypercall page address since an undefined virt_base could be used to compute hypercall page address. If there is no VIRT_BASE entry in

[Xen-devel] [PATCH v3] xen: stop_machine: fill fn_result only in case of error.

2017-05-31 Thread gregory . herrero
From: Gregory Herrero When stop_machine_run() is called with NR_CPUS as last argument, fn_result member must be filled only if an error happens since it is shared across all cpus. Assume CPU1 detects an error and set fn_result to -1, then CPU2 doesn't detect an error and set fn_result to 0

[Xen-devel] [PATCH v2] xen: stop_machine: fill fn_result only in case of error.

2017-05-31 Thread gregory . herrero
From: Gregory Herrero When stop_machine_run() is called with NR_CPUS as last argument, fn_result member must be filled only if an error happens since it is shared across all cpus. Assume CPU1 detects an error and set fn_result to -1, then CPU2 doesn't detect an error and set fn_result to 0

Re: [Xen-devel] [PATCH] xen: stop_machine: fill fn_result only in case of error.

2017-05-31 Thread Gregory Herrero
On Wed, May 31, 2017 at 02:30:29AM -0600, Jan Beulich wrote: > >>> On 30.05.17 at 18:19, wrote: > > Since fn_result member is shared across all cpus, it must be filled > > only if an error happens. Assume CPU1 detects an error and set fn_result > > to -1, then CPU2 doesn't detect an error and set

[Xen-devel] [PATCH] xen: stop_machine: fill fn_result only in case of error.

2017-05-30 Thread gregory . herrero
From: Gregory Herrero Since fn_result member is shared across all cpus, it must be filled only if an error happens. Assume CPU1 detects an error and set fn_result to -1, then CPU2 doesn't detect an error and set fn_result to 0. The error detected by CPU1 will be ignored. Signed-off-by: Gr