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

Re: [Xen-devel] [PATCH 2/2] dom_cow is needed for mem-sharing only



>>> On 31.05.19 at 11:42, <julien.grall@xxxxxxx> wrote:
> On 5/31/19 10:35 AM, Jan Beulich wrote:
>> --- a/xen/include/xen/mm.h
>> +++ b/xen/include/xen/mm.h
>> @@ -644,6 +644,9 @@ static inline void filtered_flush_tlb_ma
>>   
>>   /* Private domain structs for DOMID_XEN, DOMID_IO, etc. */
>>   extern struct domain *dom_xen, *dom_io, *dom_cow;
> 
> Don't you want to protect dom_cow with "#ifdef CONFIG_HAS_MEM_SHARING"?

There's no need to with ...

>> +#ifndef CONFIG_HAS_MEM_SHARING
>> +# define dom_cow NULL
>> +#endif

... this, and this way there's less clutter overall.

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®.