|WARNING| pw136219 [RFC 2/2] tap: rework BPF handling

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#1009: FILE: drivers/net/tap/bpf/tap_rss.c:92:
+		return 0;
+	} else {

WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#1014: FILE: drivers/net/tap/bpf/tap_rss.c:97:
+						&src_dst_port, sizeof(src_dst_port), BPF_HDR_START_NET))

WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#1044: FILE: drivers/net/tap/bpf/tap_rss.c:127:
+			if (bpf_skb_load_bytes_relative(skb, *off, &xh, sizeof(xh), BPF_HDR_START_NET))

WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#1051: FILE: drivers/net/tap/bpf/tap_rss.c:134:
+			if (bpf_skb_load_bytes_relative(skb, *off, &xh, sizeof(xh), BPF_HDR_START_NET))

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#1099: FILE: drivers/net/tap/bpf/tap_rss.c:182:
+		return 0;
+	} else {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#1141: FILE: drivers/net/tap/bpf/tap_rss.c:224:
+		__u16		slave_dev_queue_mapping;

WARNING:EMBEDDED_FUNCTION_NAME: Prefer using '"%s...", __func__' to using 'rss_flow_action', this function's name, in a string
#1160: FILE: drivers/net/tap/bpf/tap_rss.c:243:
+	char fmt[] = "rss_flow_action classid:%u
";

ERROR:FUNCTION_WITHOUT_ARGS: Bad function definition - struct tap_rss *open_and_load() should probably be struct tap_rss *open_and_load(void)
#3973: FILE: drivers/net/tap/tap_rss.skel.h:28:
+	static inline struct tap_rss *open_and_load();

WARNING:LINE_CONTINUATIONS: Avoid line continuations in quoted strings
#4125: FILE: drivers/net/tap/tap_rss.skel.h:180:
+	return (const void *)"\

WARNING:LONG_LINE: line length of 107 exceeds 100 columns
#15550: FILE: drivers/net/tap/tap_rss.skel.h:11605:
+struct tap_rss *tap_rss::open(const struct bpf_object_open_opts *opts) { return tap_rss__open_opts(opts); }

CHECK:CAMELCASE: Avoid CamelCase: <_Static_assert>
#15563: FILE: drivers/net/tap/tap_rss.skel.h:11618:
+#define _Static_assert static_assert

total: 1 errors, 9 warnings, 1 checks, 12742 lines checked