Hi Atsushi, On 5/10/07, Atsushi Nemoto <anemo@xxxxxxxxxxxxx> wrote:
On 64-bit MIPS, only N64 ABI is checked by default. This patch adds some rules for other ABIs. This results in these warnings at the moment:
nice to see this second version which is nicer IMHO.
CALL-N32 /home/git/linux-mips/scripts/checksyscalls.sh <stdin>:148:2: warning: #warning syscall time not implemented <stdin>:424:2: warning: #warning syscall select not implemented <stdin>:440:2: warning: #warning syscall uselib not implemented <stdin>:856:2: warning: #warning syscall vfork not implemented <stdin>:868:2: warning: #warning syscall truncate64 not implemented <stdin>:872:2: warning: #warning syscall ftruncate64 not implemented <stdin>:876:2: warning: #warning syscall stat64 not implemented <stdin>:880:2: warning: #warning syscall lstat64 not implemented <stdin>:884:2: warning: #warning syscall fstat64 not implemented <stdin>:980:2: warning: #warning syscall getdents64 not implemented <stdin>:1176:2: warning: #warning syscall fadvise64_64 not implemented <stdin>:1284:2: warning: #warning syscall fstatat64 not implemented <stdin>:1364:2: warning: #warning syscall utimensat not implemented CALL-O32 /home/git/linux-mips/scripts/checksyscalls.sh <stdin>:424:2: warning: #warning syscall select not implemented <stdin>:856:2: warning: #warning syscall vfork not implemented <stdin>:1176:2: warning: #warning syscall fadvise64_64 not implemented <stdin>:1364:2: warning: #warning syscall utimensat not implemented CALL /home/git/linux-mips/scripts/checksyscalls.sh <stdin>:148:2: warning: #warning syscall time not implemented <stdin>:424:2: warning: #warning syscall select not implemented <stdin>:440:2: warning: #warning syscall uselib not implemented <stdin>:856:2: warning: #warning syscall vfork not implemented <stdin>:980:2: warning: #warning syscall getdents64 not implemented <stdin>:1364:2: warning: #warning syscall utimensat not implemented
woah, quite a lot of works are waiting for you ;)
Signed-off-by: Atsushi Nemoto <anemo@xxxxxxxxxxxxx> --- diff --git a/arch/mips/Makefile b/arch/mips/Makefile index a68d462..f450066 100644 --- a/arch/mips/Makefile +++ b/arch/mips/Makefile @@ -709,3 +709,25 @@ archclean: CLEAN_FILES += vmlinux.32 \ vmlinux.64 \ vmlinux.ecoff + +quiet_cmd_syscalls_n32 = CALL-N32 $< + cmd_syscalls_n32 = $(CONFIG_SHELL) $< $(CC) $(c_flags) -mabi=n32 + +quiet_cmd_syscalls_o32 = CALL-O32 $< + cmd_syscalls_o32 = $(CONFIG_SHELL) $< $(CC) $(c_flags) -mabi=32 + +PHONY += missing-syscalls-n32 missing-syscalls-o32 + +missing-syscalls-n32: scripts/checksyscalls.sh FORCE + $(call cmd,syscalls_n32) + +missing-syscalls-o32: scripts/checksyscalls.sh FORCE + $(call cmd,syscalls_o32) + +archprepare:
I didn't know about that rule. However I'm not sure it's a good idea to add a set of commands to it since it's multiple rule...
+ifdef CONFIG_MIPS32_N32 + $(Q)$(MAKE) $(build)=arch/mips missing-syscalls-n32
or can't we do instead: $(Q)$(MAKE) $(build)=. missing-syscalls EXTRA_CFLAGS="-mabi=n32" and get rid of "missing-syscalls-n32" rule. Thus this avoids to duplicate "missing-syscalls" command.
+endif +ifdef CONFIG_MIPS32_O32 + $(Q)$(MAKE) $(build)=arch/mips missing-syscalls-o32
ditto: $(Q)$(MAKE) $(build)=. missing-syscalls EXTRA_CFLAGS="-mabi=32"
+endif
I'm wondering if this should stay at the end where cleaning rules live... Anyways thanks. -- Franck