WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #45: new file mode 100644 WARNING: line length of 81 exceeds 80 columns #105: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:56: +static int send_cmd(int fd, __u16 nlmsg_type, __u32 nlmsg_pid, __u16 nlmsg_flags, CHECK: Alignment should match open parenthesis #106: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:57: +static int send_cmd(int fd, __u16 nlmsg_type, __u32 nlmsg_pid, __u16 nlmsg_flags, + __u8 genl_cmd, __u16 nla_type, CHECK: No space is necessary after a cast #123: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:74: + na = (struct nlattr *) GENLMSG_DATA(&msg); CHECK: No space is necessary after a cast #129: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:80: + buf = (char *) &msg; CHECK: No space is necessary after a cast #134: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:85: + while ((r = sendto(fd, buf, buflen, 0, (struct sockaddr *) &nladdr, CHECK: braces {} should be used on all arms of this statement #136: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:87: + if (r > 0) { [...] + } else if (errno != EAGAIN) [...] WARNING: line length of 84 exceeds 80 columns #160: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:111: + if (!ASSERT_OK(system("ip addr add 127.0.1.0/8 dev lo"), "add server node")) WARNING: line length of 89 exceeds 80 columns #163: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:114: + if (!ASSERT_OK(system("ip addr add 127.0.2.0/8 dev lo"), "server via risk path")) CHECK: No space is necessary after a cast #190: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:141: + ret = bind(fd, (struct sockaddr *) &nl_src, sizeof(nl_src)); CHECK: Unnecessary parentheses around 'ret < 0' #202: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:153: + if (!ASSERT_FALSE(msg.n.nlmsg_type == NLMSG_ERROR || (ret < 0) || + !NLMSG_OK((&msg.n), ret), "nl_family response")) CHECK: Unnecessary parentheses around '&msg.n' #202: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:153: + if (!ASSERT_FALSE(msg.n.nlmsg_type == NLMSG_ERROR || (ret < 0) || + !NLMSG_OK((&msg.n), ret), "nl_family response")) CHECK: No space is necessary after a cast #206: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:157: + nl = (struct nlattr *) GENLMSG_DATA(&msg); CHECK: No space is necessary after a cast #207: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:158: + nl = (struct nlattr *) ((char *) nl + NLA_ALIGN(nl->nla_len)); CHECK: No space is necessary after a cast #211: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:162: + smc_nl_family_id = *(uint16_t *) NLA_DATA(nl); CHECK: No space is necessary after a cast #240: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:191: + ret = bind(fd, (struct sockaddr *) &nl_src, sizeof(nl_src)); WARNING: line length of 82 exceeds 80 columns #251: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:202: + if (!ASSERT_FALSE((ret < 0) || !NLMSG_OK((&msg.n), ret), "ueid response")) CHECK: Unnecessary parentheses around 'ret < 0' #251: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:202: + if (!ASSERT_FALSE((ret < 0) || !NLMSG_OK((&msg.n), ret), "ueid response")) CHECK: Unnecessary parentheses around '&msg.n' #251: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:202: + if (!ASSERT_FALSE((ret < 0) || !NLMSG_OK((&msg.n), ret), "ueid response")) WARNING: line length of 96 exceeds 80 columns #258: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:209: + if (!ASSERT_FALSE((err->error && err->error != -ENOENT), "ueid remove")) CHECK: No space is necessary after a cast #291: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:242: + (void) smc_ueid(SMC_NETLINK_REMOVE_UEID); CHECK: No space is necessary after a cast #301: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:252: + (void) smc_ueid(SMC_NETLINK_REMOVE_UEID); WARNING: line length of 84 exceeds 80 columns #304: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:255: +static pthread_t create_service(const char *ip, int port, void *(*handler) (void *)) WARNING: line length of 84 exceeds 80 columns #493: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:444: + * 2. link-2 is not suitable for using SMC, because the mode of this link is kind of WARNING: line length of 89 exceeds 80 columns #495: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:446: + * 3. link-3 is also not suitable for using SMC, because the RDMA link is unavailable and WARNING: line length of 85 exceeds 80 columns #518: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:469: + /* Mock the process of transparent replacement, since we will modify protocol WARNING: line length of 81 exceeds 80 columns #539: FILE: tools/testing/selftests/bpf/prog_tests/test_bpf_smc.c:490: + service_3 = create_service(SERVER_IP_VIA_RISK_PATH, SERVICE_3, service3); WARNING: line length of 100 exceeds 80 columns #681: FILE: tools/testing/selftests/bpf/progs/bpf_smc.c:91: +int BPF_PROG(bpf_smc_set_tcp_option_cond, const struct tcp_sock *tp, struct inet_request_sock *ireq) WARNING: line length of 81 exceeds 80 columns #690: FILE: tools/testing/selftests/bpf/progs/bpf_smc.c:100: + return smc_check_ip(tp->inet_conn.icsk_inet.sk.__sk_common.skc_rcv_saddr, CHECK: No space is necessary after a cast #697: FILE: tools/testing/selftests/bpf/progs/bpf_smc.c:107: + .set_option = (void *) bpf_smc_set_tcp_option, CHECK: No space is necessary after a cast #698: FILE: tools/testing/selftests/bpf/progs/bpf_smc.c:108: + .set_option_cond = (void *) bpf_smc_set_tcp_option_cond, total: 0 errors, 13 warnings, 18 checks, 651 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit e2591f87dfcd ("bpf/selftests: add selftest for bpf_smc_ops") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 13 warnings, 18 checks, 651 lines checked