[gentoo-commits] proj/sandbox:master commit in: libsandbox/trace/linux/, libsandbox/, /

[gentoo-commits] proj/sandbox:master commit in: libsandbox/trace/linux/, libsandbox/, /

[Mike Frysinger]

commit:     46fe624223cfe62fb6c2fbb609be42f2f1d1734b
Author:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
AuthorDate: Sun Sep 20 08:51:41 2015 +0000
Commit:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
CommitDate: Sun Sep 20 08:51:41 2015 +0000
URL:        https://gitweb.gentoo.org/proj/sandbox.git/commit/?id=46fe6242

libsandbox: rework abi syscall header generation

Probe the availability of multilib headers at configure time so that we
can show the status more cleanly.  This allows the header generation to
be done in parallel and not output confusing warning messages to users.

URL: https://bugs.gentoo.org/536582
Reported-by: cmue81 <AT> gmx.de
Signed-off-by: Mike Frysinger <vapier <AT> gentoo.org>

 configure.ac                    | 44 ++++++++++++++++++++++++++++++++---------
 libsandbox/Makefile.am          | 19 +++++++++++-------
 libsandbox/trace/linux/x86_64.c |  6 ++++++
 3 files changed, 53 insertions(+), 16 deletions(-)

diff --git a/configure.ac b/configure.ac
index dec9686..b57263e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -50,19 +50,45 @@ dnl multiple personality support (x86 & x86_64: multilib)
 AC_MSG_CHECKING([for multiple personalities])
 AC_ARG_ENABLE([schizo],
 	[AS_HELP_STRING([--enable-schizo],[Support multiple personalities])],
-	[],[enable_schizo="yes"])
-SB_SCHIZO_SETTINGS="no"
-if test "x$enable_schizo" = "xyes" ; then
-	case $host_alias in
-		x86_64*linux*) SB_SCHIZO_SETTINGS="x86_64:-m64 x86:-m32 x32:-mx32";;
+	[],[enable_schizo="auto"])
+AC_MSG_RESULT([$enable_schizo])
+SB_SCHIZO_SETTINGS=
+AC_DEFUN([SB_CHECK_SCHIZO],[dnl
+		AC_MSG_CHECKING([checking for $1/$2 compiler support])
+		ac_save_CFLAGS=$CFLAGS
+		CFLAGS="$CFLAGS $2"
+		AC_TRY_COMPILE([
+			#include <stdio.h>
+		], [
+			return 0
+		], [
+			enable_schizo=yes
+			AS_VAR_APPEND([SB_SCHIZO_SETTINGS], " $1:$2")
+			AS_VAR_APPEND([SB_SCHIZO_HEADERS], " trace_syscalls_$1.h")
+			AC_MSG_RESULT([yes])
+			AC_DEFINE_UNQUOTED([SB_SCHIZO_$1], 1, [Support for $1/$2 is available])
+		], [
+			AC_MSG_RESULT([no])
+		])
+		CFLAGS=$ac_save_CFLAGS
+])
+if test "x$enable_schizo" != "xno" ; then
+	enable_schizo=no
+	case $host in
+	i686*linux|x86_64*linux*)
+		SB_CHECK_SCHIZO([x86_64], [-m64])
+		SB_CHECK_SCHIZO([x86], [-m32])
+		SB_CHECK_SCHIZO([x32], [-mx32])
+		;;
 	esac
