Changeset f8ddd17 in mainline for kernel/arch/sparc64/src/mm/as.c

Timestamp:

2006-12-09T20:20:50Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

b82a13c

Parents:

9ab9c2ec

Message:

Rework support for virtually indexed cache.
Instead of repeatedly flushing the data cache, which was a huge overkill, refuse to create an illegal address alias
in the kernel (again) and allocate appropriate page color in userspace instead. Extend the detection also to
SYS_PHYSMEM_MAP syscall.

Add support for tracking physical memory areas mappable by SYS_PHYSMEM_MAP.

Lots of coding style changes.

File:

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

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

@@ -49 +49 @@
 #include <macros.h>
 #endif /* CONFIG_TSB */
-
-#ifdef CONFIG_VIRT_IDX_DCACHE
-#include <arch/mm/cache.h>
-#endif /* CONFIG_VIRT_IDX_DCACHE */
 
 /** Architecture dependent address space init. */
@@ -163 +159 @@
         dtsb_base_write(tsb_base.value);
 #endif
-#ifdef CONFIG_VIRT_IDX_DCACHE
-        if (as->dcache_flush_on_install) {
-                /*
-                 * Some mappings in this address space are illegal address
-                 * aliases. Upon their creation, the dcache_flush_on_install
-                 * flag was set.
-                 *
-                 * We are now obliged to flush the D-cache in order to guarantee
-                 * that there will be at most one cache line for each address
-                 * alias.
-                 *
-                 * This flush performs a cleanup after another address space in
-                 * which the alias might have existed.
-                 */
-                dcache_flush();
-        }
-#endif /* CONFIG_VIRT_IDX_DCACHE */
 }
 
@@ -214 +193 @@
         }
 #endif
-#ifdef CONFIG_VIRT_IDX_DCACHE
-        if (as->dcache_flush_on_deinstall) {
-                /*
-                 * Some mappings in this address space are illegal address
-                 * aliases. Upon their creation, the dcache_flush_on_deinstall
-                 * flag was set.
-                 *
-                 * We are now obliged to flush the D-cache in order to guarantee
-                 * that there will be at most one cache line for each address
-                 * alias.
-                 *
-                 * This flush performs a cleanup after this address space. It is
-                 * necessary because other address spaces that contain the same
-                 * alias are not necessarily aware of the need to carry out the
-                 * cache flush. The only address spaces that are aware of it are
-                 * those that created the illegal alias. 
-                 */
-                dcache_flush();
-        }
-#endif /* CONFIG_VIRT_IDX_DCACHE */
 }