linux-kernel - RE: [PATCH 1/3] kbuild: complain about missing system calls.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <617E1C2C70743745A92448908E030B2A01719AFB@scsmsx411.amr.corp.intel.com>
Date:	Tue, 8 May 2007 11:35:27 -0700
From:	"Luck, Tony" <tony.luck@...el.com>
To:	"Sam Ravnborg" <sam@...nborg.org>
CC:	"David Woodhouse" <dwmw2@...radead.org>,
	"Andrew Morton" <akpm@...ux-foundation.org>,
	"Kees Cook" <kees@...flux.net>,
	"Randy Dunlap" <randy.dunlap@...cle.com>,
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 1/3] kbuild: complain about missing system calls.

Looks more complex than necessary.  Why do you want to execute
an arch specific script to list the __IGNORE symbols?  That would
allow an arch to generate a list with sed/perl/etc. but that
looks like overkill.

If you just have an arch specific file with the right defines.
E.g. for x86_64 in include/asm-x86_64/ignore_syscalls:

#include __IGNORE_getcpu

Then the checksyscalls.sh changes would just be something like
(do I need a ${srctree} before the include??):

diff --git a/scripts/checksyscalls.sh b/scripts/checksyscalls.sh
index f98171f..616f7c5 100755
--- a/scripts/checksyscalls.sh
+++ b/scripts/checksyscalls.sh
@@ -105,6 +105,10 @@ cat << EOF
 #define __IGNORE_putpmsg
 #define __IGNORE_vserver
 EOF
+if [ -f include/asm/ignore_syscalls ]
+then
+	cat include/asm/ignore_syscalls
+fi
 }
 
 syscall_list() {

-Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/