From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48955) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cVhhL-0002nh-T0 for qemu-devel@nongnu.org; Mon, 23 Jan 2017 11:42:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cVhhH-0003de-31 for qemu-devel@nongnu.org; Mon, 23 Jan 2017 11:42:27 -0500 Received: from mo69.mail-out.ovh.net ([178.32.228.69]:36883) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1cVhhG-0003dP-PC for qemu-devel@nongnu.org; Mon, 23 Jan 2017 11:42:23 -0500 Received: from player798.ha.ovh.net (b7.ovh.net [213.186.33.57]) by mo69.mail-out.ovh.net (Postfix) with ESMTP id 5088311164 for ; Mon, 23 Jan 2017 17:42:21 +0100 (CET) Date: Mon, 23 Jan 2017 17:42:14 +0100 From: Greg Kurz Message-ID: <20170123174214.3a86cb92@bahia.lan> In-Reply-To: <3478b86e-7a32-5e92-58cc-1cfe7ccecb41@huawei.com> References: <1485186641-12220-1-git-send-email-pradeep.jagadeesh@huawei.com> <148518739693.56.12407051583748682941@b042e9bf0dd0> <20170123172254.345fbecb@bahia.lan> <3478b86e-7a32-5e92-58cc-1cfe7ccecb41@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2 V13] fsdev: add IO throttle support to fsdev devices List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Pradeep Jagadeesh Cc: pradeepkiruvale@gmail.com, qemu-devel@nongnu.org, famz@redhat.com, aneesh.kumar@linux.vnet.ibm.com, berto@igalia.com On Mon, 23 Jan 2017 17:30:13 +0100 Pradeep Jagadeesh wrote: > On 1/23/2017 5:22 PM, Greg Kurz wrote: > > On Mon, 23 Jan 2017 08:03:18 -0800 (PST) > > no-reply@patchew.org wrote: > >> Hi, > >> > >> Your series seems to have some coding style problems. See output below for > >> more information: > >> > > > Ya, I observed signoff issue. I have created next version files. > I will send them tomorrow. > > But, I do not know how to solve the second warning. > i.e multi line macro issue. > As I had written in my mail: > > I guess your patch is ok here: checkpatch.pl is simply not smart enough to > > parse this. You can ignore this warning. Also, you had a positive review for patch 2/2: https://lists.gnu.org/archive/html/qemu-devel/2017-01/msg04637.html You can add this line below your S-o-b. > -Pradeep > > > Pradeep, > > > > One should usually take patchew's findings into account. See below. > > > >> Type: series > >> Subject: [Qemu-devel] [PATCH 0/2 V13] fsdev: add IO throttle support to fsdev devices > >> Message-id: 1485186641-12220-1-git-send-email-pradeep.jagadeesh@huawei.com > >> > >> === 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 > >> Switched to a new branch 'test' > >> 7686dc8 throttle: factor out duplicate code > >> 7568614 fsdev: add IO throttle support to fsdev devices > >> > >> === OUTPUT BEGIN === > >> Checking PATCH 1/2: fsdev: add IO throttle support to fsdev devices... > >> Checking PATCH 2/2: throttle: factor out duplicate code... > >> ERROR: Macros with multiple statements should be enclosed in a do - while loop > > > > I guess your patch is ok here: checkpatch.pl is simply not smart enough to > > parse this. > > > >> #228: FILE: include/qemu/throttle-options.h:13: > >> +#define THROTTLE_OPTS \ > >> + { \ > >> + .name = "throttling.iops-total",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit total I/O operations per second",\ > >> + },{ \ > >> + .name = "throttling.iops-read",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit read operations per second",\ > >> + },{ \ > >> + .name = "throttling.iops-write",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit write operations per second",\ > >> + },{ \ > >> + .name = "throttling.bps-total",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit total bytes per second",\ > >> + },{ \ > >> + .name = "throttling.bps-read",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit read bytes per second",\ > >> + },{ \ > >> + .name = "throttling.bps-write",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "limit write bytes per second",\ > >> + },{ \ > >> + .name = "throttling.iops-total-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "I/O operations burst",\ > >> + },{ \ > >> + .name = "throttling.iops-read-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "I/O operations read burst",\ > >> + },{ \ > >> + .name = "throttling.iops-write-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "I/O operations write burst",\ > >> + },{ \ > >> + .name = "throttling.bps-total-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "total bytes burst",\ > >> + },{ \ > >> + .name = "throttling.bps-read-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "total bytes read burst",\ > >> + },{ \ > >> + .name = "throttling.bps-write-max",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "total bytes write burst",\ > >> + },{ \ > >> + .name = "throttling.iops-total-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the iops-total-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.iops-read-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the iops-read-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.iops-write-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the iops-write-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.bps-total-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the bps-total-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.bps-read-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the bps-read-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.bps-write-max-length",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "length of the bps-write-max burst period, in seconds",\ > >> + },{ \ > >> + .name = "throttling.iops-size",\ > >> + .type = QEMU_OPT_NUMBER,\ > >> + .help = "when limiting by iops max size of an I/O in bytes",\ > >> + } > >> > >> ERROR: Missing Signed-off-by: line(s) > >> > > > > As Eric pointed out in another mail, all patches you send must have your > > Signed-off-by line. > > > >> total: 2 errors, 0 warnings, 278 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. > >> > >> === 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 >