|WARNING| pw107048 [PATCH] [V2] pipeline: support checksum for variable size headers

[dpdklab@iol.unh.edu]

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

Test-Label: iol-testing
Test-Status: WARNING
http://dpdk.org/patch/107048

_apply patch failure_

Submitter: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Date: Tuesday, February 08 2022 21:52:05 
Applied on: CommitID:0e4dc6af06228c8504a5538512cb31ed7bf6cc23
Apply patch set 107048 failed:

Checking patch lib/pipeline/rte_swx_pipeline.c...
Hunk #1 succeeded at 2865 (offset -22 lines).
Hunk #2 succeeded at 2886 (offset -22 lines).
Hunk #3 succeeded at 2915 (offset -22 lines).
Checking patch lib/pipeline/rte_swx_pipeline_internal.h...
error: while searching for:
	INSTR_ALU_SUB_HI, /* dst = H, src = I */

	/* ckadd dst src
	 * dst = dst '+ src[0:1] '+ src[2:3] + ...
	 * dst = H, src = {H, h.header}
	 */
	INSTR_ALU_CKADD_FIELD,    /* src = H */
	INSTR_ALU_CKADD_STRUCT20, /* src = h.header, with sizeof(header) = 20 */
	INSTR_ALU_CKADD_STRUCT,   /* src = h.hdeader, with any sizeof(header) */

	/* cksub dst src
	 * dst = dst '- src
	 * dst = H, src = H
	 */
	INSTR_ALU_CKSUB_FIELD,


error: patch failed: lib/pipeline/rte_swx_pipeline_internal.h:305
Hunk #2 succeeded at 2680 (offset -20 lines).
Hunk #3 succeeded at 2708 (offset -20 lines).
Hunk #4 succeeded at 2738 (offset -20 lines).
Hunk #5 succeeded at 2778 (offset -20 lines).
Hunk #6 succeeded at 2791 (offset -20 lines).
Hunk #7 succeeded at 2800 (offset -20 lines).
Hunk #8 succeeded at 2832 (offset -20 lines).
Hunk #9 succeeded at 2844 (offset -20 lines).
Applied patch lib/pipeline/rte_swx_pipeline.c cleanly.
Applying patch lib/pipeline/rte_swx_pipeline_internal.h with 1 reject...
Rejected hunk #1.
Hunk #2 applied cleanly.
Hunk #3 applied cleanly.
Hunk #4 applied cleanly.
Hunk #5 applied cleanly.
Hunk #6 applied cleanly.
Hunk #7 applied cleanly.
Hunk #8 applied cleanly.
Hunk #9 applied cleanly.
diff a/lib/pipeline/rte_swx_pipeline_internal.h b/lib/pipeline/rte_swx_pipeline_internal.h	(rejected hunks)
@@ -305,16 +305,16 @@ enum instruction_type {
 	INSTR_ALU_SUB_HI, /* dst = H, src = I */
 
 	/* ckadd dst src
-	 * dst = dst '+ src[0:1] '+ src[2:3] + ...
-	 * dst = H, src = {H, h.header}
+	 * dst = dst '+ src[0:1] '+ src[2:3] '+ ...
+	 * dst = H, src = {H, h.header}, '+ = 1's complement addition operator
 	 */
 	INSTR_ALU_CKADD_FIELD,    /* src = H */
-	INSTR_ALU_CKADD_STRUCT20, /* src = h.header, with sizeof(header) = 20 */
-	INSTR_ALU_CKADD_STRUCT,   /* src = h.hdeader, with any sizeof(header) */
+	INSTR_ALU_CKADD_STRUCT20, /* src = h.header, with sizeof(header) = 20 bytes. */
+	INSTR_ALU_CKADD_STRUCT,   /* src = h.header, with sizeof(header) any 4-byte multiple. */
 
 	/* cksub dst src
 	 * dst = dst '- src
-	 * dst = H, src = H
+	 * dst = H, src = H, '- = 1's complement subtraction operator
 	 */
 	INSTR_ALU_CKSUB_FIELD,
 

https://lab.dpdk.org/results/dashboard/patchsets/20954/

UNH-IOL DPDK Community Lab