-fi
-if test "$SB_SCHIZO_SETTINGS" != "no" ; then
-	AC_DEFINE_UNQUOTED([SB_SCHIZO], ["$SB_SCHIZO_SETTINGS"], [Enable multiple personalities support])
+	SB_SCHIZO_SETTINGS=${SB_SCHIZO_SETTINGS# }
+	if test "x$enable_schizo" != "xno" ; then
+		AC_DEFINE_UNQUOTED([SB_SCHIZO], ["$SB_SCHIZO_SETTINGS"], [Enable multiple personalities support])
+	fi
 fi
 AC_SUBST(SB_SCHIZO_SETTINGS)
+AC_SUBST(SB_SCHIZO_HEADERS)
 AM_CONDITIONAL([SB_SCHIZO], test "$SB_SCHIZO_SETTINGS" != "no")
-AC_MSG_RESULT($SB_SCHIZO_SETTINGS)
 
 dnl this test fills up the stack and then triggers a segfault ...
 dnl but it's hard to wrap things without a stack, so let's ignore

diff --git a/libsandbox/Makefile.am b/libsandbox/Makefile.am
index 529d835..cbc73ba 100644
--- a/libsandbox/Makefile.am
+++ b/libsandbox/Makefile.am
@@ -70,18 +70,23 @@ TRACE_MAKE_HEADER = \
 	$(SB_AWK) $(GEN_TRACE_SCRIPT) -v MODE=gen | \
 		$(COMPILE) -E -P -include $(top_srcdir)/headers.h - $$f | \
 		$(SB_AWK) $(GEN_TRACE_SCRIPT) -v syscall_prefix=$$t > $$header
-trace_syscalls.h: $(GEN_TRACE_SCRIPT) Makefile
+trace_syscalls.h: $(GEN_TRACE_SCRIPT) $(SB_SCHIZO_HEADERS)
 if SB_SCHIZO
+	$(AM_V_GEN)touch $@
+else
+	$(AM_V_GEN)t= f= header=$@; $(TRACE_MAKE_HEADER)
+endif
+
+$(SB_SCHIZO_HEADERS): $(GEN_TRACE_SCRIPT)
 	$(AM_V_GEN)for pers in $(SB_SCHIZO_SETTINGS) ; do \
 		t=_$${pers%:*}; \
 		f=$${pers#*:}; \
-		header=trace_syscalls$${t}.h; \
-		$(TRACE_MAKE_HEADER) || exit $$?; \
+		header="trace_syscalls$${t}.h"; \
+		if [ "$$header" = "$@" ]; then \
+			$(TRACE_MAKE_HEADER) || exit $$?; \
+			break; \
+		fi; \
 	done
-	@touch $@
-else
-	$(AM_V_GEN)t= f= header=$@; $(TRACE_MAKE_HEADER)
-endif
 
 EXTRA_DIST = $(SYMBOLS_FILE) $(SYMBOLS_WRAPPERS) $(SB_NR_FILE) $(TRACE_FILES) headers.h
 

diff --git a/libsandbox/trace/linux/x86_64.c b/libsandbox/trace/linux/x86_64.c
index 5bd1361..82c492d 100644
--- a/libsandbox/trace/linux/x86_64.c
+++ b/libsandbox/trace/linux/x86_64.c
@@ -4,21 +4,27 @@
 #ifdef SB_SCHIZO
 
 static const struct syscall_entry syscall_table_32[] = {
+#ifdef SB_SCHIZO_x86
 #define S(s) { SB_SYS_x86_##s, SB_NR_##s, #s },
 #include "trace_syscalls_x86.h"
 #undef S
+#endif
 	{ SB_NR_UNDEF, SB_NR_UNDEF, NULL },
 };
 static const struct syscall_entry syscall_table_64[] = {
+#ifdef SB_SCHIZO_x86_64
 #define S(s) { SB_SYS_x86_64_##s, SB_NR_##s, #s },
 #include "trace_syscalls_x86_64.h"
 #undef S
+#endif
 	{ SB_NR_UNDEF, SB_NR_UNDEF, NULL },
 };
 static const struct syscall_entry syscall_table_x32[] = {
+#ifdef SB_SCHIZO_x32
 #define S(s) { SB_SYS_x32_##s, SB_NR_##s, #s },
 #include "trace_syscalls_x32.h"
 #undef S
+#endif
 	{ SB_NR_UNDEF, SB_NR_UNDEF, NULL },
 };