Changes in kernel/generic/src/ddi/ddi.c [5a5269d:597fa24] in mainline

File:

• kernel/generic/src/ddi/ddi.c (modified) (8 diffs)

kernel/generic/src/ddi/ddi.c

@@ -54 +54 @@
 #include <align.h>
 #include <errno.h>
-#include <mem.h>
+#include <memw.h>
 #include <trace.h>
 #include <bitops.h>
@@ -60 +60 @@
 
 /** This lock protects the @c pareas ordered dictionary. */
-static mutex_t pareas_lock;
+static MUTEX_INITIALIZE(pareas_lock, MUTEX_PASSIVE);
 
 /** Ordered dictionary of enabled physical memory areas by base address. */
@@ -74 +74 @@
 {
         odict_initialize(&pareas, pareas_getkey, pareas_cmp);
-        mutex_initialize(&pareas_lock, MUTEX_PASSIVE);
 }
 
@@ -105 +104 @@
 
         mutex_unlock(&pareas_lock);
+}
+
+/** Norify physical area has been unmapped.
+ *
+ * @param parea Physical area
+ */
+void ddi_parea_unmap_notify(parea_t *parea)
+{
+        parea->mapped = false;
+        if (parea->mapped_changed != NULL)
+                parea->mapped_changed(parea->arg);
 }
 
@@ -204 +214 @@
 
 map:
+        backend_data.parea = parea;
+
         if (!as_area_create(TASK->as, flags, FRAMES2SIZE(pages),
             AS_AREA_ATTR_NONE, &phys_backend, &backend_data, virt, bound)) {
@@ -323 +335 @@
                 return EPERM;
 
-        irq_spinlock_lock(&tasks_lock, true);
-
         task_t *task = task_find_by_id(id);
 
-        if ((!task) || (!container_check(CONTAINER, task->container))) {
-                /*
-                 * There is no task with the specified ID
-                 * or the task belongs to a different security
-                 * context.
-                 */
-                irq_spinlock_unlock(&tasks_lock, true);
+        if (!task)
                 return ENOENT;
-        }
-
-        /* Lock the task and release the lock protecting tasks dictionary. */
-        irq_spinlock_exchange(&tasks_lock, &task->lock);
-        errno_t rc = ddi_iospace_enable_arch(task, ioaddr, size);
+
+        errno_t rc = ENOENT;
+
+        irq_spinlock_lock(&task->lock, true);
+
+        /* Check that the task belongs to the correct security context. */
+        if (container_check(CONTAINER, task->container))
+                rc = ddi_iospace_enable_arch(task, ioaddr, size);
+
         irq_spinlock_unlock(&task->lock, true);
-
+        task_release(task);
         return rc;
 }
@@ -364 +372 @@
                 return EPERM;
 
-        irq_spinlock_lock(&tasks_lock, true);
-
         task_t *task = task_find_by_id(id);
 
-        if ((!task) || (!container_check(CONTAINER, task->container))) {
-                /*
-                 * There is no task with the specified ID
-                 * or the task belongs to a different security
-                 * context.
-                 */
-                irq_spinlock_unlock(&tasks_lock, true);
+        if (!task)
                 return ENOENT;
-        }
-
-        /* Lock the task and release the lock protecting tasks dictionary. */
-        irq_spinlock_exchange(&tasks_lock, &task->lock);
-        errno_t rc = ddi_iospace_disable_arch(task, ioaddr, size);
+
+        errno_t rc = ENOENT;
+
+        irq_spinlock_lock(&task->lock, true);
+
+        /* Check that the task belongs to the correct security context. */
+        if (container_check(CONTAINER, task->container))
+                rc = ddi_iospace_disable_arch(task, ioaddr, size);
+
         irq_spinlock_unlock(&task->lock, true);
-
+        task_release(task);
         return rc;
 }
@@ -443 +447 @@
         backend_data.frames = frames;
         backend_data.anonymous = true;
+        backend_data.parea = NULL;
 
         if (!as_area_create(TASK->as, map_flags, size,