From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51436) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yb8TQ-0003z7-ND for qemu-devel@nongnu.org; Thu, 26 Mar 2015 10:09:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yb8TM-000856-Fi for qemu-devel@nongnu.org; Thu, 26 Mar 2015 10:09:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39141) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yb8TM-000851-8B for qemu-devel@nongnu.org; Thu, 26 Mar 2015 10:09:24 -0400 From: Juan Quintela In-Reply-To: <87oanfx4i8.fsf@neno.neno> (Juan Quintela's message of "Thu, 26 Mar 2015 14:38:55 +0100") References: <1427279169-3213-1-git-send-email-eduardo.otubo@profitbricks.com> <1427279169-3213-2-git-send-email-eduardo.otubo@profitbricks.com> <87oanfx4i8.fsf@neno.neno> Date: Thu, 26 Mar 2015 15:09:19 +0100 Message-ID: <87fv8rx33k.fsf@neno.neno> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PULL for v2.3 01/01] seccomp: update libseccomp version and remove arch restriction Reply-To: quintela@redhat.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Otubo Cc: pmoore@redhat.com, peter.maydell@linaro.org, qemu-devel@nongnu.org Juan Quintela wrote: > Eduardo Otubo wrote: >> Libseccomp version updated to 2.2.0 and arch restriction to x86/x86_64 >> is now removed. It's supposed to work on armv7l as well. >> >> Related bug: https://bugs.launchpad.net/qemu/+bug/1363641 >> >> Signed-off-by: Eduardo Otubo Just in case anyone is interested, default virt-test output with current qemu 15:05:01 ERROR| Qemu output: 15:05:01 ERROR| qemu-system-x86_64: -sandbox on: sandboxing request but seccomp is not compiled into this build 15:05:01 ERROR| 15:05:01 ERROR| 15:05:01 ERROR| FAIL type_specific.io-github-autotest-qemu.migrate.default.tcp -> VMStartError: VM 'virt-tests-vm1' failed to start: Qemu is defunct. Qemu output: qemu-system-x86_64: -sandbox on: sandboxing request but seccomp is not compiled into this build Notice that you can "fix" the problem running with --qemu_sandbox=off, but breaking the main test suite just before release looks wrong? Later, Juan. > > This prevent compliation on x86_64 on Fedora 21. > > (migration/next)$ make -j5 -C /scratch/tmp/next/all/ > make: Entering directory '/scratch/tmp/next/all' > config-host.mak is out-of-date, running configure > > ERROR: User requested feature libseccomp > configure was not able to find it. > Install libseccomp devel >= 2.2.0 > > Makefile:30: recipe for target 'config-host.mak' failed > make: *** [config-host.mak] Error 1 > make: Leaving directory '/scratch/tmp/next/all' > (migration/next)$ rpm -qa | grep seccomp > libseccomp-2.1.1-5.fc21.x86_64 > libseccomp-devel-2.1.1-5.fc21.x86_64 > libseccomp-debuginfo-2.1.1-5.fc21.x86_64 > (migration/next)$ > > This was compiling correctly until this patch got in. And virt-test > uses seccomp by default. > > Fedora 21 is less than 3 months old. Do we really want to avoid > compilation there? > > Thanks, Juan. > > >> --- >> configure | 5 ++--- >> 1 file changed, 2 insertions(+), 3 deletions(-) >> >> diff --git a/configure b/configure >> index 589798e..cbe6495 100755 >> --- a/configure >> +++ b/configure >> @@ -1848,14 +1848,13 @@ fi >> # libseccomp check >> >> if test "$seccomp" != "no" ; then >> - if test "$cpu" = "i386" || test "$cpu" = "x86_64" && >> - $pkg_config --atleast-version=2.1.1 libseccomp; then >> + if $pkg_config --atleast-version=2.2.0 libseccomp; then >> libs_softmmu="$libs_softmmu `$pkg_config --libs libseccomp`" >> QEMU_CFLAGS="$QEMU_CFLAGS `$pkg_config --cflags libseccomp`" >> seccomp="yes" >> else >> if test "$seccomp" = "yes"; then >> - feature_not_found "libseccomp" "Install libseccomp devel >= 2.1.1" >> + feature_not_found "libseccomp" "Install libseccomp devel >= 2.2.0" >> fi >> seccomp="no" >> fi