* [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output"
@ 2026-03-06 7:59 Xuewen Yan
2026-03-06 15:06 ` Steven Rostedt
0 siblings, 1 reply; 4+ messages in thread
From: Xuewen Yan @ 2026-03-06 7:59 UTC (permalink / raw)
To: rostedt, mhiramat, mathieu.desnoyers, elver, kees
Cc: lorenzo.stoakes, brauner, schuster.simon, david, linux-kernel,
linux-trace-kernel, guohua.yan, ke.wang, xuewen.yan94, jing.xia
This reverts commit e3f6a42272e028c46695acc83fc7d7c42f2750ad.
The commit says that the tracepoint only deals with the current task,
however the following case is not current task:
comm_write() {
p = get_proc_task(inode);
if (!p)
return -ESRCH;
if (same_thread_group(current, p))
set_task_comm(p, buffer);
}
where set_task_comm() calls __set_task_comm() which records
the update of p and not current.
So revert the patch to show pid.
Fixes: e3f6a42272e0 ("tracing: Remove pid in task_rename tracing output")
Reported-by: Guohua Yan <guohua.yan@unisoc.com>
Signed-off-by: Xuewen Yan <xuewen.yan@unisoc.com>
---
v2:
- update commit message (Steven)
---
include/trace/events/task.h | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/include/trace/events/task.h b/include/trace/events/task.h
index 4f0759634306..b9a129eb54d9 100644
--- a/include/trace/events/task.h
+++ b/include/trace/events/task.h
@@ -38,19 +38,22 @@ TRACE_EVENT(task_rename,
TP_ARGS(task, comm),
TP_STRUCT__entry(
+ __field( pid_t, pid)
__array( char, oldcomm, TASK_COMM_LEN)
__array( char, newcomm, TASK_COMM_LEN)
__field( short, oom_score_adj)
),
TP_fast_assign(
+ __entry->pid = task->pid;
memcpy(entry->oldcomm, task->comm, TASK_COMM_LEN);
strscpy(entry->newcomm, comm, TASK_COMM_LEN);
__entry->oom_score_adj = task->signal->oom_score_adj;
),
- TP_printk("oldcomm=%s newcomm=%s oom_score_adj=%hd",
- __entry->oldcomm, __entry->newcomm, __entry->oom_score_adj)
+ TP_printk("pid=%d oldcomm=%s newcomm=%s oom_score_adj=%hd",
+ __entry->pid, __entry->oldcomm,
+ __entry->newcomm, __entry->oom_score_adj)
);
/**
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output"
2026-03-06 7:59 [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output" Xuewen Yan
@ 2026-03-06 15:06 ` Steven Rostedt
2026-03-16 2:00 ` Xuewen Yan
0 siblings, 1 reply; 4+ messages in thread
From: Steven Rostedt @ 2026-03-06 15:06 UTC (permalink / raw)
To: Xuewen Yan
Cc: mhiramat, mathieu.desnoyers, elver, kees, lorenzo.stoakes,
brauner, schuster.simon, david, linux-kernel, linux-trace-kernel,
guohua.yan, ke.wang, xuewen.yan94, jing.xia
On Fri, 6 Mar 2026 15:59:54 +0800
Xuewen Yan <xuewen.yan@unisoc.com> wrote:
> This reverts commit e3f6a42272e028c46695acc83fc7d7c42f2750ad.
>
> The commit says that the tracepoint only deals with the current task,
> however the following case is not current task:
>
> comm_write() {
> p = get_proc_task(inode);
> if (!p)
> return -ESRCH;
>
> if (same_thread_group(current, p))
> set_task_comm(p, buffer);
> }
> where set_task_comm() calls __set_task_comm() which records
> the update of p and not current.
>
> So revert the patch to show pid.
>
> Fixes: e3f6a42272e0 ("tracing: Remove pid in task_rename tracing output")
> Reported-by: Guohua Yan <guohua.yan@unisoc.com>
> Signed-off-by: Xuewen Yan <xuewen.yan@unisoc.com>
Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>
-- Steve
> ---
> v2:
> - update commit message (Steven)
> ---
> include/trace/events/task.h | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/include/trace/events/task.h b/include/trace/events/task.h
> index 4f0759634306..b9a129eb54d9 100644
> --- a/include/trace/events/task.h
> +++ b/include/trace/events/task.h
> @@ -38,19 +38,22 @@ TRACE_EVENT(task_rename,
> TP_ARGS(task, comm),
>
> TP_STRUCT__entry(
> + __field( pid_t, pid)
> __array( char, oldcomm, TASK_COMM_LEN)
> __array( char, newcomm, TASK_COMM_LEN)
> __field( short, oom_score_adj)
> ),
>
> TP_fast_assign(
> + __entry->pid = task->pid;
> memcpy(entry->oldcomm, task->comm, TASK_COMM_LEN);
> strscpy(entry->newcomm, comm, TASK_COMM_LEN);
> __entry->oom_score_adj = task->signal->oom_score_adj;
> ),
>
> - TP_printk("oldcomm=%s newcomm=%s oom_score_adj=%hd",
> - __entry->oldcomm, __entry->newcomm, __entry->oom_score_adj)
> + TP_printk("pid=%d oldcomm=%s newcomm=%s oom_score_adj=%hd",
> + __entry->pid, __entry->oldcomm,
> + __entry->newcomm, __entry->oom_score_adj)
> );
>
> /**
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output"
2026-03-06 15:06 ` Steven Rostedt
@ 2026-03-16 2:00 ` Xuewen Yan
2026-03-16 13:54 ` Steven Rostedt
0 siblings, 1 reply; 4+ messages in thread
From: Xuewen Yan @ 2026-03-16 2:00 UTC (permalink / raw)
To: Steven Rostedt
Cc: Xuewen Yan, mhiramat, mathieu.desnoyers, elver, kees,
lorenzo.stoakes, brauner, schuster.simon, david, linux-kernel,
linux-trace-kernel, guohua.yan, ke.wang, jing.xia
Hi Steven,
Unless there are any further comments, could you please help to take
this through the tracing tree?
Thanks!
On Fri, Mar 6, 2026 at 11:06 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> On Fri, 6 Mar 2026 15:59:54 +0800
> Xuewen Yan <xuewen.yan@unisoc.com> wrote:
>
> > This reverts commit e3f6a42272e028c46695acc83fc7d7c42f2750ad.
> >
> > The commit says that the tracepoint only deals with the current task,
> > however the following case is not current task:
> >
> > comm_write() {
> > p = get_proc_task(inode);
> > if (!p)
> > return -ESRCH;
> >
> > if (same_thread_group(current, p))
> > set_task_comm(p, buffer);
> > }
> > where set_task_comm() calls __set_task_comm() which records
> > the update of p and not current.
> >
> > So revert the patch to show pid.
> >
> > Fixes: e3f6a42272e0 ("tracing: Remove pid in task_rename tracing output")
> > Reported-by: Guohua Yan <guohua.yan@unisoc.com>
> > Signed-off-by: Xuewen Yan <xuewen.yan@unisoc.com>
>
> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>
>
> -- Steve
>
> > ---
> > v2:
> > - update commit message (Steven)
> > ---
> > include/trace/events/task.h | 7 +++++--
> > 1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/include/trace/events/task.h b/include/trace/events/task.h
> > index 4f0759634306..b9a129eb54d9 100644
> > --- a/include/trace/events/task.h
> > +++ b/include/trace/events/task.h
> > @@ -38,19 +38,22 @@ TRACE_EVENT(task_rename,
> > TP_ARGS(task, comm),
> >
> > TP_STRUCT__entry(
> > + __field( pid_t, pid)
> > __array( char, oldcomm, TASK_COMM_LEN)
> > __array( char, newcomm, TASK_COMM_LEN)
> > __field( short, oom_score_adj)
> > ),
> >
> > TP_fast_assign(
> > + __entry->pid = task->pid;
> > memcpy(entry->oldcomm, task->comm, TASK_COMM_LEN);
> > strscpy(entry->newcomm, comm, TASK_COMM_LEN);
> > __entry->oom_score_adj = task->signal->oom_score_adj;
> > ),
> >
> > - TP_printk("oldcomm=%s newcomm=%s oom_score_adj=%hd",
> > - __entry->oldcomm, __entry->newcomm, __entry->oom_score_adj)
> > + TP_printk("pid=%d oldcomm=%s newcomm=%s oom_score_adj=%hd",
> > + __entry->pid, __entry->oldcomm,
> > + __entry->newcomm, __entry->oom_score_adj)
> > );
> >
> > /**
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output"
2026-03-16 2:00 ` Xuewen Yan
@ 2026-03-16 13:54 ` Steven Rostedt
0 siblings, 0 replies; 4+ messages in thread
From: Steven Rostedt @ 2026-03-16 13:54 UTC (permalink / raw)
To: Xuewen Yan
Cc: Xuewen Yan, mhiramat, mathieu.desnoyers, elver, kees,
lorenzo.stoakes, brauner, schuster.simon, david, linux-kernel,
linux-trace-kernel, guohua.yan, ke.wang, jing.xia
On Mon, 16 Mar 2026 10:00:13 +0800
Xuewen Yan <xuewen.yan94@gmail.com> wrote:
> Unless there are any further comments, could you please help to take
> this through the tracing tree?
I guess this file doesn't really have an owner, so yeah, I'll take it.
-- Steve
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-03-16 13:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-06 7:59 [PATCH V2] tracing: Revert "tracing: Remove pid in task_rename tracing output" Xuewen Yan
2026-03-06 15:06 ` Steven Rostedt
2026-03-16 2:00 ` Xuewen Yan
2026-03-16 13:54 ` Steven Rostedt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox