Re: [PATCH] trace: atomic64_read(&ioc->vtime_rate) is assigned an extra semicolon

From: Steven Rostedt
Date: Mon Jul 11 2022 - 16:55:44 EST


Jens,

Want to take this one? Or I can if you don't.

-- Steve


On Wed, 29 Jun 2022 11:00:13 +0800
Li kunyu <kunyu@xxxxxxxxxxxx> wrote:

> I think this semicolon could be deleted.
>
> Signed-off-by: Li kunyu <kunyu@xxxxxxxxxxxx>
> ---
> include/trace/events/iocost.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/trace/events/iocost.h b/include/trace/events/iocost.h
> index e282ce02fa2d..6d1626e7a4ce 100644
> --- a/include/trace/events/iocost.h
> +++ b/include/trace/events/iocost.h
> @@ -160,7 +160,7 @@ TRACE_EVENT(iocost_ioc_vrate_adj,
>
> TP_fast_assign(
> __assign_str(devname, ioc_name(ioc));
> - __entry->old_vrate = atomic64_read(&ioc->vtime_rate);;
> + __entry->old_vrate = atomic64_read(&ioc->vtime_rate);
> __entry->new_vrate = new_vrate;
> __entry->busy_level = ioc->busy_level;
> __entry->read_missed_ppm = missed_ppm[READ];