automatic DPDK test reports
 help / color / mirror / Atom feed
From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Jakub Grajciar <jgrajcia@cisco.com>
Subject: [dpdk-test-report] |WARNING| pw54751 [PATCH v1] net/memif: multi-process support
Date: Thu, 13 Jun 2019 15:57:35 +0200 (CEST)	[thread overview]
Message-ID: <20190613135735.B90B01D549@dpdk.org> (raw)
In-Reply-To: <20190613064248.4930-1-jgrajcia@cisco.com>

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/54751

_coding style issues_


WARNING:LONG_LINE: line over 90 characters
#399: FILE: drivers/net/memif/rte_eth_memif.c:243:
+	struct pmd_process_private *proc_private = rte_eth_devices[mq->in_port].process_private;

WARNING:LONG_LINE: line over 90 characters
#436: FILE: drivers/net/memif/rte_eth_memif.c:322:
+			       (uint8_t *)memif_get_buffer(proc_private, d0) + src_off, cp_len);

WARNING:LONG_LINE: line over 90 characters
#447: FILE: drivers/net/memif/rte_eth_memif.c:372:
+	struct pmd_process_private *proc_private = rte_eth_devices[mq->in_port].process_private;

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ring == NULL'
#743: FILE: drivers/net/memif/rte_eth_memif.c:728:
+			if ((ring == NULL) || (ring->cookie != MEMIF_COOKIE)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ring->cookie != MEMIF_COOKIE'
#743: FILE: drivers/net/memif/rte_eth_memif.c:728:
+			if ((ring == NULL) || (ring->cookie != MEMIF_COOKIE)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ring == NULL'
#776: FILE: drivers/net/memif/rte_eth_memif.c:744:
+			if ((ring == NULL) || (ring->cookie != MEMIF_COOKIE)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ring->cookie != MEMIF_COOKIE'
#776: FILE: drivers/net/memif/rte_eth_memif.c:744:
+			if ((ring == NULL) || (ring->cookie != MEMIF_COOKIE)) {

CHECK:BRACES: braces {} should be used on all arms of this statement
#815: FILE: drivers/net/memif/rte_eth_memif.c:794:
+	if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
[...]
-		(*dev->dev_ops->rx_queue_release)(dev->data->rx_queues[i]);
[...]

CHECK:BRACES: Unbalanced braces around else statement
#829: FILE: drivers/net/memif/rte_eth_memif.c:804:
+	} else

WARNING:LONG_LINE: line over 90 characters
#905: FILE: drivers/net/memif/rte_eth_memif.c:1051:
+		rte_zmalloc(name, sizeof(struct pmd_process_private), RTE_CACHE_LINE_SIZE);

ERROR:CODE_INDENT: code indent should use tabs where possible
#947: FILE: drivers/net/memif/rte_eth_memif.c:1264:
+ ^I^Iif (!eth_dev) {$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#947: FILE: drivers/net/memif/rte_eth_memif.c:1264:
+ ^I^Iif (!eth_dev) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#947: FILE: drivers/net/memif/rte_eth_memif.c:1264:
+ ^I^Iif (!eth_dev) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#948: FILE: drivers/net/memif/rte_eth_memif.c:1265:
+ ^I^I^IMIF_LOG(ERR, "Failed to probe %s", name);$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#948: FILE: drivers/net/memif/rte_eth_memif.c:1265:
+ ^I^I^IMIF_LOG(ERR, "Failed to probe %s", name);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#948: FILE: drivers/net/memif/rte_eth_memif.c:1265:
+ ^I^I^IMIF_LOG(ERR, "Failed to probe %s", name);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#949: FILE: drivers/net/memif/rte_eth_memif.c:1266:
+ ^I^I^Ireturn -1;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#949: FILE: drivers/net/memif/rte_eth_memif.c:1266:
+ ^I^I^Ireturn -1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#949: FILE: drivers/net/memif/rte_eth_memif.c:1266:
+ ^I^I^Ireturn -1;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#950: FILE: drivers/net/memif/rte_eth_memif.c:1267:
+ ^I^I}$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#950: FILE: drivers/net/memif/rte_eth_memif.c:1267:
+ ^I^I}$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#950: FILE: drivers/net/memif/rte_eth_memif.c:1267:
+ ^I^I}$

ERROR:CODE_INDENT: code indent should use tabs where possible
#952: FILE: drivers/net/memif/rte_eth_memif.c:1269:
+ ^I^Ieth_dev->dev_ops = &ops;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#952: FILE: drivers/net/memif/rte_eth_memif.c:1269:
+ ^I^Ieth_dev->dev_ops = &ops;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#952: FILE: drivers/net/memif/rte_eth_memif.c:1269:
+ ^I^Ieth_dev->dev_ops = &ops;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#953: FILE: drivers/net/memif/rte_eth_memif.c:1270:
+ ^I^Ieth_dev->device = &vdev->device;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#953: FILE: drivers/net/memif/rte_eth_memif.c:1270:
+ ^I^Ieth_dev->device = &vdev->device;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#953: FILE: drivers/net/memif/rte_eth_memif.c:1270:
+ ^I^Ieth_dev->device = &vdev->device;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#954: FILE: drivers/net/memif/rte_eth_memif.c:1271:
+ ^I^Ieth_dev->rx_pkt_burst = eth_memif_rx;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#954: FILE: drivers/net/memif/rte_eth_memif.c:1271:
+ ^I^Ieth_dev->rx_pkt_burst = eth_memif_rx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#954: FILE: drivers/net/memif/rte_eth_memif.c:1271:
+ ^I^Ieth_dev->rx_pkt_burst = eth_memif_rx;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#955: FILE: drivers/net/memif/rte_eth_memif.c:1272:
+ ^I^Ieth_dev->tx_pkt_burst = eth_memif_rx;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#955: FILE: drivers/net/memif/rte_eth_memif.c:1272:
+ ^I^Ieth_dev->tx_pkt_burst = eth_memif_rx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#955: FILE: drivers/net/memif/rte_eth_memif.c:1272:
+ ^I^Ieth_dev->tx_pkt_burst = eth_memif_rx;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#957: FILE: drivers/net/memif/rte_eth_memif.c:1274:
+ ^I^Iif (!rte_eal_primary_proc_alive(NULL)) {$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#957: FILE: drivers/net/memif/rte_eth_memif.c:1274:
+ ^I^Iif (!rte_eal_primary_proc_alive(NULL)) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#957: FILE: drivers/net/memif/rte_eth_memif.c:1274:
+ ^I^Iif (!rte_eal_primary_proc_alive(NULL)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#958: FILE: drivers/net/memif/rte_eth_memif.c:1275:
+ ^I^I^IMIF_LOG(ERR, "Primary process is missing");$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#958: FILE: drivers/net/memif/rte_eth_memif.c:1275:
+ ^I^I^IMIF_LOG(ERR, "Primary process is missing");$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#958: FILE: drivers/net/memif/rte_eth_memif.c:1275:
+ ^I^I^IMIF_LOG(ERR, "Primary process is missing");$

ERROR:CODE_INDENT: code indent should use tabs where possible
#959: FILE: drivers/net/memif/rte_eth_memif.c:1276:
+ ^I^I^Ireturn -1;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#959: FILE: drivers/net/memif/rte_eth_memif.c:1276:
+ ^I^I^Ireturn -1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#959: FILE: drivers/net/memif/rte_eth_memif.c:1276:
+ ^I^I^Ireturn -1;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#960: FILE: drivers/net/memif/rte_eth_memif.c:1277:
+ ^I^I}$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#960: FILE: drivers/net/memif/rte_eth_memif.c:1277:
+ ^I^I}$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#960: FILE: drivers/net/memif/rte_eth_memif.c:1277:
+ ^I^I}$

