example/ringbuffer: fix comm len

The Linux kernel uses TASK_COMM_LEN to limit the length of the comm string. By default TASK_COMM_LEN is set to 16. For simplicity set TASK_COMM_LEN to 16 if it is not defined. As an alternative one could also include linux/sched.h. Signed-off-by: Florian Lehner <[email protected]>
cilium · Aug 5, 2024 · ccdd12c · ccdd12c
1 parent 0cb38bf
commit ccdd12c
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 4 deletions.
diff --git a/examples/ringbuffer/bpf_bpfeb.go b/examples/ringbuffer/bpf_bpfeb.go
diff --git a/examples/ringbuffer/bpf_bpfeb.o b/examples/ringbuffer/bpf_bpfeb.o
diff --git a/examples/ringbuffer/bpf_bpfel.go b/examples/ringbuffer/bpf_bpfel.go
diff --git a/examples/ringbuffer/bpf_bpfel.o b/examples/ringbuffer/bpf_bpfel.o
diff --git a/examples/ringbuffer/ringbuffer.c b/examples/ringbuffer/ringbuffer.c
@@ -2,11 +2,15 @@
 
 #include "common.h"
 
+#ifndef TASK_COMM_LEN
+#define TASK_COMM_LEN 16
+#endif
+
 char __license[] SEC("license") = "Dual MIT/GPL";
 
 struct event {
 	u32 pid;
-	u8 comm[80];
+	u8 comm[TASK_COMM_LEN];
 };
 
 struct {
@@ -29,7 +33,7 @@ int kprobe_execve(struct pt_regs *ctx) {
 	}
 
 	task_info->pid = tgid;
-	bpf_get_current_comm(&task_info->comm, 80);
+	bpf_get_current_comm(&task_info->comm, TASK_COMM_LEN);
 
 	bpf_ringbuf_submit(task_info, 0);