[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v4 4/4] x86/mem_sharing: compile mem_sharing subsystem only when kconfig is enabled



>>> On 03.05.19 at 00:13, <tamas@xxxxxxxxxxxxx> wrote:
> --- a/xen/arch/x86/mm.c
> +++ b/xen/arch/x86/mm.c
> @@ -368,7 +368,9 @@ void __init arch_init_memory(void)
>  
>      efi_init_memory();
>  
> +#ifdef CONFIG_MEM_SHARING
>      mem_sharing_init();
> +#endif

While for domctl code and alike using #ifdef may indeed be the
better choice, I think here an inline stub to avoid the #ifdef
would be preferable. Then again - recall you've already ack-ed
my patch to drop the function altogether? Perhaps you should
base your patch on mine (or Andrew could pull that other patch
into x86-next)? In that case (with the hunk above simply
dropped)
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.