WARNING: Avoid line continuations in quoted strings #196: FILE: tools/bpf/bpftool/gen.c:807: + codegen("\ WARNING: line length of 94 exceeds 80 columns #198: FILE: tools/bpf/bpftool/gen.c:809: + static const char opts_sig[] __attribute__((__aligned__(8))) = \"\\\n\ WARNING: Prefer __aligned(8) over __attribute__((__aligned__(8))) #198: FILE: tools/bpf/bpftool/gen.c:809: + static const char opts_sig[] __attribute__((__aligned__(8))) = \"\\\n\ WARNING: Avoid line continuations in quoted strings #201: FILE: tools/bpf/bpftool/gen.c:812: + codegen("\ WARNING: Avoid line continuations in quoted strings #205: FILE: tools/bpf/bpftool/gen.c:816: + codegen("\ WARNING: line length of 100 exceeds 80 columns #207: FILE: tools/bpf/bpftool/gen.c:818: + static const char opts_excl_hash[] __attribute__((__aligned__(8))) = \"\\\n\ WARNING: Prefer __aligned(8) over __attribute__((__aligned__(8))) #207: FILE: tools/bpf/bpftool/gen.c:818: + static const char opts_excl_hash[] __attribute__((__aligned__(8))) = \"\\\n\ WARNING: Avoid line continuations in quoted strings #210: FILE: tools/bpf/bpftool/gen.c:821: + codegen("\ WARNING: Avoid line continuations in quoted strings #214: FILE: tools/bpf/bpftool/gen.c:825: + codegen("\ WARNING: line length of 83 exceeds 80 columns #216: FILE: tools/bpf/bpftool/gen.c:827: + opts.signature = (void *)opts_sig; \n\ WARNING: line length of 83 exceeds 80 columns #217: FILE: tools/bpf/bpftool/gen.c:828: + opts.signature_sz = sizeof(opts_sig) - 1; \n\ WARNING: line length of 83 exceeds 80 columns #218: FILE: tools/bpf/bpftool/gen.c:829: + opts.excl_prog_hash = (void *)opts_excl_hash; \n\ WARNING: line length of 83 exceeds 80 columns #219: FILE: tools/bpf/bpftool/gen.c:830: + opts.excl_prog_hash_sz = sizeof(opts_excl_hash) - 1; \n\ WARNING: line length of 83 exceeds 80 columns #220: FILE: tools/bpf/bpftool/gen.c:831: + opts.keyring_id = skel->keyring_id; \n\ WARNING: Avoid line continuations in quoted strings #224: FILE: tools/bpf/bpftool/gen.c:835: + codegen("\ WARNING: Avoid line continuations in quoted strings #243: FILE: tools/bpf/bpftool/gen.c:1411: + codegen("\ CHECK: Comparison to NULL could be written "!private_key_path" #321: FILE: tools/bpf/bpftool/main.c:551: + if (sign_progs && (private_key_path == NULL || cert_path == NULL)) { CHECK: Comparison to NULL could be written "!cert_path" #321: FILE: tools/bpf/bpftool/main.c:551: + if (sign_progs && (private_key_path == NULL || cert_path == NULL)) { CHECK: Comparison to NULL could be written "private_key_path" #326: FILE: tools/bpf/bpftool/main.c:556: + if (!sign_progs && (private_key_path != NULL || cert_path != NULL)) { CHECK: Comparison to NULL could be written "cert_path" #326: FILE: tools/bpf/bpftool/main.c:556: + if (!sign_progs && (private_key_path != NULL || cert_path != NULL)) { WARNING: unnecessary whitespace before a quoted newline #449: FILE: tools/bpf/bpftool/prog.c:2592: + " {-L|--use-loader} | [ {-S|--sign } {-k} {-i} ] \n" WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #454: new file mode 100644 WARNING: braces {} are not necessary for any arm of this statement #501: FILE: tools/bpf/bpftool/sign.c:43: + if (data && (flags & ERR_TXT_STRING)) { [...] + } else { [...] CHECK: Alignment should match open parenthesis #576: FILE: tools/bpf/bpftool/sign.c:118: + key_id = syscall(__NR_add_key, "asymmetric", key_der_path, der_buf, + (size_t)der_len, KEY_SPEC_SESSION_KEYRING); WARNING: line length of 82 exceeds 80 columns #656: FILE: tools/bpf/bpftool/sign.c:198: + if (BIO_read(bd_out, opts->signature, actual_sig_len) != actual_sig_len) { total: 0 errors, 20 warnings, 5 checks, 562 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 0f749d75a3a3 ("bpftool: Add support for signing 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, 20 warnings, 5 checks, 562 lines checked