The branch main has been updated by markj:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=015daf5221f7588b9258fe0242cee09bde39fe21

commit 015daf5221f7588b9258fe0242cee09bde39fe21
Author:     Mark Johnston <ma...@freebsd.org>
AuthorDate: 2023-09-27 12:23:58 +0000
Commit:     Mark Johnston <ma...@freebsd.org>
CommitDate: 2023-09-27 12:28:27 +0000

    hdac: Defer interrupt allocation in hdac_attach()
    
    hdac_attach() registers an interrupt handler before allocating various
    driver resources which are accessed by the interrupt handler.  On some
    platforms we observe what appear to be spurious interrupts upon a cold
    boot, resulting in panics.
    
    Partially work around the problem by deferring irq allocation until
    after other resources are allocated.  I think this is not a complete
    solution, but is correct and sufficient to work around the problems
    reported in the PR.
    
    PR:             268393
    Tested by:      Alexander Sherikov <asheri...@yandex.com>
    Tested by:      Oleh Hushchenkov <o.hushchen...@gmail.com>
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D41883
---
 sys/dev/sound/pci/hda/hdac.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/sys/dev/sound/pci/hda/hdac.c b/sys/dev/sound/pci/hda/hdac.c
index 422f7f930ea5..2cf9239499af 100644
--- a/sys/dev/sound/pci/hda/hdac.c
+++ b/sys/dev/sound/pci/hda/hdac.c
@@ -1270,9 +1270,6 @@ hdac_attach(device_t dev)
 
        /* Allocate resources */
        result = hdac_mem_alloc(sc);
-       if (result != 0)
-               goto hdac_attach_fail;
-       result = hdac_irq_alloc(sc);
        if (result != 0)
                goto hdac_attach_fail;
 
@@ -1347,6 +1344,10 @@ hdac_attach(device_t dev)
        hdac_corb_init(sc);
        hdac_rirb_init(sc);
 
+       result = hdac_irq_alloc(sc);
+       if (result != 0)
+               goto hdac_attach_fail;
+
        /* Defer remaining of initialization until interrupts are enabled */
        sc->intrhook.ich_func = hdac_attach2;
        sc->intrhook.ich_arg = (void *)sc;

Reply via email to