Changeset c626117 in mainline for kernel/generic/src/mm/frame.c

Timestamp:

2025-04-10T17:31:25Z (5 days ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

master

Children:

9f2f5ee

Parents:

0b47781

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2025-04-10 16:34:47)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2025-04-10 17:31:25)

Message:

Replace uses of MUTEX_ACTIVE with irq_spinlock_t

File:

• kernel/generic/src/mm/frame.c (modified) (5 diffs)

kernel/generic/src/mm/frame.c

@@ -72 +72 @@
  * available.
  */
-static MUTEX_INITIALIZE(mem_avail_mtx, MUTEX_ACTIVE);
+static IRQ_SPINLOCK_INITIALIZE(mem_avail_lock);
 static CONDVAR_INITIALIZE(mem_avail_cv);
 static size_t mem_avail_req = 0;  /**< Number of frames requested. */
@@ -951 +951 @@
 #endif
 
-                /*
-                 * Since the mem_avail_mtx is an active mutex, we need to
-                 * disable interrupts to prevent deadlock with TLB shootdown.
-                 */
-                ipl_t ipl = interrupts_disable();
-                mutex_lock(&mem_avail_mtx);
+                /* Disabled interrupts needed to prevent deadlock with TLB shootdown. */
+                irq_spinlock_lock(&mem_avail_lock, true);
 
                 if (mem_avail_req > 0)
@@ -966 +962 @@
 
                 while (gen == mem_avail_gen)
-                        condvar_wait(&mem_avail_cv, &mem_avail_mtx);
-
-                mutex_unlock(&mem_avail_mtx);
-                interrupts_restore(ipl);
+                        condvar_wait(&mem_avail_cv, &mem_avail_lock);
+
+                irq_spinlock_unlock(&mem_avail_lock, true);
 
 #ifdef CONFIG_DEBUG
@@ -1027 +1022 @@
         irq_spinlock_unlock(&zones.lock, true);
 
-        /*
-         * Signal that some memory has been freed.
-         * Since the mem_avail_mtx is an active mutex,
-         * we need to disable interruptsto prevent deadlock
-         * with TLB shootdown.
-         */
-
-        ipl_t ipl = interrupts_disable();
-        mutex_lock(&mem_avail_mtx);
+        /* Signal that some memory has been freed. */
+
+        /* Disabled interrupts needed to prevent deadlock with TLB shootdown. */
+        irq_spinlock_lock(&mem_avail_lock, true);
 
         if (mem_avail_req > 0)
@@ -1045 +1035 @@
         }
 
-        mutex_unlock(&mem_avail_mtx);
-        interrupts_restore(ipl);
+        irq_spinlock_unlock(&mem_avail_lock, true);
 
         if (!(flags & FRAME_NO_RESERVE))