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/arm/versatile | |
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/arm/versatile')
-rw-r--r-- | sys/arm/versatile/versatile_clcd.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sys/arm/versatile/versatile_clcd.c b/sys/arm/versatile/versatile_clcd.c index 1dd7778..804c24f 100644 --- a/sys/arm/versatile/versatile_clcd.c +++ b/sys/arm/versatile/versatile_clcd.c @@ -363,8 +363,6 @@ versatile_clcdc_attach(device_t dev) fail: if (sc->fb_base) bus_dmamem_free(sc->dma_tag, sc->fb_base, sc->dma_map); - if (sc->dma_map) - bus_dmamap_destroy(sc->dma_tag, sc->dma_map); if (sc->dma_tag) bus_dma_tag_destroy(sc->dma_tag); return (err); |