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

Re: [Xen-devel] [PATCH v6 02/23] xen: move NUMA_NO_NODE to public memory.h as XEN_NUMA_NO_NODE



>>> Andrew Cooper <andrew.cooper3@xxxxxxxxxx> 02/27/15 5:58 PM >>>
>On 27/02/15 16:51, Wei Liu wrote:
>> During last round review, Andrew wanted me to move this to Xen public
>> header to avoid reinventing it in libxc. Now this value is used in libxc
>> patch.
>>
>> But I don't particularly mind whether we move it or not, it's up to you
>> maintainers to decide.
>
>It is a sentinel value used in the public ABI.  It should therefore
>appear in the public API.

Which it already does, as XENMEMF_get_node(0). I don't think it needs
particular naming as a new constant, even more that it isn't intended to
be used explicitly in any of the memops.

Jan


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


 


Rackspace

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