>>> On 15.03.16 at 20:04, <car...@cardoe.com> wrote: > On 3/15/16 3:31 AM, Jan Beulich wrote: >>>>> On 14.03.16 at 21:29, <car...@cardoe.com> wrote: >>> --- a/xen/arch/x86/hvm/hvm.c >>> +++ b/xen/arch/x86/hvm/hvm.c >>> @@ -5311,6 +5311,10 @@ typedef unsigned long hvm_hypercall_t( >>> #define compat_grant_table_op hvm_grant_table_op_compat32 >>> #define do_arch_1 paging_domctl_continuation >>> >>> +#ifndef CONFIG_TMEM >>> +#define do_tmem_op do_ni_hypercall >>> +#endif >> >> This being repeated in 3 places, wouldn't be better to put this in, >> say, xen/hypercall.h, next to the function declaration? > > Actually I tried this and it doesn't compile because that header isn't > included. Including that header into the relevant files (entry.S) > doesn't work either. Got a different suggestion?
Well, if the header can't be used in assembly files, so be it (for now at least, as mentioned by Andrew on IRC). But this then still calls for the #define to go into the header for the C source(s) sake. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