ERROR:CODE_INDENT: code indent should use tabs where possible
#962: FILE: drivers/net/memif/rte_eth_memif.c:1279:
+ ^I^Ieth_dev->process_private = (struct pmd_process_private *)$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#962: FILE: drivers/net/memif/rte_eth_memif.c:1279:
+ ^I^Ieth_dev->process_private = (struct pmd_process_private *)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#962: FILE: drivers/net/memif/rte_eth_memif.c:1279:
+ ^I^Ieth_dev->process_private = (struct pmd_process_private *)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#963: FILE: drivers/net/memif/rte_eth_memif.c:1280:
+ ^I^I^Irte_zmalloc(name,$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#963: FILE: drivers/net/memif/rte_eth_memif.c:1280:
+ ^I^I^Irte_zmalloc(name,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#963: FILE: drivers/net/memif/rte_eth_memif.c:1280:
+ ^I^I^Irte_zmalloc(name,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#964: FILE: drivers/net/memif/rte_eth_memif.c:1281:
+ ^I^I^I^Isizeof(struct pmd_process_private),$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#964: FILE: drivers/net/memif/rte_eth_memif.c:1281:
+ ^I^I^I^Isizeof(struct pmd_process_private),$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#964: FILE: drivers/net/memif/rte_eth_memif.c:1281:
+ ^I^I^I^Isizeof(struct pmd_process_private),$

ERROR:CODE_INDENT: code indent should use tabs where possible
#965: FILE: drivers/net/memif/rte_eth_memif.c:1282:
+ ^I^I^I^IRTE_CACHE_LINE_SIZE);$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#965: FILE: drivers/net/memif/rte_eth_memif.c:1282:
+ ^I^I^I^IRTE_CACHE_LINE_SIZE);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#965: FILE: drivers/net/memif/rte_eth_memif.c:1282:
+ ^I^I^I^IRTE_CACHE_LINE_SIZE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#966: FILE: drivers/net/memif/rte_eth_memif.c:1283:
+ ^I^Iif (eth_dev->process_private == NULL) {$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#966: FILE: drivers/net/memif/rte_eth_memif.c:1283:
+ ^I^Iif (eth_dev->process_private == NULL) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#966: FILE: drivers/net/memif/rte_eth_memif.c:1283:
+ ^I^Iif (eth_dev->process_private == NULL) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#967: FILE: drivers/net/memif/rte_eth_memif.c:1284:
+ ^I^I^IMIF_LOG(ERR,$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#967: FILE: drivers/net/memif/rte_eth_memif.c:1284:
+ ^I^I^IMIF_LOG(ERR,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#967: FILE: drivers/net/memif/rte_eth_memif.c:1284:
+ ^I^I^IMIF_LOG(ERR,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#968: FILE: drivers/net/memif/rte_eth_memif.c:1285:
+ ^I^I^I^I"Failed to alloc memory for process private");$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#968: FILE: drivers/net/memif/rte_eth_memif.c:1285:
+ ^I^I^I^I"Failed to alloc memory for process private");$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#968: FILE: drivers/net/memif/rte_eth_memif.c:1285:
+ ^I^I^I^I"Failed to alloc memory for process private");$

ERROR:CODE_INDENT: code indent should use tabs where possible
#969: FILE: drivers/net/memif/rte_eth_memif.c:1286:
+ ^I^I^Ireturn -1;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#969: FILE: drivers/net/memif/rte_eth_memif.c:1286:
+ ^I^I^Ireturn -1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#969: FILE: drivers/net/memif/rte_eth_memif.c:1286:
+ ^I^I^Ireturn -1;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#970: FILE: drivers/net/memif/rte_eth_memif.c:1287:
+ ^I^I}$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#970: FILE: drivers/net/memif/rte_eth_memif.c:1287:
+ ^I^I}$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#970: FILE: drivers/net/memif/rte_eth_memif.c:1287:
+ ^I^I}$

ERROR:CODE_INDENT: code indent should use tabs where possible
#972: FILE: drivers/net/memif/rte_eth_memif.c:1289:
+ ^I^Irte_eth_dev_probing_finish(eth_dev);$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#972: FILE: drivers/net/memif/rte_eth_memif.c:1289:
+ ^I^Irte_eth_dev_probing_finish(eth_dev);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#972: FILE: drivers/net/memif/rte_eth_memif.c:1289:
+ ^I^Irte_eth_dev_probing_finish(eth_dev);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#974: FILE: drivers/net/memif/rte_eth_memif.c:1291:
+ ^I^Ireturn 0;$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#974: FILE: drivers/net/memif/rte_eth_memif.c:1291:
+ ^I^Ireturn 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#974: FILE: drivers/net/memif/rte_eth_memif.c:1291:
+ ^I^Ireturn 0;$

total: 23 errors, 50 warnings, 6 checks, 924 lines checked

           reply	other threads:[~2019-06-13 13:57 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20190613064248.4930-1-jgrajcia@cisco.com>]

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=20190613135735.B90B01D549@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=jgrajcia@cisco.com \
    --cc=test-report@dpdk.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).