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

[Paolo Bonzini <pbonzini@redhat.com>]

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/<20150313212337.31142.3991.stgit@bahia.local>


I'll fix this up when applying.

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