diff options
| author | jhb <jhb@FreeBSD.org> | 2014-06-10 20:25:45 +0000 |
|---|---|---|
| committer | jhb <jhb@FreeBSD.org> | 2014-06-10 20:25:45 +0000 |
| commit | 02e632b71287ac53801c62c26a1c00926fca8e2d (patch) | |
| tree | fce070802b2d12919f979d51576d5aad6077a6c2 /sys/dev/advansys | |
| parent | e250a91c1da4cfd9eb0dab8e0aac2bc6fc7a8176 (diff) | |
| download | FreeBSD-src-02e632b71287ac53801c62c26a1c00926fca8e2d.zip FreeBSD-src-02e632b71287ac53801c62c26a1c00926fca8e2d.tar.gz | |
Don't destroy bus_dma maps created by bus_dmamem_alloc(). In some cases,
don't create a map before calling bus_dmamem_alloc() (such maps were
leaked). It is believed that the extra destroy of the map was generally
harmless since bus_dmamem_alloc() often uses special maps for which
bus_dmamap_destroy() is a no-op (e.g. on x86).
Reviewed by: scottl
Diffstat (limited to 'sys/dev/advansys')
| -rw-r--r-- | sys/dev/advansys/adwcam.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sys/dev/advansys/adwcam.c b/sys/dev/advansys/adwcam.c index d925045..cb38a61 100644 --- a/sys/dev/advansys/adwcam.c +++ b/sys/dev/advansys/adwcam.c @@ -777,7 +777,6 @@ adw_free(struct adw_softc *adw) case 7: bus_dmamem_free(adw->acb_dmat, adw->acbs, adw->acb_dmamap); - bus_dmamap_destroy(adw->acb_dmat, adw->acb_dmamap); case 6: bus_dma_tag_destroy(adw->acb_dmat); case 5: @@ -785,7 +784,6 @@ adw_free(struct adw_softc *adw) case 4: bus_dmamem_free(adw->carrier_dmat, adw->carriers, adw->carrier_dmamap); - bus_dmamap_destroy(adw->carrier_dmat, adw->carrier_dmamap); case 3: bus_dma_tag_destroy(adw->carrier_dmat); case 2: |
