Skip to content

Commit

Permalink
ARM: Remove __flush_icache_all() from __flush_dcache_page()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Russell King committed Dec 4, 2009
1 parent 2df341e commit f91fb05
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions arch/arm/mm/flush.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit f91fb05

Please sign in to comment.