diff options
author | Daniel T. Lee <danieltimlee@gmail.com> | 2019-05-23 16:24:48 +0900 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2019-05-28 11:10:37 +0200 |
commit | 37b54aed123faa19eb21d7ef2534756c5a152a7c (patch) | |
tree | 8a98500856f244697175426754936ed5231154b8 /samples/bpf | |
parent | 486d3f22c0cab818ba6b4324fd8eda643d5f16b2 (diff) | |
download | talos-op-linux-37b54aed123faa19eb21d7ef2534756c5a152a7c.tar.gz talos-op-linux-37b54aed123faa19eb21d7ef2534756c5a152a7c.zip |
samples/bpf: fix a couple of style issues in bpf_load
This commit fixes a few style problems in samples/bpf/bpf_load.c:
- Magic string use of 'DEBUGFS'
- Useless zero initialization of a global variable
- Minor style fix with whitespace
Signed-off-by: Daniel T. Lee <danieltimlee@gmail.com>
Acked-by: Yonghong Song <yhs@fb.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Diffstat (limited to 'samples/bpf')
-rw-r--r-- | samples/bpf/bpf_load.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/samples/bpf/bpf_load.c b/samples/bpf/bpf_load.c index eae7b635343d..1734ade04f7f 100644 --- a/samples/bpf/bpf_load.c +++ b/samples/bpf/bpf_load.c @@ -40,7 +40,7 @@ int prog_cnt; int prog_array_fd = -1; struct bpf_map_data map_data[MAX_MAPS]; -int map_data_count = 0; +int map_data_count; static int populate_prog_array(const char *event, int prog_fd) { @@ -65,7 +65,7 @@ static int write_kprobe_events(const char *val) else flags = O_WRONLY | O_APPEND; - fd = open("/sys/kernel/debug/tracing/kprobe_events", flags); + fd = open(DEBUGFS "kprobe_events", flags); ret = write(fd, val, strlen(val)); close(fd); @@ -490,8 +490,8 @@ static int load_elf_maps_section(struct bpf_map_data *maps, int maps_shndx, /* Verify no newer features were requested */ if (validate_zero) { - addr = (unsigned char*) def + map_sz_copy; - end = (unsigned char*) def + map_sz_elf; + addr = (unsigned char *) def + map_sz_copy; + end = (unsigned char *) def + map_sz_elf; for (; addr < end; addr++) { if (*addr != 0) { free(sym); |