[src/trunk]: src/sys/arch/sparc/sparc Protect extent(9) ops with splhigh().

[pk <pk%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/a39ba0242acc
branches:  trunk
changeset: 472085:a39ba0242acc
user:      pk <pk%NetBSD.org@localhost>
date:      Tue Apr 20 20:15:48 1999 +0000

description:
Protect extent(9) ops with splhigh().

diffstat:

 sys/arch/sparc/sparc/iommu.c |  44 +++++++++++++++++++++++++++++++++-----------
 1 files changed, 33 insertions(+), 11 deletions(-)

diffs (114 lines):

diff -r 11f2aa891065 -r a39ba0242acc sys/arch/sparc/sparc/iommu.c
--- a/sys/arch/sparc/sparc/iommu.c      Tue Apr 20 20:15:47 1999 +0000
+++ b/sys/arch/sparc/sparc/iommu.c      Tue Apr 20 20:15:48 1999 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: iommu.c,v 1.32 1999/03/26 23:41:35 mycroft Exp $ */
+/*     $NetBSD: iommu.c,v 1.33 1999/04/20 20:15:48 pk Exp $ */
 
 /*
  * Copyright (c) 1996
@@ -71,6 +71,11 @@
 int    has_iocache;
 u_long dvma_cachealign;
 
+/*
+ * Note: operations on the extent map are being protected with
+ * splhigh(), since we cannot predict at which interrupt priority
+ * our clients will run.
+ */
 struct extent *iommu_dvmamap;
 
 
@@ -456,6 +461,7 @@
        vaddr_t va = (vaddr_t)buf;
        u_long align, voff;
        pmap_t pmap;
+       int s, error;
 
        /*
         * Remember page offset, then truncate the buffer address to
@@ -476,9 +482,13 @@
        align = dvma_cachealign ? dvma_cachealign : NBPG;
        boundary = map->_dm_boundary;
 
-       if (extent_alloc1(iommu_dvmamap, sgsize, align, va & (align-1),
-                         boundary, EX_NOWAIT, (u_long *)&dva) != 0)
-               return (ENOMEM);
+       s = splhigh();
+       error = extent_alloc1(iommu_dvmamap, sgsize, align, va & (align-1),
+                         boundary, EX_NOWAIT, (u_long *)&dva);
+       splx(s);
+
+       if (error != 0)
+               return (error);
 
        cpuinfo.cache_flush(buf, buflen);
 
@@ -567,6 +577,7 @@
 {
        bus_addr_t addr;
        bus_size_t len;
+       int s, error;
 
        if (map->dm_nsegs != 1)
                panic("_bus_dmamap_unload: nsegs = %d", map->dm_nsegs);
@@ -577,7 +588,10 @@
        addr &= ~PGOFSET;
 
        iommu_remove(addr, len);
-       if (extent_free(iommu_dvmamap, addr, len, EX_NOWAIT) != 0)
+       s = splhigh();
+       error = extent_free(iommu_dvmamap, addr, len, EX_NOWAIT);
+       splx(s);
+       if (error != 0)
                printf("warning: %ld of DVMA space lost\n", (long)len);
 
        /* Mark the mappings as invalid. */
@@ -619,7 +633,7 @@
        paddr_t pa;
        bus_addr_t dva;
        vm_page_t m;
-       int error;
+       int s, error;
        struct pglist *mlist;
 
        size = round_page(size);
@@ -628,10 +642,14 @@
        if (error != 0)
                return (error);
 
-       if (extent_alloc(iommu_dvmamap, size, alignment, boundary,
-                        (flags & BUS_DMA_NOWAIT) == 0 ? EX_WAITOK : EX_NOWAIT,
-                        (u_long *)&dva) != 0)
-               return (ENOMEM);
+       s = splhigh();
+       error = extent_alloc(iommu_dvmamap, size, alignment, boundary,
+                            (flags & BUS_DMA_NOWAIT) == 0
+                               ? EX_WAITOK : EX_NOWAIT,
+                            (u_long *)&dva);
+       splx(s);
+       if (error != 0)
+               return (error);
 
        /*
         * Compute the location, size, and number of segments actually
@@ -665,6 +683,7 @@
 {
        bus_addr_t addr;
        bus_size_t len;
+       int s, error;
 
        if (nsegs != 1)
                panic("bus_dmamem_free: nsegs = %d", nsegs);
@@ -673,7 +692,10 @@
        len = segs[0].ds_len;
 
        iommu_remove(addr, len);
-       if (extent_free(iommu_dvmamap, addr, len, EX_NOWAIT) != 0)
+       s = splhigh();
+       error = extent_free(iommu_dvmamap, addr, len, EX_NOWAIT);
+       splx(s);
+       if (error != 0)
                printf("warning: %ld of DVMA space lost\n", (long)len);
        /*
         * Return the list of pages back to the VM system.