qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] kvm: fix ioeventfd endianness on bi-endian architectures
Date: Sun, 15 Mar 2015 10:14:18 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0


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.

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]