Skip to content

Commit 203b7b6

Browse files
Matthew Wilcox (Oracle)akpm00
authored andcommitted
mm: rationalise flush_icache_pages() and flush_icache_page()
Move the default (no-op) implementation of flush_icache_pages() to <linux/cacheflush.h> from <asm-generic/cacheflush.h>. Remove the flush_icache_page() wrapper from each architecture into <linux/cacheflush.h>. Link: https://lkml.kernel.org/r/20230802151406.3735276-32-willy@infradead.org Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
1 parent 29269ad commit 203b7b6

17 files changed

Lines changed: 14 additions & 56 deletions

File tree

arch/alpha/include/asm/cacheflush.h

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,6 @@ extern void flush_icache_user_page(struct vm_area_struct *vma,
5353
#define flush_icache_user_page flush_icache_user_page
5454
#endif /* CONFIG_SMP */
5555

56-
/* This is used only in __do_fault and do_swap_page. */
57-
#define flush_icache_page(vma, page) \
58-
flush_icache_user_page((vma), (page), 0, 0)
59-
6056
/*
6157
* Both implementations of flush_icache_user_page flush the entire
6258
* address space, so one call, no matter how many pages.
@@ -66,6 +62,7 @@ static inline void flush_icache_pages(struct vm_area_struct *vma,
6662
{
6763
flush_icache_user_page(vma, page, 0, 0);
6864
}
65+
#define flush_icache_pages flush_icache_pages
6966

7067
#include <asm-generic/cacheflush.h>
7168

arch/arc/include/asm/cacheflush.h

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,6 @@
1818
#include <linux/mm.h>
1919
#include <asm/shmparam.h>
2020

21-
/*
22-
* Semantically we need this because icache doesn't snoop dcache/dma.
23-
* However ARC Cache flush requires paddr as well as vaddr, latter not available
24-
* in the flush_icache_page() API. So we no-op it but do the equivalent work
25-
* in update_mmu_cache()
26-
*/
27-
#define flush_icache_page(vma, page)
28-
#define flush_icache_pages(vma, page, nr)
29-
3021
void flush_cache_all(void);
3122

3223
void flush_icache_range(unsigned long kstart, unsigned long kend);

arch/arm/include/asm/cacheflush.h

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -321,13 +321,6 @@ static inline void flush_anon_page(struct vm_area_struct *vma,
321321
#define flush_dcache_mmap_lock(mapping) xa_lock_irq(&mapping->i_pages)
322322
#define flush_dcache_mmap_unlock(mapping) xa_unlock_irq(&mapping->i_pages)
323323

324-
/*
325-
* We don't appear to need to do anything here. In fact, if we did, we'd
326-
* duplicate cache flushing elsewhere performed by flush_dcache_page().
327-
*/
328-
#define flush_icache_page(vma,page) do { } while (0)
329-
#define flush_icache_pages(vma, page, nr) do { } while (0)
330-
331324
/*
332325
* flush_cache_vmap() is used when creating mappings (eg, via vmap,
333326
* vmalloc, ioremap etc) in kernel space for pages. On non-VIPT

arch/csky/abiv1/inc/abi/cacheflush.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ extern void flush_cache_range(struct vm_area_struct *vma, unsigned long start, u
4545
#define flush_cache_vmap(start, end) cache_wbinv_all()
4646
#define flush_cache_vunmap(start, end) cache_wbinv_all()
4747

48-
#define flush_icache_page(vma, page) do {} while (0);
4948
#define flush_icache_range(start, end) cache_wbinv_range(start, end)
5049
#define flush_icache_mm_range(mm, start, end) cache_wbinv_range(start, end)
5150
#define flush_icache_deferred(mm) do {} while (0);

arch/csky/abiv2/inc/abi/cacheflush.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ static inline void flush_dcache_page(struct page *page)
3333

3434
#define flush_dcache_mmap_lock(mapping) do { } while (0)
3535
#define flush_dcache_mmap_unlock(mapping) do { } while (0)
36-
#define flush_icache_page(vma, page) do { } while (0)
3736

3837
#define flush_icache_range(start, end) cache_wbinv_range(start, end)
3938

arch/hexagon/include/asm/cacheflush.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
* - flush_cache_range(vma, start, end) flushes a range of pages
1919
* - flush_icache_range(start, end) flush a range of instructions
2020
* - flush_dcache_page(pg) flushes(wback&invalidates) a page for dcache
21-
* - flush_icache_page(vma, pg) flushes(invalidates) a page for icache
21+
* - flush_icache_pages(vma, pg, nr) flushes(invalidates) nr pages for icache
2222
*
2323
* Need to doublecheck which one is really needed for ptrace stuff to work.
2424
*/

arch/loongarch/include/asm/cacheflush.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,6 @@ void local_flush_icache_range(unsigned long start, unsigned long end);
4646
#define flush_cache_page(vma, vmaddr, pfn) do { } while (0)
4747
#define flush_cache_vmap(start, end) do { } while (0)
4848
#define flush_cache_vunmap(start, end) do { } while (0)
49-
#define flush_icache_page(vma, page) do { } while (0)
50-
#define flush_icache_pages(vma, page) do { } while (0)
5149
#define flush_icache_user_page(vma, page, addr, len) do { } while (0)
5250
#define flush_dcache_page(page) do { } while (0)
5351
#define flush_dcache_mmap_lock(mapping) do { } while (0)

arch/m68k/include/asm/cacheflush_mm.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,6 @@ static inline void __flush_pages_to_ram(void *vaddr, unsigned int nr)
261261
#define flush_dcache_mmap_unlock(mapping) do { } while (0)
262262
#define flush_icache_pages(vma, page, nr) \
263263
__flush_pages_to_ram(page_address(page), nr)
264-
#define flush_icache_page(vma, page) flush_icache_pages(vma, page, 1)
265264

266265
extern void flush_icache_user_page(struct vm_area_struct *vma, struct page *page,
267266
unsigned long addr, int len);

arch/mips/include/asm/cacheflush.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,6 @@ static inline void flush_anon_page(struct vm_area_struct *vma,
8282
__flush_anon_page(page, vmaddr);
8383
}
8484

85-
static inline void flush_icache_pages(struct vm_area_struct *vma,
86-
struct page *page, unsigned int nr)
87-
{
88-
}
89-
#define flush_icache_page(vma, page) flush_icache_pages(vma, page, 1)
90-
9185
extern void (*flush_icache_range)(unsigned long start, unsigned long end);
9286
extern void (*local_flush_icache_range)(unsigned long start, unsigned long end);
9387
extern void (*__flush_icache_user_range)(unsigned long start,

arch/nios2/include/asm/cacheflush.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ void flush_dcache_folio(struct folio *folio);
3535
extern void flush_icache_range(unsigned long start, unsigned long end);
3636
void flush_icache_pages(struct vm_area_struct *vma, struct page *page,
3737
unsigned int nr);
38-
#define flush_icache_page(vma, page) flush_icache_pages(vma, page, 1);
38+
#define flush_icache_pages flush_icache_pages
3939

4040
#define flush_cache_vmap(start, end) flush_dcache_range(start, end)
4141
#define flush_cache_vunmap(start, end) flush_dcache_range(start, end)

0 commit comments

Comments
 (0)