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

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

[qemudev]

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

_apply patch failure_

Submitter: Martin Weiser <martin.weiser@allegro-packets.com>
Date: Mon, 28 Oct 2024 14:51:23 +0100
DPDK git baseline: Repo:dpdk-next-net-intel
  Branch: main
  CommitID: 69c0ffb893fa9c1c7b00947a2801cef50ebc65d4

Apply patch set 147513 failed:

error: patch fragment without header at line 161: @@ -509,6 +511,12 @@ igc_recv_scattered_pkts(void *rx_queue, struct 

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

[checkpatch]

[-- 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