ARM: Remove __flush_icache_all() from __flush_dcache_page()
authorRussell King <rmk+kernel@arm.linux.org.uk>
Sat, 24 Oct 2009 22:05:34 +0000 (23:05 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Fri, 4 Dec 2009 14:58:50 +0000 (14:58 +0000)
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>
arch/arm/mm/flush.c

index 302d665..dc66f86 100644 (file)
@@ -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
         * 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),
                                page->index << PAGE_CACHE_SHIFT);
-               __flush_icache_all();
-       }
 }
 
 static void __flush_dcache_aliases(struct address_space *mapping, struct page *page)