Re: [PATCH v5 0/8] Generic IRQ entry/exit support for powerpc
From: John Ogness <john.ogness@linutronix.de>
Date: 2026-07-15 11:28:32
Also in:
lkml
Hi Mukesh, On 2026-07-07, Mukesh Kumar Chaurasiya [off-list ref] wrote:
On Tue, Jul 07, 2026 at 06:34:45PM +0200, Andreas Schwab wrote:quoted
On Jul 07 2026, Mukesh Kumar Chaurasiya wrote:quoted
diff --git a/arch/powerpc/include/asm/entry-common.h b/arch/powerpc/include/asm/entry-common.h index fc636c42e89a..b6373bae4524 100644 --- a/arch/powerpc/include/asm/entry-common.h +++ b/arch/powerpc/include/asm/entry-common.h@@ -286,14 +286,6 @@ static inline void arch_interrupt_async_enter_prepare(struct pt_regs *regs) static inline void arch_interrupt_async_exit_prepare(struct pt_regs *regs) { - /* - * Adjust at exit so the main handler sees the true NIA. This must - * come before irq_exit() because irq_exit can enable interrupts, and - * if another interrupt is taken before nap_adjust_return has run - * here, then that interrupt would return directly to idle nap return. - */ - nap_adjust_return(regs); - arch_interrupt_exit_prepare(regs); }diff --git a/arch/powerpc/include/asm/interrupt.h b/arch/powerpc/include/asm/interrupt.h index fb42a664ae54..1b45a49e9bed 100644 --- a/arch/powerpc/include/asm/interrupt.h +++ b/arch/powerpc/include/asm/interrupt.h@@ -246,6 +246,7 @@ interrupt_handler void func(struct pt_regs *regs) \ instrumentation_begin(); \ irq_enter_rcu(); \ ____##func (regs); \ + nap_adjust_return(regs); \ irq_exit_rcu(); \ instrumentation_end(); \ arch_interrupt_async_exit_prepare(regs); \This allows the system to boot normally.Thanks for testing this. I am assuming you tried this without "powersave=off" command line option.
I am here after bisecting the same problem. (I also use a G5.) I can also confirm that the above hack on top of v7.2-rc3 will boot (without specifying powersave=off). John Ogness