diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2014-11-16 12:59:19 -0500 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2014-11-16 12:59:19 -0500 |
commit | 0dbcae884779fdf7e2239a97ac7488877f0693d9 (patch) | |
tree | c220ca7af9542397538dcd0f288794b258739f8f /arch/x86/mm | |
parent | 47591df505129c9774af6cca2debf283a6e56ed7 (diff) |
x86: mm: Move PAT only functions to mm/pat.c
Commit e00c8cc93c1a "x86: Use new cache mode type in memtype related
functions" broke the ARCH=um build.
arch/x86/include/asm/cacheflush.h:67:36: error: return type is an incomplete type
static inline enum page_cache_mode get_page_memtype(struct page *pg)
The reason is simple. get_page_memtype() and set_page_memtype()
require enum page_cache_mode now, which is defined in
asm/pgtable_types.h. UM does not include that file for obvious reasons.
The simple solution is to move that functions to arch/x86/mm/pat.c
where the only callsites of this are located. They should have been
there in the first place.
Fixes: e00c8cc93c1a "x86: Use new cache mode type in memtype related functions"
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Juergen Gross <jgross@suse.com>
Cc: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/x86/mm')
-rw-r--r-- | arch/x86/mm/pat.c | 69 |
1 files changed, 69 insertions, 0 deletions
diff --git a/arch/x86/mm/pat.c b/arch/x86/mm/pat.c index 4c601276a556..25d2c58489e0 100644 --- a/arch/x86/mm/pat.c +++ b/arch/x86/mm/pat.c | |||
@@ -67,6 +67,75 @@ __setup("debugpat", pat_debug_setup); | |||
67 | 67 | ||
68 | static u64 __read_mostly boot_pat_state; | 68 | static u64 __read_mostly boot_pat_state; |
69 | 69 | ||
70 | #ifdef CONFIG_X86_PAT | ||
71 | /* | ||
72 | * X86 PAT uses page flags WC and Uncached together to keep track of | ||
73 | * memory type of pages that have backing page struct. X86 PAT supports 3 | ||
74 | * different memory types, _PAGE_CACHE_MODE_WB, _PAGE_CACHE_MODE_WC and | ||
75 | * _PAGE_CACHE_MODE_UC_MINUS and fourth state where page's memory type has not | ||
76 | * been changed from its default (value of -1 used to denote this). | ||
77 | * Note we do not support _PAGE_CACHE_MODE_UC here. | ||
78 | */ | ||
79 | |||
80 | #define _PGMT_DEFAULT 0 | ||
81 | #define _PGMT_WC (1UL << PG_arch_1) | ||
82 | #define _PGMT_UC_MINUS (1UL << PG_uncached) | ||
83 | #define _PGMT_WB (1UL << PG_uncached | 1UL << PG_arch_1) | ||
84 | #define _PGMT_MASK (1UL << PG_uncached | 1UL << PG_arch_1) | ||
85 | #define _PGMT_CLEAR_MASK (~_PGMT_MASK) | ||
86 | |||
87 | static inline enum page_cache_mode get_page_memtype(struct page *pg) | ||
88 | { | ||
89 | unsigned long pg_flags = pg->flags & _PGMT_MASK; | ||
90 | |||
91 | if (pg_flags == _PGMT_DEFAULT) | ||
92 | return -1; | ||
93 | else if (pg_flags == _PGMT_WC) | ||
94 | return _PAGE_CACHE_MODE_WC; | ||
95 | else if (pg_flags == _PGMT_UC_MINUS) | ||
96 | return _PAGE_CACHE_MODE_UC_MINUS; | ||
97 | else | ||
98 | return _PAGE_CACHE_MODE_WB; | ||
99 | } | ||
100 | |||
101 | static inline void set_page_memtype(struct page *pg, | ||
102 | enum page_cache_mode memtype) | ||
103 | { | ||
104 | unsigned long memtype_flags; | ||
105 | unsigned long old_flags; | ||
106 | unsigned long new_flags; | ||
107 | |||
108 | switch (memtype) { | ||
109 | case _PAGE_CACHE_MODE_WC: | ||
110 | memtype_flags = _PGMT_WC; | ||
111 | break; | ||
112 | case _PAGE_CACHE_MODE_UC_MINUS: | ||
113 | memtype_flags = _PGMT_UC_MINUS; | ||
114 | break; | ||
115 | case _PAGE_CACHE_MODE_WB: | ||
116 | memtype_flags = _PGMT_WB; | ||
117 | break; | ||
118 | default: | ||
119 | memtype_flags = _PGMT_DEFAULT; | ||
120 | break; | ||
121 | } | ||
122 | |||
123 | do { | ||
124 | old_flags = pg->flags; | ||
125 | new_flags = (old_flags & _PGMT_CLEAR_MASK) | memtype_flags; | ||
126 | } while (cmpxchg(&pg->flags, old_flags, new_flags) != old_flags); | ||
127 | } | ||
128 | #else | ||
129 | static inline enum page_cache_mode get_page_memtype(struct page *pg) | ||
130 | { | ||
131 | return -1; | ||
132 | } | ||
133 | static inline void set_page_memtype(struct page *pg, | ||
134 | enum page_cache_mode memtype) | ||
135 | { | ||
136 | } | ||
137 | #endif | ||
138 | |||
70 | enum { | 139 | enum { |
71 | PAT_UC = 0, /* uncached */ | 140 | PAT_UC = 0, /* uncached */ |
72 | PAT_WC = 1, /* Write combining */ | 141 | PAT_WC = 1, /* Write combining */ |