Changeset 879585a3 in mainline for kernel/arch/ia64/src/mm/as.c

Timestamp:

2007-03-31T22:22:50Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

31d8e10

Parents:

563c2dd

Message:

Simplify synchronization in as_switch().
The function was oversynchronized, which
was causing deadlocks on the address
space mutex.

Now, address spaces can only be switched
when the asidlock is held. This also protects
stealing of ASIDs. No other synchronization
is necessary.

File:

• kernel/arch/ia64/src/mm/as.c (modified) (3 diffs)

kernel/arch/ia64/src/mm/as.c

@@ -37 +37 @@
 #include <arch/mm/page.h>
 #include <genarch/mm/as_ht.h>
+#include <genarch/mm/page_ht.h>
 #include <genarch/mm/asid_fifo.h>
 #include <mm/asid.h>
-#include <arch.h>
 #include <arch/barrier.h>
-#include <synch/spinlock.h>
 
 /** Architecture dependent address space init. */
@@ -56 +55 @@
 void as_install_arch(as_t *as)
 {
-        ipl_t ipl;
         region_register rr;
         int i;
-        
-        ipl = interrupts_disable();
-        spinlock_lock(&as->lock);
         
         ASSERT(as->asid != ASID_INVALID);
@@ -81 +76 @@
         srlz_d();
         srlz_i();
-        
-        spinlock_unlock(&as->lock);
-        interrupts_restore(ipl);
 }