All of lore.kernel.org
 help / color / mirror / Atom feed
From: no-reply@patchew.org
To: jasowang@redhat.com
Cc: jasowang@redhat.com, qemu-devel@nongnu.org, ppandit@redhat.com
Subject: Re: [PATCH 0/6] Detect reentrant RX casue by loopback
Date: Tue, 23 Feb 2021 22:03:50 -0800 (PST)	[thread overview]
Message-ID: <161414662972.13232.10666773702775968385@c667a6b167f6> (raw)
In-Reply-To: <20210224055401.492407-1-jasowang@redhat.com>

Patchew URL: https://patchew.org/QEMU/20210224055401.492407-1-jasowang@redhat.com/



Hi,

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

Type: series
Message-id: 20210224055401.492407-1-jasowang@redhat.com
Subject: [PATCH 0/6] Detect reentrant RX casue by loopback

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/20210224055401.492407-1-jasowang@redhat.com -> patchew/20210224055401.492407-1-jasowang@redhat.com
Switched to a new branch 'test'
6a41d09 tx_pkt: switch to use qemu_receive_packet_iov() for loopback
5480e55 sungem: switch to use qemu_receive_packet() for loopback
f35fab6 msf2-mac: switch to use qemu_receive_packet() for loopback
1dc29cf dp8393x: switch to use qemu_receive_packet() for loopback packet
6c79834 e1000: switch to use qemu_receive_packet() for loopback
04660c0 net: introduce qemu_receive_packet()

=== OUTPUT BEGIN ===
1/6 Checking commit 04660c06d88e (net: introduce qemu_receive_packet())
ERROR: space required after that ',' (ctx:VxV)
#42: FILE: include/net/net.h:154:
+ssize_t qemu_receive_packet(NetClientState *nc, const uint8_t *buf,int size);
                                                                   ^

total: 1 errors, 0 warnings, 115 lines checked

Patch 1/6 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

2/6 Checking commit 6c7983468e94 (e1000: switch to use qemu_receive_packet() for loopback)
3/6 Checking commit 1dc29cfa3117 (dp8393x: switch to use qemu_receive_packet() for loopback packet)
4/6 Checking commit f35fab6cb737 (msf2-mac: switch to use qemu_receive_packet() for loopback)
5/6 Checking commit 5480e5557678 (sungem: switch to use qemu_receive_packet() for loopback)
6/6 Checking commit 6a41d093a705 (tx_pkt: switch to use qemu_receive_packet_iov() for loopback)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20210224055401.492407-1-jasowang@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com

  parent reply	other threads:[~2021-02-24  6:04 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-24  5:53 [PATCH 0/6] Detect reentrant RX casue by loopback Jason Wang
2021-02-24  5:53 ` [PATCH 1/6] net: introduce qemu_receive_packet() Jason Wang
2021-02-24 10:11   ` Philippe Mathieu-Daudé
2021-02-24 13:17     ` Jason Wang
2021-02-24 13:43       ` Philippe Mathieu-Daudé
2021-02-25 14:01         ` P J P
2021-02-25 16:28           ` Alexander Bulekov
2021-02-25 16:29             ` Alexander Bulekov
2021-02-26 18:14               ` [QEMU-SECURITY] " P J P
2021-02-26 18:53                 ` Alexander Bulekov
2021-02-25 14:31   ` Philippe Mathieu-Daudé
2021-02-24  5:53 ` [PATCH 2/6] e1000: switch to use qemu_receive_packet() for loopback Jason Wang
2021-02-25 14:34   ` Philippe Mathieu-Daudé
2021-02-24  5:53 ` [PATCH 3/6] dp8393x: switch to use qemu_receive_packet() for loopback packet Jason Wang
2021-02-24  6:13   ` Stefan Weil
2021-02-25 14:36     ` Philippe Mathieu-Daudé
2021-02-25 14:42       ` Stefan Weil
2021-02-26  7:04         ` Jason Wang
2021-02-24  5:53 ` [PATCH 4/6] msf2-mac: switch to use qemu_receive_packet() for loopback Jason Wang
2021-02-25 14:34   ` Philippe Mathieu-Daudé
2021-02-24  5:54 ` [PATCH 5/6] sungem: " Jason Wang
2021-02-25 14:35   ` Philippe Mathieu-Daudé
2021-02-24  5:54 ` [PATCH 6/6] tx_pkt: switch to use qemu_receive_packet_iov() " Jason Wang
2021-02-25 14:35   ` Philippe Mathieu-Daudé
2021-02-24  6:03 ` no-reply [this message]
2021-02-26 18:47 ` [PATCH] rtl8193: switch to use qemu_receive_packet() " Alexander Bulekov
2021-03-01  7:14   ` Jason Wang

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=161414662972.13232.10666773702775968385@c667a6b167f6 \
    --to=no-reply@patchew.org \
    --cc=jasowang@redhat.com \
    --cc=ppandit@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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.