David Rientjes wrote: > Fix build failures on sparc: > > In file included from include/linux/mm.h:39, > from include/linux/memcontrol.h:24, > from include/linux/swap.h:8, > from include/linux/suspend.h:7, > from init/do_mounts.c:6: > include/asm/pgtable.h:344: warning: parameter names (without > types) in function declaration > include/asm/pgtable.h:345: warning: parameter names (without > types) in function declaration > include/asm/pgtable.h:346: error: expected '=', ',', ';', 'asm' or > '__attribute__' before '___f___swp_entry' > > and > > arch/sparc/kernel/led.c: In function 'led_blink': > arch/sparc/kernel/led.c:35: error: invalid use of undefined type > 'struct timer_list' > arch/sparc/kernel/led.c:35: error: 'jiffies' undeclared (first use > in this function) > arch/sparc/kernel/led.c:35: error: (Each undeclared identifier is > reported only once > arch/sparc/kernel/led.c:35: error: for each function it appears > in.) > arch/sparc/kernel/led.c:36: error: 'avenrun' undeclared (first use > in this function) > arch/sparc/kernel/led.c:36: error: 'FSHIFT' undeclared (first use > in this function) > arch/sparc/kernel/led.c:36: error: 'HZ' undeclared (first use in > this function) > arch/sparc/kernel/led.c:37: error: invalid use of undefined type > 'struct timer_list' > arch/sparc/kernel/led.c:39: error: invalid use of undefined type > 'struct timer_list' > arch/sparc/kernel/led.c:40: error: invalid use of undefined type > 'struct timer_list' > arch/sparc/kernel/led.c:42: error: implicit declaration of > function 'add_timer' > arch/sparc/kernel/led.c: In function 'led_write_proc': > arch/sparc/kernel/led.c:70: error: implicit declaration of > function 'copy_from_user' > arch/sparc/kernel/led.c:84: error: implicit declaration of > function 'del_timer_sync' > arch/sparc/kernel/led.c: In function 'led_init': > arch/sparc/kernel/led.c:109: error: implicit declaration of > function 'init_timer' > arch/sparc/kernel/led.c:110: error: invalid use of undefined type > 'struct timer_list' > > Cc: Adrian Bunk <bunk@xxxxxxxxxx> > Cc: Robert Reif <reif@xxxxxxxxxxxxx> > Signed-off-by: David Rientjes <rientjes@xxxxxxxxxx> > --- > arch/sparc/kernel/led.c | 3 ++- > include/linux/memcontrol.h | 3 --- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/arch/sparc/kernel/led.c b/arch/sparc/kernel/led.c > --- a/arch/sparc/kernel/led.c > +++ b/arch/sparc/kernel/led.c > @@ -1,8 +1,9 @@ > -#include <linux/kernel.h> > #include <linux/module.h> > #include <linux/init.h> > #include <linux/proc_fs.h> > #include <linux/string.h> > +#include <linux/sched.h> > +#include <linux/uaccess.h> > > #include <asm/auxio.h> > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -20,9 +20,6 @@ > #ifndef _LINUX_MEMCONTROL_H > #define _LINUX_MEMCONTROL_H > > -#include <linux/rcupdate.h> > -#include <linux/mm.h> > - > struct mem_cgroup; > struct page_cgroup; > struct page; This patch allowed my sparc cross compiler to build beyond mm/memcontrol.o, but I faced a build issue at drivers/net/e1000e/netdev.c: In function 'e1000e_update_stats': drivers/net/e1000e/netdev.c:2583: error: unable to find a register to spill in class 'FP_REGS' drivers/net/e1000e/netdev.c:2583: error: this is the insn: (insn 66 2379 2236 2 drivers/net/e1000e/netdev.c:2459 (set (reg:DI 487 [ D.34076 ]) (mem/s/j:DI (plus:SI (reg/v/f:SI 11 %o3 [orig:492 adapter ] [492]) (reg:SI 7 %g7 [514])) [0 <variable>.stats.gprc+0 S8 A128])) 45 {*movdi_insn_sp32} (insn_list:REG_DEP_TRUE 65 (insn_list:REG_DEP_ANTI 32 (nil))) (expr_list:REG_EQUIV (mem/s/j:DI (plus:SI (reg/v/f:SI 11 %o3 [orig:492 adapter ] [492]) (reg:SI 7 %g7 [514])) [0 <variable>.stats.gprc+0 S8 A128]) (nil))) drivers/net/e1000e/netdev.c:2583: confused by earlier errors, bailing out make[3]: *** [drivers/net/e1000e/netdev.o] Error 1 make[2]: *** [drivers/net/e1000e] Error 2 make[1]: *** [drivers/net] Error 2 So for the problem reported Tested-by: Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> -- Warm Regards, Balbir Singh Linux Technology Center IBM, ISTL - To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html