Re: [PATCH 13/35] Disintegrate asm/system.h for M68K [ver #2]

From: Greg Ungerer
Date: Wed Mar 14 2012 - 08:09:52 EST


Hi David,

On 03/13/2012 09:38 AM, David Howells wrote:
Disintegrate asm/system.h for M68K.

Signed-off-by: David Howells<dhowells@xxxxxxxxxx>
cc: linux-m68k@xxxxxxxxxxxxxxxxxxxx
---

arch/m68k/amiga/amisound.c | 1
arch/m68k/amiga/config.c | 1
arch/m68k/apollo/config.c | 1
arch/m68k/atari/ataints.c | 1
arch/m68k/atari/atasound.c | 1
arch/m68k/atari/config.c | 1
arch/m68k/bvme6000/config.c | 1
arch/m68k/bvme6000/rtc.c | 1
arch/m68k/hp300/time.c | 1
arch/m68k/include/asm/atomic.h | 2
arch/m68k/include/asm/barrier.h | 20 ++++
arch/m68k/include/asm/cmpxchg.h | 129 +++++++++++++++++++++++
arch/m68k/include/asm/exec.h | 6 +
arch/m68k/include/asm/sun3xflop.h | 1
arch/m68k/include/asm/switch_to.h | 41 +++++++
arch/m68k/include/asm/system.h | 198 +----------------------------------
arch/m68k/kernel/ints.c | 1
arch/m68k/kernel/irq.c | 1
arch/m68k/kernel/process_mm.c | 1
arch/m68k/kernel/process_no.c | 1
arch/m68k/kernel/ptrace_mm.c | 1
arch/m68k/kernel/ptrace_no.c | 1
arch/m68k/kernel/traps.c | 1
arch/m68k/kernel/vectors.c | 1
arch/m68k/mac/config.c | 1
arch/m68k/mac/misc.c | 1
arch/m68k/mm/fault.c | 1
arch/m68k/mm/init_mm.c | 1
arch/m68k/mm/init_no.c | 1
arch/m68k/mm/kmap.c | 1
arch/m68k/mm/memory.c | 1
arch/m68k/mm/motorola.c | 1
arch/m68k/mm/sun3mmu.c | 1
arch/m68k/mvme147/config.c | 1
arch/m68k/mvme16x/config.c | 1
arch/m68k/mvme16x/rtc.c | 1
arch/m68k/platform/68328/config.c | 1
arch/m68k/platform/68328/timers.c | 1
arch/m68k/platform/68360/config.c | 1
arch/m68k/platform/68EZ328/config.c | 1
arch/m68k/platform/68VZ328/config.c | 1
arch/m68k/q40/config.c | 1
arch/m68k/q40/q40ints.c | 1
arch/m68k/sun3/intersil.c | 1
arch/m68k/sun3/mmu_emu.c | 1
arch/m68k/sun3/prom/console.c | 1
arch/m68k/sun3x/config.c | 1
arch/m68k/sun3x/time.c | 1
48 files changed, 202 insertions(+), 236 deletions(-)
create mode 100644 arch/m68k/include/asm/barrier.h
create mode 100644 arch/m68k/include/asm/cmpxchg.h
create mode 100644 arch/m68k/include/asm/exec.h
create mode 100644 arch/m68k/include/asm/switch_to.h

[snip]
>
diff --git a/arch/m68k/include/asm/cmpxchg.h b/arch/m68k/include/asm/cmpxchg.h
new file mode 100644
index 0000000..f8acc0f
--- /dev/null
+++ b/arch/m68k/include/asm/cmpxchg.h
@@ -0,0 +1,129 @@
+#ifndef __ARCH_M68K_CMPXCHG__
+#define __ARCH_M68K_CMPXCHG__
+
+#include<linux/irqflags.h>
+
+struct __xchg_dummy { unsigned long a[100]; };
+#define __xg(x) ((volatile struct __xchg_dummy *)(x))
+
+#ifndef CONFIG_RMW_INSNS
+static inline unsigned long __xchg(unsigned long x, volatile void * ptr, int size)
+{
+ unsigned long flags, tmp;
+
+ local_irq_save(flags);
+
+ switch (size) {
+ case 1:
+ tmp = *(u8 *)ptr;
+ *(u8 *)ptr = x;
+ x = tmp;
+ break;
+ case 2:
+ tmp = *(u16 *)ptr;
+ *(u16 *)ptr = x;
+ x = tmp;
+ break;
+ case 4:
+ tmp = *(u32 *)ptr;
+ *(u32 *)ptr = x;
+ x = tmp;
+ break;
+ default:
+ BUG();

This breaks compilation for m68k targets with:

CC net/ipv4/ip_input.o
In file included from /home/gerg/new-wave.320/linux-3.x/arch/m68k/include/asm/system.h:3:0,
from net/ipv4/ip_input.c:116:
/home/gerg/new-wave.320/linux-3.x/arch/m68k/include/asm/cmpxchg.h: In function ‘__xchg’:
/home/gerg/new-wave.320/linux-3.x/arch/m68k/include/asm/cmpxchg.h:33:3: error: implicit declaration of function ‘BUG’

The old system.h included linux/kernel.h which includes asm/bug.h.
Just including asm/bug.h here in cmpxchg.h is not enough to fix it.
If you include linux/kernel.h though it compiles cleanly.

Regards
Greg



------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: gerg@xxxxxxxxxxxx
SnapGear Group, McAfee PHONE: +61 7 3435 2888
8 Gardner Close, FAX: +61 7 3891 3630
Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/