Re: [Xen-devel] [PATCH 6/9] xen/multicall: Rework arch multicall handling

2016-08-03 Thread Andrew Cooper
On 03/08/16 16:02, Jan Beulich wrote: On 18.07.16 at 11:51, wrote: >> --- a/xen/arch/x86/hypercall.c >> +++ b/xen/arch/x86/hypercall.c >> @@ -338,6 +338,34 @@ long pv_hypercall(struct cpu_user_regs *regs) >> return ret; >> } >> >> +void arch_do_multicall_call(struct mc_state *state) >

Re: [Xen-devel] [PATCH 6/9] xen/multicall: Rework arch multicall handling

2016-08-03 Thread Jan Beulich
>>> On 18.07.16 at 11:51, wrote: > --- a/xen/arch/x86/hypercall.c > +++ b/xen/arch/x86/hypercall.c > @@ -338,6 +338,34 @@ long pv_hypercall(struct cpu_user_regs *regs) > return ret; > } > > +void arch_do_multicall_call(struct mc_state *state) > +{ > +if ( !is_pv_32bit_vcpu(current) ) >

Re: [Xen-devel] [PATCH 6/9] xen/multicall: Rework arch multicall handling

2016-07-20 Thread Julien Grall
Hi Andrew, On 18/07/16 10:51, Andrew Cooper wrote: The x86 multicall handling was previously some very hairy inline assembly, and is hard to follow and maintain. Replace the existing do_multicall_call() with arch_do_multicall_call(). The x86 side needs to handle both compat and non-compat call

[Xen-devel] [PATCH 6/9] xen/multicall: Rework arch multicall handling

2016-07-18 Thread Andrew Cooper
The x86 multicall handling was previously some very hairy inline assembly, and is hard to follow and maintain. Replace the existing do_multicall_call() with arch_do_multicall_call(). The x86 side needs to handle both compat and non-compat calls, so pass the full multicall state, rather than just