Commit ed4289e8 authored by Michael Ellerman's avatar Michael Ellerman

Revert "powerpc: slightly improve cache helpers"

This reverts commit 6c587584.

It triggers a probable compiler bug on clang which leads to crashes.

With GCC it allows the compiler to use a more efficient register
allocation but current GCC versions never do that at any of the current
call sites, so there's no benefit.
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent d7e23b88
...@@ -107,22 +107,22 @@ extern void _set_L3CR(unsigned long); ...@@ -107,22 +107,22 @@ extern void _set_L3CR(unsigned long);
static inline void dcbz(void *addr) static inline void dcbz(void *addr)
{ {
__asm__ __volatile__ ("dcbz %y0" : : "Z"(*(u8 *)addr) : "memory"); __asm__ __volatile__ ("dcbz 0, %0" : : "r"(addr) : "memory");
} }
static inline void dcbi(void *addr) static inline void dcbi(void *addr)
{ {
__asm__ __volatile__ ("dcbi %y0" : : "Z"(*(u8 *)addr) : "memory"); __asm__ __volatile__ ("dcbi 0, %0" : : "r"(addr) : "memory");
} }
static inline void dcbf(void *addr) static inline void dcbf(void *addr)
{ {
__asm__ __volatile__ ("dcbf %y0" : : "Z"(*(u8 *)addr) : "memory"); __asm__ __volatile__ ("dcbf 0, %0" : : "r"(addr) : "memory");
} }
static inline void dcbst(void *addr) static inline void dcbst(void *addr)
{ {
__asm__ __volatile__ ("dcbst %y0" : : "Z"(*(u8 *)addr) : "memory"); __asm__ __volatile__ ("dcbst 0, %0" : : "r"(addr) : "memory");
} }
#endif /* !__ASSEMBLY__ */ #endif /* !__ASSEMBLY__ */
#endif /* __KERNEL__ */ #endif /* __KERNEL__ */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment