Hi,
On 19/03/15 21:53, Boris Ostrovsky wrote:
> A few patches that add interface for querying hypervisor about device
> topology and allow 'xl info -n' display this information if PXM object
> is provided by ACPI.
>
> This series also makes some optimizations and cleanup of current CPU
> topology and NUMA sysctl queries.
I saw that some of these patches was pushed upstream last week. It
actually breaks compilation on ARM.
While the first error is trivial to fix (it's a missing include
xen/numa.h in xen/pci.h ). The second one is more difficult
sysctl.c: In function ‘do_sysctl’:
sysctl.c:349:42: error: ‘BAD_APICID’ undeclared (first use in this function)
if ( cputopo.core == BAD_APICID )
^
The value BAD_APICID doesn't have any meaning on ARM. Therefore the
usage in common code looks wrong to me. I'm not sure what should be the
testing value here given that cpu_to_core is not yet correctly
implemented on ARM.
It would be nice to at least build test it ARM/ARM64 before pushing
patches which modify common code.
Regards,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
