Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-10 Thread Jan Beulich
On 09.11.2022 21:16, George Dunlap wrote: >> On 4 Nov 2022, at 05:01, Andrew Cooper wrote: >> On 03/11/2022 16:36, Juergen Gross wrote: >>> The code generated for the call_handlers_*() macros needs to avoid >>> undefined behavior when multiple handlers share the same priority. >>> The issue is the

Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-09 Thread Juergen Gross
On 09.11.22 21:16, George Dunlap wrote: On 4 Nov 2022, at 05:01, Andrew Cooper wrote: On 03/11/2022 16:36, Juergen Gross wrote: The code generated for the call_handlers_*() macros needs to avoid undefined behavior when multiple handlers share the same priority. The issue is the hypercall num

Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-09 Thread George Dunlap
> On 4 Nov 2022, at 05:01, Andrew Cooper wrote: > > On 03/11/2022 16:36, Juergen Gross wrote: >> The code generated for the call_handlers_*() macros needs to avoid >> undefined behavior when multiple handlers share the same priority. >> The issue is the hypercall number being unverified fed into

Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-04 Thread George Dunlap
> On 4 Nov 2022, at 05:01, Andrew Cooper wrote: > > The series claims "This is beneficial to performance and avoids > speculation issues.", c/s 8523851dbc4. > > That half sentence is literally the sum total of justification given for > this being related to speculation. The cover letter, writt

Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-04 Thread Jan Beulich
On 04.11.2022 06:01, Andrew Cooper wrote: > On 03/11/2022 16:36, Juergen Gross wrote: >> The code generated for the call_handlers_*() macros needs to avoid >> undefined behavior when multiple handlers share the same priority. >> The issue is the hypercall number being unverified fed into the macros

Re: Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-03 Thread Juergen Gross
On 04.11.22 06:01, Andrew Cooper wrote: On 03/11/2022 16:36, Juergen Gross wrote: The code generated for the call_handlers_*() macros needs to avoid undefined behavior when multiple handlers share the same priority. The issue is the hypercall number being unverified fed into the macros and then

Revert of the 4.17 hypercall handler changes Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-03 Thread Andrew Cooper
On 03/11/2022 16:36, Juergen Gross wrote: > The code generated for the call_handlers_*() macros needs to avoid > undefined behavior when multiple handlers share the same priority. > The issue is the hypercall number being unverified fed into the macros > and then used to set a mask via "mask = 1ULL

RE: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-03 Thread Henry Wang
Hi Juergen and Jan, > -Original Message- > From: Jan Beulich > Subject: Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall > handlers > > On 03.11.2022 17:36, Juergen Gross wrote: > > The code generated for the call_handlers_*() macros need

Re: [PATCH-for-4.17] xen: fix generated code for calling hypercall handlers

2022-11-03 Thread Jan Beulich
On 03.11.2022 17:36, Juergen Gross wrote: > The code generated for the call_handlers_*() macros needs to avoid > undefined behavior when multiple handlers share the same priority. > The issue is the hypercall number being unverified fed into the macros > and then used to set a mask via "mask = 1ULL