Thread (9 messages) flat view 9 messages, 3 authors, 2020-08-04

Re: powerpc: build failures in Linus' tree

From: Willy Tarreau <w@1wt.eu>
Date: 2020-08-03 12:18:04
Also in: lkml

On Mon, Aug 03, 2020 at 09:18:00PM +1000, Michael Ellerman wrote:
quoted hunk ↗ jump to hunk
If we just move the include of asm/paca.h below asm-generic/percpu.h
then it avoids the bad circular dependency and we still have paca.h
included from percpu.h as before.

eg:
diff --git a/arch/powerpc/include/asm/percpu.h b/arch/powerpc/include/asm/percpu.h
index dce863a7635c..8e5b7d0b851c 100644
--- a/arch/powerpc/include/asm/percpu.h
+++ b/arch/powerpc/include/asm/percpu.h
@@ -10,8 +10,6 @@
 
 #ifdef CONFIG_SMP
 
-#include <asm/paca.h>
-
 #define __my_cpu_offset local_paca->data_offset
 
 #endif /* CONFIG_SMP */
@@ -19,4 +17,6 @@
 
 #include <asm-generic/percpu.h>
 
+#include <asm/paca.h>
+
 #endif /* _ASM_POWERPC_PERCPU_H_ */

So I think I'm inclined to merge that as a minimal fix that's easy to
backport.
This totally makes sense indeed!
Willy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help