Re: [PATCH 3/5] irqchip/bcm2836: Make IPIs use handle_percpu_devid_irq()

[Marc Zyngier]

On 2020-12-15 15:21, Guenter Roeck wrote:
> Hi Marc,
>
> On 12/15/20 2:19 AM, Marc Zyngier wrote:
> > Hi Gunter,
> >
> > On 2020-12-15 00:21, Guenter Roeck wrote:
> > > On Mon, Nov 09, 2020 at 09:41:19AM +0000, Valentin Schneider wrote:
> > > > As done for the Arm GIC irqchips, move IPIs to 
> > > > handle_percpu_devid_irq() as
> > > > handle_percpu_devid_fasteoi_ipi() isn't actually required.
> > > >
> > > > Signed-off-by: Valentin Schneider <valentin.schneider@xxxxxxx>
> > >
> > > This patch results in boot failures (silent stall) for the qemu
> > > raspi2 emulation. Unfortunately it can not be reverted because
> > > handle_percpu_devid_fasteoi_ipi no longer exists in next-20201214,
> > > so I don't know if it is the only problem.
> >
> > This is odd. This works just fine for me on both the RPi2 and 3
> > emulation, running a full Debian userspace. Could this be caused
> > by the version of QEMU you are using? Here's what I have:
> >
> > $ qemu-system-arm --version
> > QEMU emulator version 5.1.0 (Debian 1:5.1+dfsg-4+b1)
>
> I used qemu 5.0 which - up to now - worked fine. Let me try with
> 5.1 and 5.2. Sorry, I should have tried first before bothering you.

No bother at all. If this works for you on more recent versions of
QEMU, that'd be an interesting data point. I'm also interested in
the result of the patch either way.

Thanks,

        M.
--
Jazz is not dead. It just smells funny...