On 11.11.2020 21:07, Paul Durrant wrote: > From: Paul Durrant <pdurr...@amazon.com> > > This patch moves the implementation of HVCALL_SEND_IPI that is currently > inline in viridian_hypercall() into a new hvcall_ipi() function. > > The new function returns Xen errno values similarly to hvcall_flush(). Hence > the errno translation code in viridial_hypercall() is generalized, removing > the need for the local 'status' variable. > > NOTE: The formatting of the 'out' label also corrected as per CODING_STYLE
How about correcting the adjacent switch() at the same time as well? > and the code is adjusted to avoid a register copy-back if 'mode' is > neither 8 nor 4. While you mention it here, isn't this an unrelated change wanting its own justification? Jan