[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aTqH0See4rFDwMga@fedora>
Date: Thu, 11 Dec 2025 16:58:57 +0800
From: Ming Lei <ming.lei@...hat.com>
To: Caleb Sander Mateos <csander@...estorage.com>
Cc: Shuah Khan <shuah@...nel.org>, linux-block@...r.kernel.org,
linux-kselftest@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/8] selftests: ublk: remove unused ios map in seq_io.bt
On Wed, Dec 10, 2025 at 10:15:57PM -0700, Caleb Sander Mateos wrote:
> The ios map populated by seq_io.bt is never read, so remove it.
>
> Signed-off-by: Caleb Sander Mateos <csander@...estorage.com>
> ---
> tools/testing/selftests/ublk/trace/seq_io.bt | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ublk/trace/seq_io.bt b/tools/testing/selftests/ublk/trace/seq_io.bt
> index 507a3ca05abf..b2f60a92b118 100644
> --- a/tools/testing/selftests/ublk/trace/seq_io.bt
> +++ b/tools/testing/selftests/ublk/trace/seq_io.bt
> @@ -15,11 +15,10 @@ tracepoint:block:block_rq_complete
> printf("io_out_of_order: exp %llu actual %llu\n",
> args.sector, $last);
> }
> @last_rw[$dev, str($2)] = (args.sector + args.nr_sector);
> }
> - @ios = count();
BTW, it was for debug purpose, but now it isn't needed.
Reviewed-by: Ming Lei <ming.lei@...hat.com>
Thanks,
Ming
~
Powered by blists - more mailing lists