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

[Wei Liu <wei.liu2@xxxxxxxxxx>]

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@xxxxxxxxxxxxxxx>

Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

> ---
>  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.

>      DECLARE_DOMCTL;
>      DECLARE_HYPERCALL_BUFFER(void, buffer);
>      bool get_state;
> @@ -610,6 +611,7 @@ int xc_vcpu_get_extstate(xc_interface *xch,
>  out:
>      if ( get_state )
>          xc_hypercall_buffer_free(xch, buffer);
> +#endif
>  
>      return rc;
>  }
> -- 
> 1.9.1
> 

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