On Fri, 6 Mar 2020 16:01:41 +0100
Cédric Le Goater <clg@xxxxxxxx> wrote:
> The PowerNV platform has multiple IRQ chips and the xmon command
> dumping the state of the XIVE interrupt should only operate on the
> XIVE IRQ chip.
>
> Fixes: 5896163f7f91 ("powerpc/xmon: Improve output of XIVE interrupts")
> Cc: stable@xxxxxxxxxxxxxxx # v5.4+
> Signed-off-by: Cédric Le Goater <clg@xxxxxxxx>
> ---
Reviewed-by: Greg Kurz <groug@xxxxxxxx>
> arch/powerpc/sysdev/xive/common.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/powerpc/sysdev/xive/common.c b/arch/powerpc/sysdev/xive/common.c
> index 550baba98ec9..8155adc2225a 100644
> --- a/arch/powerpc/sysdev/xive/common.c
> +++ b/arch/powerpc/sysdev/xive/common.c
> @@ -261,11 +261,15 @@ notrace void xmon_xive_do_dump(int cpu)
>
> int xmon_xive_get_irq_config(u32 hw_irq, struct irq_data *d)
> {
> + struct irq_chip *chip = irq_data_get_irq_chip(d);
> int rc;
> u32 target;
> u8 prio;
> u32 lirq;
>
> + if (!is_xive_irq(chip))
> + return -EINVAL;
> +
> rc = xive_ops->get_irq_config(hw_irq, &target, &prio, &lirq);
> if (rc) {
> xmon_printf("IRQ 0x%08x : no config rc=%d\n", hw_irq, rc);
