Re: [Xen-devel] [PATCH] libs/foreignmemory: Avoid printing an error for ENOTSUPP

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: Julien Grall [mailto:julien.grall@xxxxxxx]
> Sent: 13 August 2018 18:33
> To: xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxx>; Wei Liu <wei.liu2@xxxxxxxxxx>;
> Paul Durrant <Paul.Durrant@xxxxxxxxxx>; Julien Grall <julien.grall@xxxxxxx>
> Subject: [PATCH] libs/foreignmemory: Avoid printing an error for ENOTSUPP
> 
> Resource mapping is not supported on Arm and results to an error message
> at every guest boot:
> 
> xenforeignmemory: error: ioctl failed: Operation not supported
> 
> Hide the error message when errnor is ENOTSUPP.
> 
> Signed-off-by: Julien Grall <julien.grall@xxxxxxx>

Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

> ---
>  tools/libs/foreignmemory/linux.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/libs/foreignmemory/linux.c
> b/tools/libs/foreignmemory/linux.c
> index a6b41b0b7f..3686cf41e0 100644
> --- a/tools/libs/foreignmemory/linux.c
> +++ b/tools/libs/foreignmemory/linux.c
> @@ -307,7 +307,7 @@ int osdep_xenforeignmemory_map_resource(
>      {
>          int saved_errno;
> 
> -        if ( errno != ENOTTY )
> +        if ( errno != ENOTTY && errno != EOPNOTSUPP )
>              PERROR("ioctl failed");
>          else
>              errno = EOPNOTSUPP;
> --
> 2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel