Re: [Xen-devel] [PATCH v2 01/41] arm/acpi: Build numa for x86 only

[Julien Grall <julien.grall@xxxxxxxxxx>]

On 20/05/15 16:41, Jan Beulich wrote:
>>>> On 20.05.15 at 17:21, <julien.grall@xxxxxxxxxx> wrote:
>> On 20/05/15 16:07, Jan Beulich wrote:
>>>>>> On 17.05.15 at 22:03, <parth.dixit@xxxxxxxxxx> wrote:
>>>> From: Naresh Bhat <naresh.bhat@xxxxxxxxxx>
>>>>
>>>> Numa is currently not supported for arm in xen.
>>>
>>> This doesn't represent a valid reason for the code to be disabled,
>>> the more that ARM will want to support NUMA sooner or later. A
>>> valid reason would be if you'd have to introduce _a lot_ of code on
>>> the ARM side in order to make this build - if that's the case, please
>>> say so in the description, as from an architectural pov the absence
>>> of the respective ACPI tables should be no issue at all.
>>
>> The acpi/numa.c is calling a lot of arch specific function implemented
>> in (arch/x86/srat.c). The code in acpi/numa.c is very x86 related (such
>> as x2apic).
>>
>> I think disable the compilation is preferable for now given that there
>> is no board for testing the changes.
> 
> Or, as said in reply to patch 2, stubbing out the missing pieces in
> ARM code. As said there - let's not make more spaghetti of common
> code than it already is.

That would mean we will build a code that won't be use on ARM until we
decide to implement NUMA. This is because the main function
acpi_numa_init is not called and should not be called until someone
figure out this is valid on ARM.

What about defining a new define HAVE_NUMA?

Regards,

-- 
Julien Grall

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