Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 25 Apr 2014 21:24:34 +0000 (UTC)
From:      Marius Strobl <marius@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r264938 - stable/10/sys/dev/aac
Message-ID:  <201404252124.s3PLOYK1048792@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: marius
Date: Fri Apr 25 21:24:33 2014
New Revision: 264938
URL: http://svnweb.freebsd.org/changeset/base/264938

Log:
  MFC: r260044
  
  Free the MSI again on detach if allocated. Arguably, this code would be
  better off living in aac_pci.c, but it doesn't seem worth creating a
  aac_pci_detach() and it's also not the first PCI-specific bit in aac.c

Modified:
  stable/10/sys/dev/aac/aac.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/dev/aac/aac.c
==============================================================================
--- stable/10/sys/dev/aac/aac.c	Fri Apr 25 21:20:28 2014	(r264937)
+++ stable/10/sys/dev/aac/aac.c	Fri Apr 25 21:24:33 2014	(r264938)
@@ -631,9 +631,11 @@ aac_free(struct aac_softc *sc)
 	/* disconnect the interrupt handler */
 	if (sc->aac_intr)
 		bus_teardown_intr(sc->aac_dev, sc->aac_irq, sc->aac_intr);
-	if (sc->aac_irq != NULL)
+	if (sc->aac_irq != NULL) {
 		bus_release_resource(sc->aac_dev, SYS_RES_IRQ,
 		    rman_get_rid(sc->aac_irq), sc->aac_irq);
+		pci_release_msi(sc->aac_dev);
+	}
 
 	/* destroy data-transfer DMA tag */
 	if (sc->aac_buffer_dmat)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201404252124.s3PLOYK1048792>