Don't lock in the attach routine. It isn't required. Register the

interrupt handler last.  This gets rid of the sleep while locked
messages.

Reviewed by: ambrisko

1 files changed, 7 insertions, 4 deletions

diff --git a/sys/dev/an/if_an_pci.c b/sys/dev/an/if_an_pci.c
index b3cbb4e..2969daa 100644
--- a/sys/dev/an/if_an_pci.c
+++ b/sys/dev/an/if_an_pci.c
@@ -229,14 +229,17 @@ an_attach_pci(dev)
 		goto fail;
         }
 
-	error = bus_setup_intr(dev, sc->irq_res, INTR_TYPE_NET,
-	    an_intr, sc, &sc->irq_handle);
+	sc->an_dev = dev;
+	error = an_attach(sc, device_get_unit(dev), flags);
 	if (error) {
 		goto fail;
 	}
 
-	sc->an_dev = dev;
-	error = an_attach(sc, device_get_unit(dev), flags);
+	/*
+	 * Must setup the interrupt after the an_attach to prevent racing.
+	 */
+	error = bus_setup_intr(dev, sc->irq_res, INTR_TYPE_NET,
+	    an_intr, sc, &sc->irq_handle);
 
 fail:
 	if (error)