|WARNING| pw135855 [PATCH] net/tap: Modified TAP BPF program as per the Kernel-version upgrade requirements.

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#90: 
When multiple queues configured, internally RSS will be enabled and thus TAP BPF RSS byte-code will be loaded on to the Kernel using BPF system calls.

WARNING:TYPO_SPELLING: 'sucessfully' may be misspelled - perhaps 'successfully'?
#107: 
BPF instructions able to load sucessfully on to the Kernel tap device.

WARNING:BRACES: braces {} are not necessary for single statement blocks
#657: FILE: drivers/net/tap/bpf/tap_bpf_program.c:120:
+	if (rsskey == NULL) {
 		return TC_ACT_OK;
 	}

WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#759: FILE: drivers/net/tap/bpf/tap_bpf_program.c:148:
+							bpf_ntohl(n6h.ip6h.saddr.in6_u.u6_addr32[j]);

WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#761: FILE: drivers/net/tap/bpf/tap_bpf_program.c:150:
+							bpf_ntohl(n6h.ip6h.daddr.in6_u.u6_addr32[j]);

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#767: FILE: drivers/net/tap/bpf/tap_bpf_program.c:156:
+		}
+		else {

CHECK:BRACES: Unbalanced braces around else statement
#767: FILE: drivers/net/tap/bpf/tap_bpf_program.c:156:
+		else {

ERROR:SPACING: space required before the open parenthesis '('
#788: FILE: drivers/net/tap/bpf/tap_bpf_program.c:168:
+		if(k == hash)

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#805: FILE: drivers/net/tap/bpf/tap_bpf_program.c:176:
 
+}

WARNING:TYPO_SPELLING: 'aded' may be misspelled - perhaps 'added'?
#1811: FILE: drivers/net/tap/tap_bpf_insns.h:646:
+	{0xa7,    5,    0,        0, 0x40e8aded},

total: 2 errors, 6 warnings, 2 checks, 3314 lines checked