[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] xen-hvm: use new resource mapping API
From: |
Paul Durrant |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] xen-hvm: use new resource mapping API |
Date: |
Thu, 10 May 2018 08:29:22 +0000 |
> -----Original Message-----
> From: Eric Blake [mailto:address@hidden
> Sent: 09 May 2018 18:26
> To: Paul Durrant <address@hidden>; address@hidden
> Cc: address@hidden; address@hidden
> Subject: Re: [Qemu-devel] [PATCH 0/2] xen-hvm: use new resource mapping
> API
>
> On 05/09/2018 11:05 AM, Paul Durrant wrote:
>
> >> xenforeignmemory_map_resource() to map ioreq pages...
> >> ERROR: spaces required around that '*' (ctx:WxV)
> >> #164: FILE: include/hw/xen/xen_common.h:128:
> >> + xenforeignmemory_handle *fmem, domid_t domid, unsigned int
> type,
> >> ^
> >>
> >> total: 1 errors, 0 warnings, 138 lines checked
> >>
> >> Your patch has style problems, please review. If any of these errors
> >> are false positives report them to the maintainer, see
> >> CHECKPATCH in MAINTAINERS.
> >
> > This style warning appears to be spurious.
>
> Yep, and it's because xenforeignmemory_handle doesn't follow our usual
> conventions for a type name. See commit 5ac067a if you want to add it
> to the list of whitelisted exception type names, to silence messages
> like this.
Thanks Eric. I'll send a v2 series with a suitable additional patch.
Cheers,
Paul
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org