[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240617130754.3b4dbd62@rorschach.local.home>
Date: Mon, 17 Jun 2024 13:07:54 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Bart Van Assche <bvanassche@....org>
Cc: dongliang cui <cuidongliang390@...il.com>, Dongliang Cui
<dongliang.cui@...soc.com>, axboe@...nel.dk, mhiramat@...nel.org,
mathieu.desnoyers@...icios.com, ebiggers@...nel.org, ke.wang@...soc.com,
hongyu.jin.cn@...il.com, niuzhiguo84@...il.com, hao_hao.wang@...soc.com,
linux-block@...r.kernel.org, linux-kernel@...r.kernel.org,
akailash@...gle.com
Subject: Re: [PATCH v5] block: Add ioprio to block_rq tracepoint
On Mon, 17 Jun 2024 10:02:48 -0700
Bart Van Assche <bvanassche@....org> wrote:
> >> Do we really want to include the constant "[0]" in the tracing output?
> > This is how it is printed in the source code.
> > From the code flow point of view, there is no need to print this value
> > in trace_block_rq_requeue.
> > Do we need to consider the issue of uniform printing format? If not, I
> > think we can delete it.
>
> I'm not aware of any other tracing statement that prints out a constant.
> Is there perhaps something that I'm missing or overlooking?
The only time that is done, is if the trace event is used in multiple
places and there's one place that the value will always be the same.
-- Steve
Powered by blists - more mailing lists