Re: [PATCH] tools/libxc: Fix ARM build broken by XEN_DOMCTL_getvcpuextstate commit

[Razvan Cojocaru <rcojocaru@bitdefender.com>]

On 03/15/2017 11:59 AM, Wei Liu wrote:
> On Wed, Mar 15, 2017 at 11:20:30AM +0200, Razvan Cojocaru wrote:
>> The previous "tools/libxc: Exposed XEN_DOMCTL_getvcpuextstate" broke
>> the ARM build (the hypercall does not have a corresponding DOMCTL
>> ARM struct). This patch fixes the build by returning -ENODEV for
>> ARM from xc_vcpu_get_extstate().
>>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
> 
> Acked-by: Wei Liu <wei.liu2@citrix.com>
> 
>> ---
>>  tools/libxc/xc_domain.c | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/tools/libxc/xc_domain.c b/tools/libxc/xc_domain.c
>> index 71e5d93..c7e8065 100644
>> --- a/tools/libxc/xc_domain.c
>> +++ b/tools/libxc/xc_domain.c
>> @@ -560,7 +560,8 @@ int xc_vcpu_get_extstate(xc_interface *xch,
>>                           uint32_t vcpu,
>>                           xc_vcpu_extstate_t *extstate)
>>  {
>> -    int rc;
>> +    int rc = -ENODEV;
>> +#if defined (__i386) || defined(__x86_64__)
> 
> This should be __i386__ for consistency.
> 
> Assuming you've tested ARM build, I will fix this up while committing.

Thank you. I did test it (I've cross-compiled in an ARM chroot).


Thanks,
Razvan

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