Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-12-02 Thread Julien Grall
Hi, On 02/12/2019 16:17, Andrew Cooper wrote: On 15/11/2019 15:24, Julien Grall wrote: On Fri, 15 Nov 2019, 18:13 Andrew Cooper, > wrote: On 31/10/2019 21:25, Julien Grall wrote: > Hi, > > On 31/10/2019 19:28, Andrew Cooper wrote: >> This c

Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-12-02 Thread Andrew Cooper
On 15/11/2019 15:24, Julien Grall wrote: > On Fri, 15 Nov 2019, 18:13 Andrew Cooper, > wrote: > > On 31/10/2019 21:25, Julien Grall wrote: > > Hi, > > > > On 31/10/2019 19:28, Andrew Cooper wrote: > >> This code is especially tangled.  VCPUOP_i

Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-11-15 Thread Julien Grall
On Fri, 15 Nov 2019, 18:13 Andrew Cooper, wrote: > On 31/10/2019 21:25, Julien Grall wrote: > > Hi, > > > > On 31/10/2019 19:28, Andrew Cooper wrote: > >> This code is especially tangled. VCPUOP_initialise calls into > >> arch_initialise_vcpu() which calls back into default_initialise_vcpu() > w

Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-11-15 Thread Andrew Cooper
On 31/10/2019 21:25, Julien Grall wrote: > Hi, > > On 31/10/2019 19:28, Andrew Cooper wrote: >> This code is especially tangled. VCPUOP_initialise calls into >> arch_initialise_vcpu() which calls back into default_initialise_vcpu() which >> is common code. >> >> This path is actually dead code on

Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-11-04 Thread Jan Beulich
On 31.10.2019 20:28, Andrew Cooper wrote: > This code is especially tangled. VCPUOP_initialise calls into > arch_initialise_vcpu() which calls back into default_initialise_vcpu() which > is common code. > > This path is actually dead code on ARM, because VCPUOP_initialise is filtered > out by do_

Re: [Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-10-31 Thread Julien Grall
Hi, On 31/10/2019 19:28, Andrew Cooper wrote: > This code is especially tangled. VCPUOP_initialise calls into > arch_initialise_vcpu() which calls back into default_initialise_vcpu() which > is common code. > > This path is actually dead code on ARM, because VCPUOP_initialise is filtered > out b

[Xen-devel] [PATCH] xen/vcpu: Sanitise VCPUOP_initialise call hierachy

2019-10-31 Thread Andrew Cooper
This code is especially tangled. VCPUOP_initialise calls into arch_initialise_vcpu() which calls back into default_initialise_vcpu() which is common code. This path is actually dead code on ARM, because VCPUOP_initialise is filtered out by do_arm_vcpu_op(). The only valid way to start a secondar