Skip to content

Commit 0f0a0a6

Browse files
BenjaminGrayNp1mpe
authored andcommitted
cxl: Use radix__flush_all_mm instead of generic flush_all_mm
The generic implementation of this function isn't really generic (Hash is not implemented). Unfortunately, the runtime warnings cannot be replaced with BUILD_BUG's, so it seems safer not to provide a stub in the first place. Signed-off-by: Benjamin Gray <[email protected]> Reviewed-by: Andrew Donnellan <[email protected]> Reviewed-by: Christophe Leroy <[email protected]> Signed-off-by: Michael Ellerman <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent baf1ed2 commit 0f0a0a6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

arch/powerpc/include/asm/mmu_context.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,8 +151,8 @@ static inline void mm_context_remove_copro(struct mm_struct *mm)
151151
* nMMU and/or PSL need to be cleaned up.
152152
*
153153
* Both the 'copros' and 'active_cpus' counts are looked at in
154-
* flush_all_mm() to determine the scope (local/global) of the
155-
* TLBIs, so we need to flush first before decrementing
154+
* radix__flush_all_mm() to determine the scope (local/global)
155+
* of the TLBIs, so we need to flush first before decrementing
156156
* 'copros'. If this API is used by several callers for the
157157
* same context, it can lead to over-flushing. It's hopefully
158158
* not common enough to be a problem.
@@ -164,7 +164,7 @@ static inline void mm_context_remove_copro(struct mm_struct *mm)
164164
* in-between.
165165
*/
166166
if (radix_enabled()) {
167-
flush_all_mm(mm);
167+
radix__flush_all_mm(mm);
168168

169169
c = atomic_dec_if_positive(&mm->context.copros);
170170
/* Detect imbalance between add and remove */

0 commit comments

Comments
 (0)