Re: [Xen-devel] [PATCH v2 5/9] x86/hypercall: Move the hypercall tables into C

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 06/09/16 11:37, Jan Beulich wrote:
>>>> On 06.09.16 at 12:12, <andrew.cooper3@xxxxxxxxxx> wrote:
>> Editing (and indeed, finding) the hypercall tables can be tricky, especially
>> towards the end where .rept's are used to maintain the correct layout.
>>
>> Move this all into C, and let the compiler do the hard work.
>>
>> To do this, xen/hypercall.h and asm-x86/hypercall.h need to contain 
>> prototypes
>> for all the hypercalls; some were previously missing.  This in turn requires
>> some shuffling of definitions and includes.
>>
>> One difference is that NULL function pointers are used instead of
>> {,compat_}do_ni_hypercall(), which pv_hypercall() handles correctly.  All
>> ni_hypercall() infrastructure is therefore dropped.
>>
>> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
>
> But please don't commit this without also committing patch 7 in the
> same batch.

With this review, the entire series is now ready.  I will apply it all
in one go, along with adjusting the commit message in patch 8 to refer
to the changeset of patch 7.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel