From: no-reply@patchew.org To: liang.z.li@intel.com Cc: famz@redhat.com, qemu-devel@nongnu.org, mtosatti@redhat.com, kvm@vger.kernel.org, mst@redhat.com, uintela@redhat.com, dgilbert@redhat.com, dave.hansen@intel.com, wei.w.wang@intel.com, amit.shah@redhat.com, pbonzini@redhat.com, liang.z.li@intel.com Subject: Re: [Qemu-devel] [PATCH v4 qemu 0/6] Fast (de)inflating & fast live migration Date: Wed, 11 Jan 2017 01:08:52 -0800 (PST) [thread overview] Message-ID: <148412573107.208.13981250517015114145@790289a7ca88> (raw) In-Reply-To: <1484124524-481-1-git-send-email-liang.z.li@intel.com> Hi, Your series seems to have some coding style problems. See output below for more information: Message-id: 1484124524-481-1-git-send-email-liang.z.li@intel.com Subject: [Qemu-devel] [PATCH v4 qemu 0/6] Fast (de)inflating & fast live migration Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 # Useful git options git config --local diff.renamelimit 0 git config --local diff.renames True commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/1484124524-481-1-git-send-email-liang.z.li@intel.com -> patchew/1484124524-481-1-git-send-email-liang.z.li@intel.com Switched to a new branch 'test' 7e2ce67 migration: skip unused pages during live migration 687ca7e kvm.c: Add two new arch specific functions b2682d9 bitmap: Add a new bitmap_move function 3983f00 balloon: get unused page info from guest df86151 virtio-balloon: speed up inflating & deflating process 1a58699 virtio-balloon: update linux head file === OUTPUT BEGIN === Checking PATCH 1/6: virtio-balloon: update linux head file... Checking PATCH 2/6: virtio-balloon: speed up inflating & deflating process... Checking PATCH 3/6: balloon: get unused page info from guest... ERROR: suspect code indent for conditional statements (12, 17) #147: FILE: hw/virtio/virtio-balloon.c:430: + if (s->bmap_len < hdr.data_len) { + hdr.data_len = s->bmap_len; total: 1 errors, 0 warnings, 344 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. Checking PATCH 4/6: bitmap: Add a new bitmap_move function... Checking PATCH 5/6: kvm.c: Add two new arch specific functions... Checking PATCH 6/6: migration: skip unused pages during live migration... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-devel@freelists.org
WARNING: multiple messages have this Message-ID (diff)
From: no-reply@patchew.org To: liang.z.li@intel.com Cc: famz@redhat.com, qemu-devel@nongnu.org, mtosatti@redhat.com, kvm@vger.kernel.org, mst@redhat.com, uintela@redhat.com, dgilbert@redhat.com, dave.hansen@intel.com, wei.w.wang@intel.com, amit.shah@redhat.com, pbonzini@redhat.com Subject: Re: [Qemu-devel] [PATCH v4 qemu 0/6] Fast (de)inflating & fast live migration Date: Wed, 11 Jan 2017 01:08:52 -0800 (PST) [thread overview] Message-ID: <148412573107.208.13981250517015114145@790289a7ca88> (raw) In-Reply-To: <1484124524-481-1-git-send-email-liang.z.li@intel.com> Hi, Your series seems to have some coding style problems. See output below for more information: Message-id: 1484124524-481-1-git-send-email-liang.z.li@intel.com Subject: [Qemu-devel] [PATCH v4 qemu 0/6] Fast (de)inflating & fast live migration Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 # Useful git options git config --local diff.renamelimit 0 git config --local diff.renames True commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/1484124524-481-1-git-send-email-liang.z.li@intel.com -> patchew/1484124524-481-1-git-send-email-liang.z.li@intel.com Switched to a new branch 'test' 7e2ce67 migration: skip unused pages during live migration 687ca7e kvm.c: Add two new arch specific functions b2682d9 bitmap: Add a new bitmap_move function 3983f00 balloon: get unused page info from guest df86151 virtio-balloon: speed up inflating & deflating process 1a58699 virtio-balloon: update linux head file === OUTPUT BEGIN === Checking PATCH 1/6: virtio-balloon: update linux head file... Checking PATCH 2/6: virtio-balloon: speed up inflating & deflating process... Checking PATCH 3/6: balloon: get unused page info from guest... ERROR: suspect code indent for conditional statements (12, 17) #147: FILE: hw/virtio/virtio-balloon.c:430: + if (s->bmap_len < hdr.data_len) { + hdr.data_len = s->bmap_len; total: 1 errors, 0 warnings, 344 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. Checking PATCH 4/6: bitmap: Add a new bitmap_move function... Checking PATCH 5/6: kvm.c: Add two new arch specific functions... Checking PATCH 6/6: migration: skip unused pages during live migration... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-devel@freelists.org
next prev parent reply other threads:[~2017-01-11 9:09 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-01-11 8:48 [PATCH v4 qemu 0/6] Fast (de)inflating & fast live migration Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 1/6] virtio-balloon: update linux head file Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 2/6] virtio-balloon: speed up inflating & deflating process Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 3/6] balloon: get unused page info from guest Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 4/6] bitmap: Add a new bitmap_move function Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 5/6] kvm.c: Add two new arch specific functions Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 8:48 ` [PATCH v4 qemu 6/6] migration: skip unused pages during live migration Liang Li 2017-01-11 8:48 ` [Qemu-devel] " Liang Li 2017-01-11 9:08 ` no-reply [this message] 2017-01-11 9:08 ` [Qemu-devel] [PATCH v4 qemu 0/6] Fast (de)inflating & fast " no-reply 2017-01-11 9:12 ` no-reply 2017-01-11 9:12 ` no-reply
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=148412573107.208.13981250517015114145@790289a7ca88 \ --to=no-reply@patchew.org \ --cc=amit.shah@redhat.com \ --cc=dave.hansen@intel.com \ --cc=dgilbert@redhat.com \ --cc=famz@redhat.com \ --cc=kvm@vger.kernel.org \ --cc=liang.z.li@intel.com \ --cc=mst@redhat.com \ --cc=mtosatti@redhat.com \ --cc=pbonzini@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=uintela@redhat.com \ --cc=wei.w.wang@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.