Re: [PATCH v4 1/4] kgdb: Remove irq flags from roundup

From: Will Deacon
Date: Wed Nov 14 2018 - 16:51:28 EST


On Mon, Nov 12, 2018 at 10:26:55AM -0800, Douglas Anderson wrote:
> The function kgdb_roundup_cpus() was passed a parameter that was
> documented as:
>
> > the flags that will be used when restoring the interrupts. There is
> > local_irq_save() call before kgdb_roundup_cpus().
>
> Nobody used those flags. Anyone who wanted to temporarily turn on
> interrupts just did local_irq_enable() and local_irq_disable() without
> looking at them. So we can definitely remove the flags.
>
> Signed-off-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
> ---

Acked-by: Will Deacon <will.deacon@xxxxxxx>

I'm hopeful that you'll keep hacking on kgdb, because it definitely needs
some love in its current state.

Will