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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
From: Steven Rostedt <rostedt@xxxxxxxxxxx>
Date: Sat, 11 Feb 2023 22:44:55 -0500
Cc: John Stultz <jstultz@xxxxxxxxxx>, Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Network Development <netdev@xxxxxxxxxxxxxxx>, bpf <bpf@xxxxxxxxxxxxxxx>, "linux-perf-use." <linux-perf-users@xxxxxxxxxxxxxxx>, Linux-Fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, linux-mm <linux-mm@xxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx>, kernel test robot <oliver.sang@xxxxxxxxx>, kbuild test robot <lkp@xxxxxxxxx>, Andrii Nakryiko <andrii@xxxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx>, Arnaldo Carvalho de Melo <arnaldo.melo@xxxxxxxxx>, Andrii Nakryiko <andrii.nakryiko@xxxxxxxxx>, Michal Miroslaw <mirq-linux@xxxxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Petr Mladek <pmladek@xxxxxxxx>, Kajetan Puchalski <kajetan.puchalski@xxxxxxx>, Lukasz Luba <lukasz.luba@xxxxxxx>, Qais Yousef <qyousef@xxxxxxxxxx>, Daniele Di Proietto <ddiproietto@xxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
In-reply-to: <CALOAHbAnFHAiMH4QDgS6xN16B31qfhG8tfQ+iioCr9pw3sP=bw@mail.gmail.com>

On Sun, 12 Feb 2023 11:38:52 +0800
Yafang Shao <laoar.shao@xxxxxxxxx> wrote:

> > 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

Follow-Ups:
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Namhyung Kim

References:
- [PATCH v2 0/7] task comm cleanups
  - From: Yafang Shao
- [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Yafang Shao
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: John Stultz
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Alexei Starovoitov
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: John Stultz
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Steven Rostedt
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Steven Rostedt
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Steven Rostedt
- Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
  - From: Yafang Shao

Prev by Date: Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
Next by Date: [PATCH 0/2] Allow mixing bpf2bpf calls with tailcalls on LoongArch
Previous by thread: Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
Next by thread: Re: [PATCH v2 7/7] tools/testing/selftests/bpf: replace open-coded 16 with TASK_COMM_LEN
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SoC] [Linux Rockchip SoC] [Linux Actions SoC] [Linux for Synopsys ARC Processors] [Linux NFS] [Linux NILFS] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]