bpfilter: Specify the log level for the kmsg message
authorGary Lin <glin@suse.com>
Wed, 23 Jun 2021 04:09:18 +0000 (12:09 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Fri, 25 Jun 2021 11:13:50 +0000 (13:13 +0200)
Per the kmsg document [0], if we don't specify the log level with a
prefix "<N>" in the message string, the default log level will be
applied to the message. Since the default level could be warning(4),
this would make the log utility such as journalctl treat the message,
"Started bpfilter", as a warning. To avoid confusion, this commit
adds the prefix "<5>" to make the message always a notice.

  [0] https://www.kernel.org/doc/Documentation/ABI/testing/dev-kmsg

Fixes: 36c4357c63f3 ("net: bpfilter: print umh messages to /dev/kmsg")
Reported-by: Martin Loviska <mloviska@suse.com>
Signed-off-by: Gary Lin <glin@suse.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Dmitrii Banshchikov <me@ubique.spb.ru>
Link: https://lore.kernel.org/bpf/20210623040918.8683-1-glin@suse.com
net/bpfilter/main.c

index 05e1cfc..291a925 100644 (file)
@@ -57,7 +57,7 @@ int main(void)
 {
        debug_f = fopen("/dev/kmsg", "w");
        setvbuf(debug_f, 0, _IOLBF, 0);
-       fprintf(debug_f, "Started bpfilter\n");
+       fprintf(debug_f, "<5>Started bpfilter\n");
        loop();
        fclose(debug_f);
        return 0;