[PATCH 0/4] genirq/irq_sim: misc updates

Bartosz Golaszewski posted 4 patches 2 years, 3 months ago
There is a newer version of this series
include/linux/bitmap.h |  3 +++
kernel/irq/irq_sim.c   | 27 +++++++++++----------------
2 files changed, 14 insertions(+), 16 deletions(-)
[PATCH 0/4] genirq/irq_sim: misc updates
Posted by Bartosz Golaszewski 2 years, 3 months ago
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

Here are a couple of updates to the interrupt simulator. Two are minor:
remove an unused field and reorder includes for readability. The third
one simplifies the error paths by using new cleanup macros. To that end
we also add a cleanup definition for dynamic bitmaps.

Bartosz Golaszewski (4):
  bitmap: define a cleanup function for bitmaps
  genirq/irq_sim: remove unused field from struct irq_sim_irq_ctx
  genirq/irq_sim: order headers alphabetically
  genirq/irq_sim: shrink code by using cleanup helpers

 include/linux/bitmap.h |  3 +++
 kernel/irq/irq_sim.c   | 27 +++++++++++----------------
 2 files changed, 14 insertions(+), 16 deletions(-)

-- 
2.39.2
Re: [PATCH 0/4] genirq/irq_sim: misc updates
Posted by Yury Norov 2 years, 3 months ago
On Tue, Sep 12, 2023 at 10:55:37AM +0200, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> 
> Here are a couple of updates to the interrupt simulator. Two are minor:
> remove an unused field and reorder includes for readability. The third
> one simplifies the error paths by using new cleanup macros. To that end
> we also add a cleanup definition for dynamic bitmaps.

This looks like a v2 for https://lore.kernel.org/lkml/20230812194457.6432-1-brgl@bgdev.pl/
Is it right? If so, can you keep history of submissions in the cover
email please?
Re: [PATCH 0/4] genirq/irq_sim: misc updates
Posted by Bartosz Golaszewski 2 years, 3 months ago
On Tue, Sep 12, 2023 at 3:36 PM Yury Norov <yury.norov@gmail.com> wrote:
>
> On Tue, Sep 12, 2023 at 10:55:37AM +0200, Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >
> > Here are a couple of updates to the interrupt simulator. Two are minor:
> > remove an unused field and reorder includes for readability. The third
> > one simplifies the error paths by using new cleanup macros. To that end
> > we also add a cleanup definition for dynamic bitmaps.
>
> This looks like a v2 for https://lore.kernel.org/lkml/20230812194457.6432-1-brgl@bgdev.pl/
> Is it right? If so, can you keep history of submissions in the cover
> email please?

That series was mainly about fixing a resource leak. While it
contained a part of the changes that ended up here, I consider this
one a stand-alone enough to not make it a v2. Probably should have
mentioned that one though.

Bart