Re: [Qemu-devel] [PATCH] kvm: fix ioeventfd endianness on bi-endian architectures

[Greg Kurz]

On Sun, 15 Mar 2015 10:14:18 +0100
Paolo Bonzini <address@hidden> wrote:

> 
> 
> On 13/03/2015 22:30, Patchew Tool wrote:
> > This series passed Patchew automatic testing, but there are some warnings.
> > 
> > Find the log fragments below, or open the following URL to see the full log:
> > 
> > http://qemu.patchew.org/testing/log/<address@hidden>
> 
> 
> I'll fix this up when applying.
> 

Thanks !

--
Greg

BTW I could not connect to patchew with the link above (Error code: 
ERR_CONNECTION_REFUSED)

> The patch is okay.
> 
> Paolo
> 
> > ----------8<---------
> > 
> > 
> > === Starting docker ===
> > 
> > Copying test script "/home/patchew/tests/qemu-devel.sh"
> > Starting docker...
> > docker run --net=none -v 
> > /var/tmp/patchew-tester/tmpEMd8Zd:/var/tmp/patchew-test patchew:fedora-20 
> > timeout 3600 /var/tmp/patchew-test/qemu-devel.sh /var/tmp/patchew-test
> > 
> > *** Testing 'coding style check' ***
> > 
> > Checking 0001-kvm--fix-ioeventfd-endianness-on-bi-endian-architectures.patch
> > command failed with exit code 0
> > $@
> > ERROR: space required before the open parenthesis '('
> > #26: FILE: kvm-all.c:538:
> > +    switch(size) {
> > 
> > total: 1 errors, 0 warnings, 42 lines checked
> > 
> > 0001-kvm--fix-ioeventfd-endianness-on-bi-endian-architectures.patch has 
> > style problems, please review.  If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 
> > 
> > *** Testing 'configure' ***
> > 
> > Install prefix    /usr/local
> > BIOS directory    /usr/local/share/qemu
> > binary directory  /usr/local/bin
> > library directory /usr/local/lib
> > module directory  /usr/local/lib/qemu
> > libexec directory /usr/local/libexec
> > 
> > 
> > 
>