Page MenuHomeFreeBSD
Files F171081479

D47521.id146295.diff
No OneTemporary
Actions

D47521.id146295.diff
View Options

diff --git a/sys/arm/arm/busdma_machdep.c b/sys/arm/arm/busdma_machdep.c
--- a/sys/arm/arm/busdma_machdep.c
+++ b/sys/arm/arm/busdma_machdep.c
@@ -509,6 +509,7 @@
error = EBUSY;
goto out;
}
+ destroy_bounce_zone(dmat);
free(dmat, M_BUSDMA);
}
out:
diff --git a/sys/arm64/arm64/busdma_bounce.c b/sys/arm64/arm64/busdma_bounce.c
--- a/sys/arm64/arm64/busdma_bounce.c
+++ b/sys/arm64/arm64/busdma_bounce.c
@@ -323,6 +323,7 @@
error = EBUSY;
goto out;
}
+ destroy_bounce_zone(dmat);
if (dmat->segments != NULL)
free(dmat->segments, M_DEVBUF);
free(dmat, M_DEVBUF);
diff --git a/sys/kern/subr_busdma_bounce.c b/sys/kern/subr_busdma_bounce.c
--- a/sys/kern/subr_busdma_bounce.c
+++ b/sys/kern/subr_busdma_bounce.c
@@ -76,6 +76,7 @@
#ifdef dmat_domain
int domain;
#endif
+ int refcount;
sbintime_t total_deferred_time;
bus_size_t alignment;
bus_addr_t lowaddr;
@@ -170,6 +171,7 @@
bool start_thread;
/* Check to see if we already have a suitable zone */
+ mtx_lock(&bounce_lock);
STAILQ_FOREACH(bz, &bounce_zone_list, links) {
if ((dmat_alignment(dmat) <= bz->alignment) &&
#ifdef dmat_domain
@@ -177,13 +179,18 @@
#endif
(dmat_lowaddr(dmat) >= bz->lowaddr)) {
dmat->bounce_zone = bz;
+ KASSERT(bz->refcount >= 1,
+ ("alloc_bounce_zone: refcount %d", bz->refcount));
+ bz->refcount++;
+ mtx_unlock(&bounce_lock);
return (0);
}
}
- if ((bz = (struct bounce_zone *)malloc(sizeof(*bz), M_BUSDMA,
- M_NOWAIT | M_ZERO)) == NULL)
+ if ((bz = malloc(sizeof(*bz), M_BUSDMA, M_NOWAIT | M_ZERO)) == NULL) {
+ mtx_unlock(&bounce_lock);
return (ENOMEM);
+ }
STAILQ_INIT(&bz->bounce_page_list);
STAILQ_INIT(&bz->bounce_map_waitinglist);
@@ -196,6 +203,7 @@
#ifdef dmat_domain
bz->domain = dmat_domain(dmat);
#endif
+ bz->refcount = 1;
snprintf(bz->zoneid, sizeof(bz->zoneid), "zone%d", busdma_zonecount);
busdma_zonecount++;
snprintf(bz->lowaddrid, sizeof(bz->lowaddrid), "%#jx",
@@ -203,6 +211,7 @@
start_thread = STAILQ_EMPTY(&bounce_zone_list);
STAILQ_INSERT_TAIL(&bounce_zone_list, bz, links);
dmat->bounce_zone = bz;
+ mtx_unlock(&bounce_lock);
sysctl_ctx_init(&bz->sysctl_tree);
bz->sysctl_tree_top = SYSCTL_ADD_NODE(&bz->sysctl_tree,
@@ -261,6 +270,46 @@
return (0);
}
+static void
+destroy_bounce_zone(bus_dma_tag_t dmat)
+{
+ struct bounce_page *bpage;
+ struct bounce_zone *bz;
+
+ bz = dmat->bounce_zone;
+ if (bz == NULL)
+ return;
+
+ mtx_lock(&bounce_lock);
+ KASSERT(bz->refcount > 0,
+ ("destroy_bounce_zone: refcount %d", bz->refcount));
+ if (--bz->refcount > 0) {
+ mtx_unlock(&bounce_lock);
+ return;
+ }
+
+ KASSERT(STAILQ_EMPTY(&bz->bounce_map_waitinglist),
+ ("destroy_bounce_zone: waiting list not empty"));
+ KASSERT(bz->active_bpages == 0,
+ ("destroy_bounce_zone: %d active pages still allocated",
+ bz->active_bpages));
+ KASSERT(bz->total_bpages == bz->free_bpages,
+ ("destroy_bounce_zone: %d/%d pages still allocated",
+ bz->total_bpages - bz->free_bpages, bz->total_bpages));
+
+ while ((bpage = STAILQ_FIRST(&bz->bounce_page_list)) != NULL) {
+ STAILQ_REMOVE_HEAD(&bz->bounce_page_list, links);
+ total_bpages--;
+ free((void *)bpage->vaddr, M_BOUNCE);
+ free(bpage, M_BUSDMA);
+ }
+ STAILQ_REMOVE(&bounce_zone_list, bz, bounce_zone, links);
+ mtx_unlock(&bounce_lock);
+
+ sysctl_ctx_free(&bz->sysctl_tree);
+ free(bz, M_BUSDMA);
+}
+
static int
alloc_bounce_pages(bus_dma_tag_t dmat, u_int numpages)
{
diff --git a/sys/powerpc/powerpc/busdma_machdep.c b/sys/powerpc/powerpc/busdma_machdep.c
--- a/sys/powerpc/powerpc/busdma_machdep.c
+++ b/sys/powerpc/powerpc/busdma_machdep.c
@@ -276,7 +276,7 @@
error = EBUSY;
goto out;
}
-
+ destroy_bounce_zone(dmat);
free(dmat, M_DEVBUF);
}
out:
diff --git a/sys/riscv/riscv/busdma_bounce.c b/sys/riscv/riscv/busdma_bounce.c
--- a/sys/riscv/riscv/busdma_bounce.c
+++ b/sys/riscv/riscv/busdma_bounce.c
@@ -207,6 +207,7 @@
error = EBUSY;
goto out;
}
+ destroy_bounce_zone(dmat);
if (dmat->segments != NULL)
free(dmat->segments, M_DEVBUF);
free(dmat, M_DEVBUF);
diff --git a/sys/x86/x86/busdma_bounce.c b/sys/x86/x86/busdma_bounce.c
--- a/sys/x86/x86/busdma_bounce.c
+++ b/sys/x86/x86/busdma_bounce.c
@@ -256,6 +256,7 @@
error = EBUSY;
goto out;
}
+ destroy_bounce_zone(dmat);
if (dmat->segments != NULL)
free(dmat->segments, M_DEVBUF);
free(dmat, M_DEVBUF);

File Metadata

Mime Type
text/plain
Expires
Wed, Sep 9, 2:27 PM (13 h, 52 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
38592355
Default Alt Text
D47521.id146295.diff (4 KB)

Event Timeline