diff options
| author | Daniel Borkmann <daniel@iogearbox.net> | 2016-04-13 00:10:53 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-04-15 03:40:42 +0200 |
| commit | 02413cabd6b67f1444f153ea85d44076deae2056 (patch) | |
| tree | 8d656100a8e523f2f9a5256884f6c868c0ae94df /samples/bpf/tracex2_kern.c | |
| parent | bpf: convert relevant helper args to ARG_PTR_TO_RAW_STACK (diff) | |
| download | linux-02413cabd6b67f1444f153ea85d44076deae2056.tar.xz linux-02413cabd6b67f1444f153ea85d44076deae2056.zip | |
bpf, samples: don't zero data when not needed
Remove the zero initialization in the sample programs where appropriate.
Note that this is an optimization which is now possible, old programs
still doing the zero initialization are just fine as well. Also, make
sure we don't have padding issues when we don't memset() the entire
struct anymore.
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'samples/bpf/tracex2_kern.c')
| -rw-r--r-- | samples/bpf/tracex2_kern.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/samples/bpf/tracex2_kern.c b/samples/bpf/tracex2_kern.c index 6d6eefd0d465..5e11c20ce5ec 100644 --- a/samples/bpf/tracex2_kern.c +++ b/samples/bpf/tracex2_kern.c @@ -66,7 +66,7 @@ struct hist_key { char comm[16]; u64 pid_tgid; u64 uid_gid; - u32 index; + u64 index; }; struct bpf_map_def SEC("maps") my_hist_map = { @@ -82,7 +82,7 @@ int bpf_prog3(struct pt_regs *ctx) long write_size = PT_REGS_PARM3(ctx); long init_val = 1; long *value; - struct hist_key key = {}; + struct hist_key key; key.index = log2l(write_size); key.pid_tgid = bpf_get_current_pid_tgid(); |