Re: [PATCH v4] m68k: amiga: turn off all incoming interrupts from the Warp1260 during boot

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Paolo,

On Sat, Jun 1, 2024 at 5:33 PM Paolo Pisati <p.pisati@xxxxxxxxx> wrote:
On an Amiga 1200 equipped with a Warp1260 accelerator, an interrupt storm coming
from the accelerator board causes the machine to crash in local_irq_enable() or
auto_irq_enable(). Disabling interrupts for the Warp1260 in
amiga_parse_bootinfo() fixes the problem.

Link: https://lore.kernel.org/r/ZkjwzVwYeQtyAPrL@amaterasu.local
Cc: stable <stable@xxxxxxxxxx>
Signed-off-by: Paolo Pisati <p.pisati@xxxxxxxxx>
---
Changes since v3:
-simplify commit body and subject

Thanks for the update!

--- a/arch/m68k/amiga/config.c
+++ b/arch/m68k/amiga/config.c
@@ -180,6 +180,15 @@ int __init amiga_parse_bootinfo(const struct bi_record *record)
                        dev->slotsize = be16_to_cpu(cd->cd_SlotSize);
                        dev->boardaddr = be32_to_cpu(cd->cd_BoardAddr);
                        dev->boardsize = be32_to_cpu(cd->cd_BoardSize);
+
+                       /* CS-LAB Warp 1260 workaround */
+                       if (be16_to_cpu(dev->rom.er_Manufacturer) == ZORRO_MANUF(ZORRO_PROD_CSLAB_WARP_1260) &&
+                           dev->rom.er_Product == ZORRO_PROD(ZORRO_PROD_CSLAB_WARP_1260)) {
+
+                               /* turn off all interrupts */
+                               pr_info("Warp 1260 card detected: applying interrupt storm workaround");

Missing "\n".

+                               *(uint32_t *)(dev->boardaddr + 0x1000) = 0x0FFF;
+                       }
                } else
                        pr_warn("amiga_parse_bootinfo: too many AutoConfig devices\n");
 #endif /* CONFIG_ZORRO */

Reviewed-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
i.e. will queue in the m68k tree for v6.11, with the above fixed.
No need to resend.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds





[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux