[dpdk-test-report] |WARNING| pw85504 [PATCH v1 4/5] net/iavf_be: add Rx Tx burst support

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:MASK_THEN_SHIFT: Possible precedence defect with mask then right shift - may need parentheses
#97: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:168:
+	m->l2_len = offset & IAVF_TXD_QW1_MACLEN_MASK >>

WARNING:MASK_THEN_SHIFT: Possible precedence defect with mask then right shift - may need parentheses
#99: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:170:
+	m->l3_len = offset & IAVF_TXD_QW1_IPLEN_MASK >>

WARNING:MASK_THEN_SHIFT: Possible precedence defect with mask then right shift - may need parentheses
#101: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:172:
+	m->l4_len = offset & IAVF_TXD_QW1_L4LEN_MASK >>

WARNING:BRACES: braces {} are not necessary for single statement blocks
#170: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:241:
+			if ((head & 0x3) == 0) {
+				rte_prefetch0(&ring_dma[head]);
+			}

CHECK:SPACING: spaces preferred around that '&' (ctx:WxV)
#185: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:256:
+			cmd = (d->cmd_type_offset_bsz &IAVF_TXD_QW1_CMD_MASK) >>
 			                              ^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#205: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:276:
+			desc_addr = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'first->pkt_len > rxq->max_pkt_len'
#249: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:320:
+		if ((!(ol_flags & PKT_TX_TCP_SEG)) &&
+		    (first->pkt_len > rxq->max_pkt_len)) {

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#318: FILE: drivers/net/iavf_be/iavf_be_rxtx.c:389:
+			desc_addr = (void *)(uintptr_t)rte_iavf_emu_get_dma_vaddr(

CHECK:CAMELCASE: Avoid CamelCase: <PRIx64>
#443: FILE: drivers/net/iavf_be/iavf_be_rxtx.h:116:
+	printf("Queue %d Rx_desc %d: QW0: 0x%016"PRIx64" QW1: 0x%016"PRIx64

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#443: FILE: drivers/net/iavf_be/iavf_be_rxtx.h:116:
+	printf("Queue %d Rx_desc %d: QW0: 0x%016"PRIx64" QW1: 0x%016"PRIx64

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#444: FILE: drivers/net/iavf_be/iavf_be_rxtx.h:117:
+	       " QW2: 0x%016"PRIx64" QW3: 0x%016"PRIx64"
", txq->queue_id,

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#460: FILE: drivers/net/iavf_be/iavf_be_rxtx.h:133:
+	type = (enum iavf_tx_desc_dtype_value)rte_le_to_cpu_64(

CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
#475: FILE: drivers/net/iavf_be/iavf_be_rxtx.h:148:
+	printf("Queue %d %s %d: QW0: 0x%016"PRIx64" QW1: 0x%016"PRIx64"
",

total: 0 errors, 4 warnings, 9 checks, 406 lines checked
Warning in drivers/net/iavf_be/iavf_be_rxtx.c:
Using rte_atomicNN_xxx