Browse Source

ARM: Remove __flush_icache_all() from __flush_dcache_page()

Both call sites for __flush_dcache_page() end up calling
__flush_icache_all() themselves, so having __flush_dcache_page() do
this as well is wasteful.  Remove the duplicated icache flushing.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Russell King 15 years ago
parent
commit
f91fb05d82
1 changed files with 1 additions and 3 deletions
  1. 1 3
      arch/arm/mm/flush.c

+ 1 - 3
arch/arm/mm/flush.c

@@ -134,11 +134,9 @@ void __flush_dcache_page(struct address_space *mapping, struct page *page)
 	 * we only need to do one flush - which would be at the relevant
 	 * we only need to do one flush - which would be at the relevant
 	 * userspace colour, which is congruent with page->index.
 	 * userspace colour, which is congruent with page->index.
 	 */
 	 */
-	if (mapping && cache_is_vipt_aliasing()) {
+	if (mapping && cache_is_vipt_aliasing())
 		flush_pfn_alias(page_to_pfn(page),
 		flush_pfn_alias(page_to_pfn(page),
 				page->index << PAGE_CACHE_SHIFT);
 				page->index << PAGE_CACHE_SHIFT);
-		__flush_icache_all();
-	}
 }
 }
 
 
 static void __flush_dcache_aliases(struct address_space *mapping, struct page *page)
 static void __flush_dcache_aliases(struct address_space *mapping, struct page *page)