|WARNING| pw118786 [PATCH] gro : ipv6 changes to support GRO for TCP/ipv6

[checkpatch@dpdk.org]

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

_coding style issues_


ERROR:TRAILING_WHITESPACE: trailing whitespace
#242: FILE: lib/gro/gro_tcp.h:124:
+^Iiph_orig = (char *)(rte_pktmbuf_mtod(pkt_orig, char *) + $

ERROR:CODE_INDENT: code indent should use tabs where possible
#243: FILE: lib/gro/gro_tcp.h:125:
+                l2_offset + pkt_orig->l2_len);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#243: FILE: lib/gro/gro_tcp.h:125:
+                l2_offset + pkt_orig->l2_len);$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#730: FILE: lib/gro/gro_tcp6.c:229:
+^I/* $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#731: FILE: lib/gro/gro_tcp6.c:230:
+^I * Trim the tail padding bytes. The IPv6 header is fixed to $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#732: FILE: lib/gro/gro_tcp6.c:231:
+^I * 40 bytes unlike IPv4 that is variable. The length in the IPv6 header $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#733: FILE: lib/gro/gro_tcp6.c:232:
+^I * contains only length of TCP Header + TCP Payload, whereas IPv4 header contains $

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#750: FILE: lib/gro/gro_tcp6.c:249:
+    memcpy(&key.src_addr[0], &ipv6_hdr->src_addr, sizeof(key.src_addr));$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#751: FILE: lib/gro/gro_tcp6.c:250:
+    memcpy(&key.dst_addr[0], &ipv6_hdr->dst_addr, sizeof(key.dst_addr));$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#763: FILE: lib/gro/gro_tcp6.c:262:
+^I^I^I^I/* $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#764: FILE: lib/gro/gro_tcp6.c:263:
+^I^I^I^I * IP version (4) Traffic Class (8) Flow Label (20) $

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#766: FILE: lib/gro/gro_tcp6.c:265:
+				 * All fields except Traffic class should be same
+				*/

WARNING:BRACES: braces {} are not necessary for single statement blocks
#768: FILE: lib/gro/gro_tcp6.c:267:
+				if (vtc_flow_diff & htonl(0xF00FFFFF)) {
+					continue;
+				}

ERROR:CODE_INDENT: code indent should use tabs where possible
#813: FILE: lib/gro/gro_tcp6.c:312:
+            }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#813: FILE: lib/gro/gro_tcp6.c:312:
+            }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#910: FILE: lib/gro/gro_tcp6.h:15:
+    uint8_t  src_addr[16];$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#911: FILE: lib/gro/gro_tcp6.h:16:
+    uint8_t  dst_addr[16];$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1042: FILE: lib/gro/gro_tcp6.h:147:
+    return (!memcmp(&k1, &k2, sizeof(struct tcp6_flow_key)));$

WARNING:MISSING_EOF_NEWLINE: adding a line without newline at end of file
#1045: FILE: lib/gro/gro_tcp6.h:150:
+#endif

ERROR:TRAILING_WHITESPACE: trailing whitespace
#1193: FILE: lib/gro/rte_gro.c:304:
+^I^I} else if (IS_IPV6_TCP_PKT(pkts[i]->packet_type) && $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#1234: FILE: lib/gro/rte_gro.c:350:
+^I^I^Ii += gro_tcp6_tbl_timeout_flush(&tcp6_tbl, 0, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#1254: FILE: lib/gro/rte_gro.c:371:
+^I^I^I^I^IRTE_GRO_TCP_IPV4 | RTE_GRO_TCP_IPV6 | $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#1279: FILE: lib/gro/rte_gro.c:415:
+^I^I} else if (IS_IPV6_TCP_PKT(pkts[i]->packet_type) && $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#1281: FILE: lib/gro/rte_gro.c:417:
+^I^I^Iif (gro_tcp6_reassemble(pkts[i], tcp6_tbl, $

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#1285: FILE: lib/gro/rte_gro.c:421:
+		}
+		else

total: 15 errors, 10 warnings, 1132 lines checked