Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with... | linux-perf-users

[PATCH v2 0/7] task comm cleanups · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 1/7] fs/exec: replace strlcpy with strscpy_pad in __set_task_comm · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 2/7] fs/exec: replace strncpy with strscpy_pad in __get_task_comm · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 5/7] samples/bpf/test_overhead_kprobe_kern: replace bpf_probe_read_kernel with bpf_probe_read_kernel_str to get task comm · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 6/7] tools/bpf/bpftool/skeleton: replace bpf_probe_read_kernel with bpf_probe_read_kernel_str to get task comm · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 4/7] fs/binfmt_elf: replace open-coded string copy with get_task_comm · Yafang Shao <hidden> · 2021-11-20
Re: [PATCH v2 4/7] fs/binfmt_elf: replace open-coded string copy with get_task_comm · Steven Rostedt <rostedt@goodmis.org> · 2021-11-29
Re: [PATCH v2 4/7] fs/binfmt_elf: replace open-coded string copy with get_task_comm · Yafang Shao <hidden> · 2021-11-30
Re: [PATCH v2 4/7] fs/binfmt_elf: replace open-coded string copy with get_task_comm · Steven Rostedt <rostedt@goodmis.org> · 2021-11-30
Re: [PATCH v2 4/7] fs/binfmt_elf: replace open-coded string copy with get_task_comm · Yafang Shao <hidden> · 2021-11-30
[PATCH v2 3/7] drivers/infiniband: replace open-coded string copy with get_task_comm · Yafang Shao <hidden> · 2021-11-20
[PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Yafang Shao <hidden> · 2021-11-20
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · John Stultz <jstultz@google.com> · 2023-02-08
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Alexei Starovoitov <hidden> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · John Stultz <jstultz@google.com> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Yafang Shao <hidden> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Kajetan Puchalski <hidden> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Yafang Shao <hidden> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Kajetan Puchalski <hidden> · 2023-02-10
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Qais Yousef <hidden> · 2023-02-11
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Yafang Shao <hidden> · 2023-02-12
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Steven Rostedt <rostedt@goodmis.org> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Steven Rostedt <rostedt@goodmis.org> · 2023-02-09
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Steven Rostedt <rostedt@goodmis.org> · 2023-02-11
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Yafang Shao <hidden> · 2023-02-12
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Steven Rostedt <rostedt@goodmis.org> · 2023-02-12
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Namhyung Kim <hidden> · 2023-02-13
Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2023-02-13

Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2023-02-12 03:45:18
Also in: bpf, linux-fsdevel, linux-mm, lkml, netdev

On Sun, 12 Feb 2023 11:38:52 +0800
Yafang Shao [off-list ref] wrote:

quoted

Actually, there are cases that this needs to be a number, as b3bc8547d3be6
("tracing: Have TRACE_DEFINE_ENUM affect trace event types as well") made
it update fields as well as the printk fmt.

It seems that TRACE_DEFINE_ENUM(TASK_COMM_LEN) in the trace events
header files would be a better fix.

NACK! I much prefer the proper fix that adds the length.

-- Steve

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help