Changes in / [3d4750f:bf31e3f] in mainline

Location:

kernel

Files:

• genarch/src/acpi/acpi.c (modified) (1 diff)
• generic/include/mm/km.h (modified) (1 diff)
• generic/src/mm/km.c (modified) (2 diffs)
• test/mm/mapping1.c (modified) (2 diffs)
• test/mm/mapping1.def (modified) (1 diff)

kernel/genarch/src/acpi/acpi.c

@@ -102 +102 @@
 
         /* Start with mapping the header only. */
-        vhdr = (struct acpi_sdt_header *) km_map((uintptr_t) psdt,
+        vhdr = (struct acpi_sdt_header *) km_map_structure((uintptr_t) psdt,
             sizeof(struct acpi_sdt_header), PAGE_READ | PAGE_NOT_CACHEABLE); 
 
         /* Now we can map the entire structure. */
-        vsdt = (struct acpi_sdt_header *) km_map((uintptr_t) psdt,
+        vsdt = (struct acpi_sdt_header *) km_map_structure((uintptr_t) psdt,
             vhdr->length, PAGE_WRITE | PAGE_NOT_CACHEABLE);
         

kernel/generic/include/mm/km.h

@@ -50 +50 @@
 
 extern uintptr_t km_map(uintptr_t, size_t, unsigned int);
-extern void km_unmap(uintptr_t, size_t);
+extern uintptr_t km_map_structure(uintptr_t, size_t, unsigned int);
 
 extern uintptr_t km_temporary_page_get(uintptr_t *, frame_flags_t);

kernel/generic/src/mm/km.c

@@ -124 +124 @@
 }
 
-static uintptr_t
-km_map_aligned(uintptr_t paddr, size_t size, unsigned int flags)
+uintptr_t km_map(uintptr_t paddr, size_t size, unsigned int flags)
 {
         uintptr_t vaddr;
+        size_t asize;
         size_t align;
         uintptr_t offs;
 
-        ASSERT(ALIGN_DOWN(paddr, FRAME_SIZE) == paddr);
-        ASSERT(ALIGN_UP(size, FRAME_SIZE) == size);
-
+        asize = ALIGN_UP(size, PAGE_SIZE);
         align = ispwr2(size) ? size : (1U << (fnzb(size) + 1));
-        vaddr = km_page_alloc(size, max(PAGE_SIZE, align));
+        vaddr = km_page_alloc(asize, max(PAGE_SIZE, align));
 
         page_table_lock(AS_KERNEL, true);
-        for (offs = 0; offs < size; offs += PAGE_SIZE) {
+        for (offs = 0; offs < asize; offs += PAGE_SIZE) {
                 page_mapping_insert(AS_KERNEL, vaddr + offs, paddr + offs,
                     flags);
@@ -147 +145 @@
 }
 
-static void km_unmap_aligned(uintptr_t vaddr, size_t size)
-{
-        uintptr_t offs;
-        ipl_t ipl;
-
-        ASSERT(ALIGN_DOWN(vaddr, PAGE_SIZE) == vaddr);
-        ASSERT(ALIGN_UP(size, PAGE_SIZE) == size);
-
-        page_table_lock(AS_KERNEL, true);
-
-        ipl = tlb_shootdown_start(TLB_INVL_ASID, ASID_KERNEL, 0, 0);
-
-        for (offs = 0; offs < size; offs += PAGE_SIZE)
-                page_mapping_remove(AS_KERNEL, vaddr + offs);
-
-        tlb_invalidate_asid(ASID_KERNEL);
-
-        as_invalidate_translation_cache(AS_KERNEL, 0, -1);
-        tlb_shootdown_finalize(ipl);
-        page_table_unlock(AS_KERNEL, true);
-
-        km_page_free(vaddr, size);
-}
-
-/** Map a piece of physical address space into the virtual address space.
- *
- * @param paddr         Physical address to be mapped. May be unaligned.
- * @param size          Size of area starting at paddr to be mapped.
- * @param flags         Protection flags to be used for the mapping.
- *
- * @return New virtual address mapped to paddr.
- */
-uintptr_t km_map(uintptr_t paddr, size_t size, unsigned int flags)
-{
+uintptr_t km_map_structure(uintptr_t paddr, size_t size, unsigned int flags)
+{
+        size_t offs = paddr - ALIGN_DOWN(paddr, FRAME_SIZE); 
         uintptr_t page;
-        size_t offs; 
-
-        offs = paddr - ALIGN_DOWN(paddr, FRAME_SIZE); 
-        page = km_map_aligned(ALIGN_DOWN(paddr, FRAME_SIZE),
-            ALIGN_UP(size + offs, FRAME_SIZE), flags);
-
+
+        page = km_map(ALIGN_DOWN(paddr, FRAME_SIZE), size + offs, flags);
         return page + offs;
 }
 
-/** Unmap a piece of virtual address space.
- *
- * @param vaddr         Virtual address to be unmapped. May be unaligned, but
- *                      it must a value previously returned by km_map().
- * @param size          Size of area starting at vaddr to be unmapped.
- */
-void km_unmap(uintptr_t vaddr, size_t size)
-{
-        size_t offs; 
-
-        offs = vaddr - ALIGN_DOWN(vaddr, PAGE_SIZE); 
-        km_unmap_aligned(ALIGN_DOWN(vaddr, PAGE_SIZE),
-            ALIGN_UP(size + offs, PAGE_SIZE));
-}
-
-/** Unmap kernel non-identity page.
+/** Unmap kernen non-identity page.
  *
  * @param[in] page      Non-identity page to be unmapped.

kernel/test/mm/mapping1.c

@@ -66 +66 @@
                 v = *((uint32_t *) page1);
         
-                if (v != TEST_MAGIC) {
-                        km_unmap(page0, PAGE_SIZE);
-                        km_unmap(page1, PAGE_SIZE);
-                        frame_free(frame);
+                if (v != TEST_MAGIC)
                         return "Criss-cross read does not match the value written.";
-                }
 
                 TPRINTF("Writing zero using the second virtual address.\n");
@@ -81 +77 @@
                 v = *((uint32_t *) page0);
         
-                if (v != 0) {
-                        km_unmap(page0, PAGE_SIZE);
-                        km_unmap(page1, PAGE_SIZE);
-                        frame_free(frame);
+                if (v != 0)
                         return "Criss-cross read does not match the value written.";
-                }
         }
 
-        km_unmap(page0, PAGE_SIZE);
-        km_unmap(page1, PAGE_SIZE);
-        frame_free(frame);
+        // FIXME: do not leak frame, page0 and page1
         
         return NULL;

kernel/test/mm/mapping1.def

@@ -3 +3 @@
         "Mapping test",
         &test_mapping1,
-        true    
+        false
 },