All of lore.kernel.org
 help / color / mirror / Atom feed
From: no-reply@patchew.org
To: arei.gonglei@huawei.com
Cc: famz@redhat.com, qemu-devel@nongnu.org,
	virtio-dev@lists.oasis-open.org, weidong.huang@huawei.com,
	claudio.fontana@huawei.com, mst@redhat.com, xin.zeng@intel.com,
	hanweidong@huawei.com, luonengjun@huawei.com, agraf@suse.de,
	nmorey@kalray.eu, mike.caraman@nxp.com, stefanha@redhat.com,
	jianjay.zhou@huawei.com, pbonzini@redhat.com,
	vincent.jardin@6wind.com, wu.wubin@huawei.com
Subject: Re: [Qemu-devel] [PATCH v3 00/10] virtio-crypto: introduce framework and device emulation
Date: Mon, 19 Sep 2016 01:39:52 -0700 (PDT)	[thread overview]
Message-ID: <20160919083947.19907.26000@ex-std-node742.prod.rhcloud.com> (raw)
In-Reply-To: <1474272982-275836-1-git-send-email-arei.gonglei@huawei.com>

Hi,

Your series seems to have some coding style problems. See output below for
more information:

Type: series
Subject: [Qemu-devel] [PATCH v3 00/10] virtio-crypto: introduce framework and device emulation
Message-id: 1474272982-275836-1-git-send-email-arei.gonglei@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 show --no-patch --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/1474272982-275836-1-git-send-email-arei.gonglei@huawei.com -> patchew/1474272982-275836-1-git-send-email-arei.gonglei@huawei.com
Switched to a new branch 'test'
8e80cd7 cryptodev: introduce an unified wrapper for crypto operation
8fa7033 virtio-crypto: add data queue processing handler
5b7c750 virtio-crypto: add control queue handler
b5c70fc virtio-crypto: set capacity of algorithms supported
556bc34 virtio-crypto-pci: add virtio crypto pci support
357cb7c virtio-crypto: add virtio crypto device emulation
8ec8cb5 cryptodev: introduce gcrypt lib as a new cryptodev backend
5bf4aad virtio-crypto: introduce virtio_crypto.h
6002d89 cryptodev: add symmetric algorithm operation stuff
c0d5e8b cryptodev: introduce cryptodev backend interface

=== OUTPUT BEGIN ===
Checking PATCH 1/10: cryptodev: introduce cryptodev backend interface...
ERROR: trailing whitespace
#352: FILE: include/crypto/cryptodev.h:140:
+ */                  $

total: 1 errors, 0 warnings, 327 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 2/10: cryptodev: add symmetric algorithm operation stuff...
Checking PATCH 3/10: virtio-crypto: introduce virtio_crypto.h...
Checking PATCH 4/10: cryptodev: introduce gcrypt lib as a new cryptodev backend...
Checking PATCH 5/10: virtio-crypto: add virtio crypto device emulation...
Checking PATCH 6/10: virtio-crypto-pci: add virtio crypto pci support...
Checking PATCH 7/10: virtio-crypto: set capacity of algorithms supported...
Checking PATCH 8/10: virtio-crypto: add control queue handler...
Checking PATCH 9/10: virtio-crypto: add data queue processing handler...
Checking PATCH 10/10: cryptodev: introduce an unified wrapper for crypto operation...
=== 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

  parent reply	other threads:[~2016-09-19  8:40 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-19  8:16 [Qemu-devel] [PATCH v3 00/10] virtio-crypto: introduce framework and device emulation Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 01/10] cryptodev: introduce cryptodev backend interface Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 02/10] cryptodev: add symmetric algorithm operation stuff Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 03/10] virtio-crypto: introduce virtio_crypto.h Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 04/10] cryptodev: introduce gcrypt lib as a new cryptodev backend Gonglei
2016-09-19  8:29   ` Daniel P. Berrange
2016-09-19  8:36     ` Gonglei (Arei)
2016-09-19  8:56   ` Daniel P. Berrange
2016-09-19  9:02     ` Gonglei (Arei)
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 05/10] virtio-crypto: add virtio crypto device emulation Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 06/10] virtio-crypto-pci: add virtio crypto pci support Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 07/10] virtio-crypto: set capacity of algorithms supported Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 08/10] virtio-crypto: add control queue handler Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 09/10] virtio-crypto: add data queue processing handler Gonglei
2016-09-19  8:16 ` [Qemu-devel] [PATCH v3 10/10] cryptodev: introduce an unified wrapper for crypto operation Gonglei
2016-09-19  8:39 ` no-reply [this message]
2016-09-22  5:53 ` [Qemu-devel] [PATCH v3 00/10] virtio-crypto: introduce framework and device emulation Gonglei (Arei)

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=20160919083947.19907.26000@ex-std-node742.prod.rhcloud.com \
    --to=no-reply@patchew.org \
    --cc=agraf@suse.de \
    --cc=arei.gonglei@huawei.com \
    --cc=claudio.fontana@huawei.com \
    --cc=famz@redhat.com \
    --cc=hanweidong@huawei.com \
    --cc=jianjay.zhou@huawei.com \
    --cc=luonengjun@huawei.com \
    --cc=mike.caraman@nxp.com \
    --cc=mst@redhat.com \
    --cc=nmorey@kalray.eu \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=vincent.jardin@6wind.com \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=weidong.huang@huawei.com \
    --cc=wu.wubin@huawei.com \
    --cc=xin.zeng@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: link
Be 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.