|WARNING| pw147513 [PATCH] igc: fix invalid length and corrupted multi-segment mbufs

[checkpatch@dpdk.org]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1574 bytes --]

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

_coding style issues_


ERROR:CORRUPTED_PATCH: patch seems to be corrupt (line wrapped?)
#162: FILE: :510:
rte_mbuf **rx_pkts,

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#166: FILE: :514:
+        if (rxq->offloads & RTE_ETH_RX_OFFLOAD_TIMESTAMP) {

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#167: FILE: :515:
+            if (first_seg == NULL)

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#168: FILE: :516:
+                data_len -= IGC_TS_HDR_LEN;

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#169: FILE: :517:
+            else

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#170: FILE: :518:
+                rxm->data_off -= IGC_TS_HDR_LEN;

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#171: FILE: :519:
+        }

WARNING:EMBEDDED_FILENAME: It's generally not useful to have the filename in the file
#180: FILE: :568:
+                rxm = last_seg;

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (0, 0)
#194: FILE: drivers/net/igc/igc_txrx.c:350:
+        if (rxq->offloads & RTE_ETH_RX_OFFLOAD_TIMESTAMP)
+            data_len -= IGC_TS_HDR_LEN;

total: 1 errors, 8 warnings, 0 checks, 29 lines checked