Re: [PATCH v4 0/9] add xenalyze to staging

[George Dunlap <george.dunlap@eu.citrix.com>]

On 06/03/2015 11:35 AM, Julien Grall wrote:
> On 03/06/2015 11:10, Ian Campbell wrote:
>> On Sat, 2015-05-23 at 08:24 +0000, Olaf Hering wrote:
>>> Having xenalyze in the source tree makes it much easier to keep private
>>> debug code in hypervisor and xenalyze in sync. It helped alot while
>>> debugging the root cause for commit
>>> 607e8494c42397fb249191904066cace6ac9a880.
>>
>> I'm afraid it doesn't build on arm64.
>>
>> Some of these actually look like non-arch specific failures (e.g.
>> conflicts with register_t from system headers) or issues which should
>> probably be addressed with xenalyze in tree (e.g. NR_CPUS ought to be
>> available directly now?) or with some trivial #ifdef modifications.
>>
>> That said, I don't know that xentrace actually works on ARM nor that
>> xenalyze could analyse such traces even with the build issues addressed,
>> so I'd be equally happy if this was just made x86 only.
> 
> xentrace is not working as we don't have the infrastructure for ARM in Xen.
> 
> Compiling xentrace & co only for x86 would be the more sensible solution.

>From the perspective of checking in this series, yes it probably is.
But it looks like this has turned up some deeper issues with the Xen on
ARM build environment that others will trip over at some point.  (Or
perhaps just ijc's build environment.)

 -George