[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAP4=nvQ-J9uEm-mAnBhjLvy_LOzB4KvMoCBYeeo4w5_fMwbiow@mail.gmail.com>
Date: Wed, 20 Nov 2024 06:04:05 +0100
From: Tomas Glozar <tglozar@...hat.com>
To: Steven Rostedt <rostedt@...dmis.org>
Cc: linux-trace-kernel@...r.kernel.org, linux-kernel@...r.kernel.org,
jkacur@...hat.com
Subject: Re: [PATCH] rtla/timerlat: Fix histogram ALL for zero samples
Ășt 19. 11. 2024 v 16:22 odesĂlatel Steven Rostedt <rostedt@...dmis.org> napsal:
>
> There's a lot of duplicate code here. Can you please make a helper
> function, that at least has:
Good point, I thought it's not that bad when I was writing it, but
together with the already implemented ifs, this pattern is now there
19 times. I will send a v2 once I'm back from vacation.
Tomas
Powered by blists - more mailing lists