diff options
| author | Russell King <rmk+kernel@arm.linux.org.uk> | 2009-10-24 23:05:34 +0100 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2009-12-04 14:58:50 +0000 |
| commit | f91fb05d826a43063fa0aa2ec30c23d3993a208d (patch) | |
| tree | 9ada5aefa09d40b70ea3b8a5e89d7ce6aa2a5fd9 | |
| parent | 2df341edf6b8a2db7f414d00faeadccbdd9844ab (diff) | |
| download | linux-f91fb05d826a43063fa0aa2ec30c23d3993a208d.tar.bz2 | |
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>
| -rw-r--r-- | arch/arm/mm/flush.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/arm/mm/flush.c b/arch/arm/mm/flush.c index 302d66517488..dc66f867bec4 100644 --- a/arch/arm/mm/flush.c +++ b/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 * 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) |