[dpdk-test-report] [PatchWork]|ERROR| pw13200-13207 lib/librte_ether: support device reset

[sys_stv at intel.com]

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

Patchwork ID: 13200-13207
http://www.dpdk.org/dev/patchwork/patch/13207/
Submitter: Wenzhuo Lu <wenzhuo.lu at intel.com>
Date: Mon,  6 Jun 2016 13:40:46 +0800
DPDK git baseline: f243d9b04b4bf17d7ccdae179fa43b04d8a87317

Check patch error:
13201: 
ERROR: spaces prohibited around that ':' (ctx:WxW)
#38: FILE: lib/librte_ether/rte_ethdev.h:360:
+enable_lro       : 1, /**< Enable LRO */
                 ^
 
 ERROR: spaces prohibited around that ':' (ctx:WxW)
 #39: FILE: lib/librte_ether/rte_ethdev.h:361:
 +lock_mode        : 1; /**< Using lock path */
                 ^
 
 ERROR: spaces prohibited around that ':' (ctx:WxW)
 #52: FILE: lib/librte_ether/rte_ethdev.h:646:
 +hw_vlan_insert_pvid : 1,
                    ^

ERROR: spaces prohibited around that ':' (ctx:WxW)
#54: FILE: lib/librte_ether/rte_ethdev.h:648:
+lock_mode : 1;
          ^
  
  total: 4 errors, 0 warnings, 80 lines checked
  
  /home/patchWorkOrg/patches/dpdk-dev-2-8-lib-librte_ether-defind-RX-TX-lock-mode.patch has style problems, please review.
  
  If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.
  
  13202: 
  ERROR: need consistent spacing around '*' (ctx:WxV)
  #290: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5263:
  +void __rte_unused *rx_queue,
                  ^
  
  ERROR: need consistent spacing around '*' (ctx:WxO)
  #291: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5264:
  +struct rte_mbuf __rte_unused **rx_pkts,
                             ^
 
 ERROR: need consistent spacing around '*' (ctx:WxV)
 #307: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5281:
 +void __rte_unused *rx_queue,
                  ^
  
  ERROR: need consistent spacing around '*' (ctx:WxO)
  #308: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5282:
  +struct rte_mbuf __rte_unused **rx_pkts,
                             ^
 
 total: 4 errors, 0 warnings, 322 lines checked
 
 /home/patchWorkOrg/patches/dpdk-dev-3-8-ixgbe-RX-TX-with-lock-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.
 
 13203: 
 WARNING: quoted string split across lines
 #131: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7246:
 +PMD_INIT_LOG(INFO, "Ixgbe VF reset: "
 +     "Failed to update link.");
 
 WARNING: quoted string split across lines
 #139: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7254:
 +PMD_INIT_LOG(ERR, "Ixgbe VF reset: "
 +     "Failed to start device.");
 
 WARNING: quoted string split across lines
 #148: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:7263:
 +PMD_INIT_LOG(INFO, "Ixgbe VF reset: "
 +     "Failed to update link.");
 
 ERROR: need consistent spacing around '*' (ctx:WxV)
 #287: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5316:
 +ixgbevf_recv_pkts_fake(void __rte_unused *rx_queue,
                                           ^
   
   ERROR: need consistent spacing around '*' (ctx:WxO)
   #288: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5317:
   +       struct rte_mbuf __rte_unused **rx_pkts,
                                    ^

ERROR: need consistent spacing around '*' (ctx:WxV)
#298: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5327:
+ixgbevf_xmit_pkts_fake(void __rte_unused *tx_queue,
                                          ^
  
  ERROR: need consistent spacing around '*' (ctx:WxO)
  #299: FILE: drivers/net/ixgbe/ixgbe_rxtx.c:5328:
  +       struct rte_mbuf __rte_unused **tx_pkts,
                                    ^

total: 4 errors, 3 warnings, 246 lines checked

/home/patchWorkOrg/patches/dpdk-dev-4-8-ixgbe-implement-device-reset-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.

13205: 
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#121: FILE: drivers/net/e1000/e1000_ethdev.h:324:
+volatile union e1000_adv_rx_desc *rx_ring; /**< RX ring virtual 
+address. */

WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#123: FILE: drivers/net/e1000/e1000_ethdev.h:326:
+volatile uint32_t   *rdt_reg_addr; /**< RDT register address. */

WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#124: FILE: drivers/net/e1000/e1000_ethdev.h:327:
+volatile uint32_t   *rdh_reg_addr; /**< RDH register address. */

WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#147: FILE: drivers/net/e1000/e1000_ethdev.h:350:
+volatile union e1000_adv_tx_desc *tx_ring; /**< TX ring address */

WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#150: FILE: drivers/net/e1000/e1000_ethdev.h:353:
+volatile uint32_t      *tdt_reg_addr; /**< Address of TDT register. */

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

WARNING: quoted string split across lines
#267: FILE: drivers/net/e1000/igb_ethdev.c:2724:
+PMD_INIT_LOG(ERR, "Igb VF reset: "
+     "Failed to start device.");

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

ERROR: need consistent spacing around '*' (ctx:WxV)
#456: FILE: drivers/net/e1000/igb_rxtx.c:2432:
+eth_igbvf_xmit_pkts_fake(void __rte_unused *tx_queue,
                                            ^

ERROR: need consistent spacing around '*' (ctx:WxO)
#457: FILE: drivers/net/e1000/igb_rxtx.c:2433:
+ struct rte_mbuf __rte_unused **tx_pkts,
                              ^
  
  ERROR: need consistent spacing around '*' (ctx:WxV)
  #467: FILE: drivers/net/e1000/igb_rxtx.c:2443:
  +eth_igbvf_recv_pkts_fake(void __rte_unused *rx_queue,
                                              ^
  
  ERROR: need consistent spacing around '*' (ctx:WxO)
  #468: FILE: drivers/net/e1000/igb_rxtx.c:2444:
  + struct rte_mbuf __rte_unused **rx_pkts,
                              ^
  
  total: 4 errors, 8 warnings, 413 lines checked
  
  /home/patchWorkOrg/patches/dpdk-dev-6-8-igb-implement-device-reset-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.
  
  13207: 
  ERROR: need consistent spacing around '*' (ctx:WxV)
  #158: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1377:
  +i40evf_recv_pkts_detach(void __rte_unused *rx_queue,
                                             ^
 
 ERROR: need consistent spacing around '*' (ctx:WxO)
 #159: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1378:
 +struct rte_mbuf __rte_unused **rx_pkts,
                             ^
 
 ERROR: need consistent spacing around '*' (ctx:WxV)
 #166: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1385:
 +i40evf_xmit_pkts_detach(void __rte_unused *tx_queue,
                                            ^

ERROR: need consistent spacing around '*' (ctx:WxO)
#167: FILE: drivers/net/i40e/i40e_ethdev_vf.c:1386:
+struct rte_mbuf __rte_unused **tx_pkts,
                             ^
 
 total: 4 errors, 0 warnings, 205 lines checked
 
 /home/patchWorkOrg/patches/dpdk-dev-8-8-i40e-implement-device-reset-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.
 
 
 patch file error:
 13203: 
 patching file doc/guides/rel_notes/release_16_07.rst
 Hunk #1 succeeded at 59 with fuzz 2 (offset 6 lines).
 patching file drivers/net/ixgbe/ixgbe_ethdev.c Hunk #1 succeeded at 378 (offset -3 lines).
 Hunk #2 succeeded at 585 (offset -3 lines).
 Hunk #3 succeeded at 4048 (offset -15 lines).
 Hunk #4 FAILED at 7197.
 1 out of 4 hunks FAILED -- saving rejects to file drivers/net/ixgbe/ixgbe_ethdev.c.rej
 patching file drivers/net/ixgbe/ixgbe_ethdev.h patching file drivers/net/ixgbe/ixgbe_rxtx.c
 
 13205: 
 patching file doc/guides/rel_notes/release_16_07.rst
 Hunk #1 succeeded at 59 with fuzz 2 (offset 6 lines).
 patching file drivers/net/e1000/e1000_ethdev.h patching file drivers/net/e1000/igb_ethdev.c Hunk #1 FAILED at 268.
 Hunk #2 succeeded at 400 (offset -9 lines).
 Hunk #3 FAILED at 2664.
 2 out of 3 hunks FAILED -- saving rejects to file drivers/net/e1000/igb_ethdev.c.rej
 patching file drivers/net/e1000/igb_rxtx.c
 
 DPDK STV team