Changeset 1d432f9 in mainline for kernel/generic/src/mm/as.c

Timestamp:

2010-05-31T22:33:44Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

179d052

Parents:

3a2b636

Message:

Reflect assumptions about lock and interrupt state in functions themselves.

File:

• kernel/generic/src/mm/as.c (modified) (9 diffs)

kernel/generic/src/mm/as.c

@@ -817 +817 @@
                 [PF_ACCESS_EXEC] = AS_AREA_EXEC
         };
+
+        ASSERT(interrupts_disabled());
+        ASSERT(mutex_locked(&area->lock));
         
         if (!(area->flags & flagmap[access]))
@@ -1221 +1224 @@
 unsigned int as_area_get_flags(as_area_t *area)
 {
+        ASSERT(interrupts_disabled());
+        ASSERT(mutex_locked(&area->lock));
+
         return area_flags_to_page_flags(area->flags);
 }
@@ -1322 +1328 @@
 as_area_t *find_area_and_lock(as_t *as, uintptr_t va)
 {
+        ASSERT(interrupts_disabled());
+        ASSERT(mutex_locked(&as->lock));
+
         btree_node_t *leaf;
         as_area_t *area = (as_area_t *) btree_search(&as->as_area_btree, va, &leaf);
@@ -1386 +1395 @@
     as_area_t *avoid_area)
 {
+        ASSERT(interrupts_disabled());
+        ASSERT(mutex_locked(&as->lock));
+
         /*
          * We don't want any area to have conflicts with NULL page.
@@ -1473 +1485 @@
 /** Return size of the address space area with given base.
  *
- * @param base Arbitrary address insede the address space area.
+ * @param base Arbitrary address inside the address space area.
  *
  * @return Size of the address space area in bytes or zero if it
@@ -1484 +1496 @@
         
         ipl_t ipl = interrupts_disable();
+        page_table_lock(AS, true);
         as_area_t *src_area = find_area_and_lock(AS, base);
         
@@ -1492 +1505 @@
                 size = 0;
         
+        page_table_unlock(AS, true);
         interrupts_restore(ipl);
         return size;
@@ -1509 +1523 @@
 int used_space_insert(as_area_t *area, uintptr_t page, size_t count)
 {
+        ASSERT(mutex_locked(&area->lock));
         ASSERT(page == ALIGN_DOWN(page, PAGE_SIZE));
         ASSERT(count);
@@ -1815 +1830 @@
 int used_space_remove(as_area_t *area, uintptr_t page, size_t count)
 {
+        ASSERT(mutex_locked(&area->lock));
         ASSERT(page == ALIGN_DOWN(page, PAGE_SIZE));
         ASSERT(count);