automatic DPDK test reports
 help / color / mirror / Atom feed
From: sys_stv@intel.com
To: wenzhuo.lu@intel.com, test-report@dpdk.org
Subject: [dpdk-test-report] [PatchWork]|ERROR| pw12374-12377 ixgbe: VF supports mailbox interruption for PF link up/down
Date: 04 May 2016 22:39:57 -0700	[thread overview]
Message-ID: <acb6cf$svu912@fmsmga002.fm.intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 2930 bytes --]


Test-Label: Intel Niantic on Fedora
Test-Status: ERROR

Patchwork ID: 12374-12377
http://www.dpdk.org/dev/patchwork/patch/12377/
Submitter: Wenzhuo Lu <wenzhuo.lu@intel.com>
Date: Thu,  5 May 2016 05:10:43 +0800
DPDK git baseline: 351637731a0e1589d48b0d9972a006c414378c8d

Check patch error:
12376: 
WARNING: quoted string split across lines
#174: FILE: drivers/net/e1000/igb_ethdev.c:2705:
+PMD_INIT_LOG(ERR, "Igb VF reset: "
+     "Failed to start device.");

WARNING: quoted string split across lines
#183: FILE: drivers/net/e1000/igb_ethdev.c:2714:
+PMD_INIT_LOG(INFO, "Igb VF reset: "
+     "Failed to update link.");

ERROR: need consistent spacing around '*' (ctx:WxO)
#259: FILE: drivers/net/e1000/igb_rxtx.c:2538:
+ struct rte_mbuf __rte_unused **tx_pkts,
                              ^

  ERROR: need consistent spacing around '*' (ctx:WxO)
  #276: FILE: drivers/net/e1000/igb_rxtx.c:2555:
  + struct rte_mbuf __rte_unused **rx_pkts,
                              ^

  total: 2 errors, 2 warnings, 221 lines checked

  /home/patchWorkOrg/patches/dpdk-dev-4-4-igb-automatic-link-recovery-on-VF.patch has style problems, please review.

  If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.

  12377: 
  WARNING: quoted string split across lines
  #137: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7211:
  +PMD_INIT_LOG(ERR, "Ixgbe VF reset: "
  +     "Failed to start device.");

  WARNING: suspect code indent for conditional statements (24, 24)
  #142: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7216:
  +if (dev->data->dev_conf.intr_conf.lsc == 0)
  +diag = ixgbe_dev_link_update(dev, 0);

  WARNING: quoted string split across lines
  #146: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7220:
  +PMD_INIT_LOG(INFO, "Ixgbe VF reset: "
  +     "Failed to update link.");

  ERROR: need consistent spacing around '*' (ctx:WxO)
  #251: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5257:
  +       struct rte_mbuf __rte_unused **rx_pkts,
                                    ^

ERROR: need consistent spacing around '*' (ctx:WxO)
#267: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5273:
+       struct rte_mbuf __rte_unused **tx_pkts,
                                    ^

total: 2 errors, 3 warnings, 227 lines checked

/home/patchWorkOrg/patches/dpdk-dev-3-4-ixgbe-automatic-link-recovery-on-VF.patch has style problems, please review.

If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.


Compilation:
OS: fedora
Nic: niantic
GCC: gcc_x86-64, 4.8.3
ICC:16.0.2
i686-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-combined: compile pass
i686-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-debug: compile pass
x86_64-native-linuxapp-gcc-shared: compile pass
x86_64-native-linuxapp-clang: compile pass


DPDK STV team 


                 reply	other threads:[~2016-05-05  5:39 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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='acb6cf$svu912@fmsmga002.fm.intel.com' \
    --to=sys_stv@intel.com \
    --cc=test-report@dpdk.org \
    --cc=wenzhuo.lu@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 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).