Commit afbe3c27 authored by Daniel T. Lee's avatar Daniel T. Lee Committed by Alexei Starovoitov

samples: bpf: Update outdated error message

Currently, under samples, several methods are being used to load bpf
program.

Since using libbpf is preferred solution, lots of previously used
'load_bpf_file' from bpf_load are replaced with 'bpf_prog_load_xattr'
from libbpf.

But some of the error messages still show up as 'load_bpf_file' instead
of 'bpf_prog_load_xattr'.

This commit fixes outdated errror messages under samples and fixes some
code style issues.
Signed-off-by: default avatarDaniel T. Lee <danieltimlee@gmail.com>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
Acked-by: default avatarAndrii Nakryiko <andriin@fb.com>
Link: https://lore.kernel.org/bpf/20191107005153.31541-2-danieltimlee@gmail.com
parent ed5941af
...@@ -147,7 +147,7 @@ static int prog_load(char *prog) ...@@ -147,7 +147,7 @@ static int prog_load(char *prog)
} }
if (ret) { if (ret) {
printf("ERROR: load_bpf_file failed for: %s\n", prog); printf("ERROR: bpf_prog_load_xattr failed for: %s\n", prog);
printf(" Output from verifier:\n%s\n------\n", bpf_log_buf); printf(" Output from verifier:\n%s\n------\n", bpf_log_buf);
ret = -1; ret = -1;
} else { } else {
......
...@@ -139,7 +139,7 @@ int main(int argc, char **argv) ...@@ -139,7 +139,7 @@ int main(int argc, char **argv)
map_fd = bpf_map__fd(map); map_fd = bpf_map__fd(map);
if (!prog_fd) { if (!prog_fd) {
printf("load_bpf_file: %s\n", strerror(errno)); printf("bpf_prog_load_xattr: %s\n", strerror(errno));
return 1; return 1;
} }
......
...@@ -51,8 +51,8 @@ static const struct option long_options[] = { ...@@ -51,8 +51,8 @@ static const struct option long_options[] = {
{"sec", required_argument, NULL, 's' }, {"sec", required_argument, NULL, 's' },
{"no-separators", no_argument, NULL, 'z' }, {"no-separators", no_argument, NULL, 'z' },
{"action", required_argument, NULL, 'a' }, {"action", required_argument, NULL, 'a' },
{"readmem", no_argument, NULL, 'r' }, {"readmem", no_argument, NULL, 'r' },
{"swapmac", no_argument, NULL, 'm' }, {"swapmac", no_argument, NULL, 'm' },
{"force", no_argument, NULL, 'F' }, {"force", no_argument, NULL, 'F' },
{0, 0, NULL, 0 } {0, 0, NULL, 0 }
}; };
...@@ -499,7 +499,7 @@ int main(int argc, char **argv) ...@@ -499,7 +499,7 @@ int main(int argc, char **argv)
map_fd = bpf_map__fd(map); map_fd = bpf_map__fd(map);
if (!prog_fd) { if (!prog_fd) {
fprintf(stderr, "ERR: load_bpf_file: %s\n", strerror(errno)); fprintf(stderr, "ERR: bpf_prog_load_xattr: %s\n", strerror(errno));
return EXIT_FAIL; return EXIT_FAIL;
} }
......
...@@ -150,7 +150,7 @@ int main(int argc, char **argv) ...@@ -150,7 +150,7 @@ int main(int argc, char **argv)
return 1; return 1;
if (!prog_fd) { if (!prog_fd) {
printf("load_bpf_file: %s\n", strerror(errno)); printf("bpf_prog_load_xattr: %s\n", strerror(errno));
return 1; return 1;
} }
......
...@@ -268,7 +268,7 @@ int main(int argc, char **argv) ...@@ -268,7 +268,7 @@ int main(int argc, char **argv)
return 1; return 1;
if (!prog_fd) { if (!prog_fd) {
printf("load_bpf_file: %s\n", strerror(errno)); printf("bpf_prog_load_xattr: %s\n", strerror(errno));
return 1; return 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment