WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #28: new file mode 100644 WARNING: line length of 87 exceeds 80 columns #128: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:96: + err = write_cgroup_file(cgroups[i].path, "cgroup.freeze", "1"); CHECK: Blank lines aren't necessary after an open brace '{' #135: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:103: + if (!cgroups[i].target) { + WARNING: line length of 90 exceeds 80 columns #136: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:104: + err = write_cgroup_file(cgroups[i].path, "cgroup.subtree_control", WARNING: line length of 84 exceeds 80 columns #147: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:115: + err = write_cgroup_file(cgroups[i].path, "memory.max", buf); WARNING: line length of 83 exceeds 80 columns #152: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:120: + write_cgroup_file(cgroups[i].path, "memory.swap.max", buf); CHECK: Blank lines aren't necessary before a close brace '}' #154: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:122: + + } WARNING: line length of 86 exceeds 80 columns #165: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:133: + err = write_cgroup_file(cgroups[i].path, "cgroup.procs", buf); CHECK: Unnecessary parentheses around 'pid == cgroups[i].pid' #207: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:175: + if (!ASSERT_OK(cgroups[i].victim != + (pid == cgroups[i].pid), + "correct process was killed")) { CHECK: Comparison to NULL could be written "!strstr" #226: FILE: tools/testing/selftests/bpf/prog_tests/test_oom.c:194: + if (!ASSERT_OK(strstr(buf, "oom_kill 1") == NULL, CHECK: No space is necessary after a cast #277: FILE: tools/testing/selftests/bpf/progs/test_oom.c:10: +void bpf_rcu_read_lock(void) __ksym; CHECK: No space is necessary after a cast #278: FILE: tools/testing/selftests/bpf/progs/test_oom.c:11: +void bpf_rcu_read_unlock(void) __ksym; CHECK: No space is necessary after a cast #281: FILE: tools/testing/selftests/bpf/progs/test_oom.c:14: +struct mem_cgroup *bpf_get_root_mem_cgroup(void) __ksym; WARNING: line length of 88 exceeds 80 columns #323: FILE: tools/testing/selftests/bpf/progs/test_oom.c:56: + bpf_for_each(css, css_pos, &root_memcg->css, BPF_CGROUP_ITER_DESCENDANTS_POST) { WARNING: line length of 92 exceeds 80 columns #324: FILE: tools/testing/selftests/bpf/progs/test_oom.c:57: + if (css_pos->cgroup->nr_descendants + css_pos->cgroup->nr_dying_descendants) CHECK: Unnecessary parentheses around 'usage > max_usage' #339: FILE: tools/testing/selftests/bpf/progs/test_oom.c:72: + if ((usage > max_usage) && mem_cgroup_killable(memcg)) { total: 0 errors, 8 warnings, 8 checks, 337 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 914e33eca01d ("bpf: selftests: bpf OOM handler test") 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, 8 warnings, 8 checks, 337 lines checked