CHECK: extern prototypes should be avoided in .h files #35: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:91: +extern int bpf_set_dentry_xattr(struct dentry *dentry, const char *name__str, WARNING: line length of 91 exceeds 80 columns #36: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:92: + const struct bpf_dynptr *value_p, int flags) __ksym __weak; WARNING: line length of 95 exceeds 80 columns #37: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:93: +extern int bpf_remove_dentry_xattr(struct dentry *dentry, const char *name__str) __ksym __weak; CHECK: extern prototypes should be avoided in .h files #37: FILE: tools/testing/selftests/bpf/bpf_kfuncs.h:93: +extern int bpf_remove_dentry_xattr(struct dentry *dentry, const char *name__str) __ksym __weak; WARNING: line length of 88 exceeds 80 columns #64: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:84: + err = getxattr(testfile, skel->rodata->xattr_foo, value_out, sizeof(value_out)); WARNING: line length of 92 exceeds 80 columns #66: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:86: + ASSERT_EQ(strncmp(value_out, value_foo, sizeof(value_foo)), 0, "strncmp value_foo"); WARNING: line length of 99 exceeds 80 columns #71: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:91: + ASSERT_OK(setxattr(testfile, skel->rodata->xattr_foo, value_foo, strlen(value_foo) + 1, 0), WARNING: line length of 88 exceeds 80 columns #80: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:100: + err = getxattr(testfile, skel->rodata->xattr_bar, value_out, sizeof(value_out)); WARNING: line length of 94 exceeds 80 columns #82: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:102: + ASSERT_EQ(strncmp(value_out, skel->data->value_bar, sizeof(skel->data->value_bar)), 0, WARNING: line length of 88 exceeds 80 columns #91: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:111: + err = getxattr(testfile, skel->rodata->xattr_bar, value_out, sizeof(value_out)); WARNING: line length of 95 exceeds 80 columns #112: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:132: + err = setxattr(testfile, skel->rodata->xattr_foo, value_foo, strlen(value_foo) + 1, 0); WARNING: line length of 93 exceeds 80 columns #143: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:163: + ASSERT_TRUE(skel->bss->set_security_bpf_bar_success, "set_security_bpf_bar_success"); WARNING: line length of 99 exceeds 80 columns #144: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:164: + ASSERT_TRUE(skel->bss->remove_security_bpf_bar_success, "remove_security_bpf_bar_success"); WARNING: line length of 87 exceeds 80 columns #145: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:165: + ASSERT_TRUE(skel->bss->set_security_selinux_fail, "set_security_selinux_fail"); WARNING: line length of 93 exceeds 80 columns #146: FILE: tools/testing/selftests/bpf/prog_tests/fs_kfuncs.c:166: + ASSERT_TRUE(skel->bss->remove_security_selinux_fail, "remove_security_selinux_fail"); WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #191: new file mode 100644 WARNING: const array should probably be static const #209: FILE: tools/testing/selftests/bpf/progs/test_set_remove_xattr.c:14: +const char xattr_foo[] = "security.bpf.foo"; WARNING: const array should probably be static const #210: FILE: tools/testing/selftests/bpf/progs/test_set_remove_xattr.c:15: +const char xattr_bar[] = "security.bpf.bar"; WARNING: const array should probably be static const #211: FILE: tools/testing/selftests/bpf/progs/test_set_remove_xattr.c:16: +const char xattr_linux[] = "security.selinux"; WARNING: line length of 81 exceeds 80 columns #252: FILE: tools/testing/selftests/bpf/progs/test_set_remove_xattr.c:57: + bpf_dynptr_from_mem(value_bar, sizeof(value_bar), 0, &value_ptr); WARNING: line length of 81 exceeds 80 columns #308: FILE: tools/testing/selftests/bpf/progs/test_set_remove_xattr.c:113: + bpf_dynptr_from_mem(value_bar, sizeof(value_bar), 0, &value_ptr); total: 0 errors, 19 warnings, 2 checks, 285 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 15717b3b5c84 ("selftests/bpf: Test kfuncs that set and remove xattr from BPF programs") 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, 19 warnings, 2 checks, 285 lines checked