From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51280) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XmQ2f-0000yb-3U for qemu-devel@nongnu.org; Thu, 06 Nov 2014 11:36:18 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XmQ2Z-0004MX-ST for qemu-devel@nongnu.org; Thu, 06 Nov 2014 11:36:13 -0500 Received: from mail-wg0-f50.google.com ([74.125.82.50]:64165) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XmQ2Z-0004M4-NT for qemu-devel@nongnu.org; Thu, 06 Nov 2014 11:36:07 -0500 Received: by mail-wg0-f50.google.com with SMTP id z12so1645501wgg.23 for ; Thu, 06 Nov 2014 08:36:07 -0800 (PST) Date: Thu, 6 Nov 2014 17:36:04 +0100 From: Eduardo Otubo Message-ID: <20141106163601.GB8274@vader> References: <1415285358-7446-1-git-send-email-eduardo.otubo@profitbricks.com> <2160380.fdzADCfzWy@sifl> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <2160380.fdzADCfzWy@sifl> Subject: Re: [Qemu-devel] [PATCH] seccomp: change configure to avoid arm 32 to break List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paul Moore Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org, philipp.gesang@intra2net.com On Thu, Nov 06, 2014 at 11:22:16AM -0500, Paul Moore wrote: > On Thursday, November 06, 2014 03:49:18 PM Eduardo Otubo wrote: > > Right now seccomp is breaking the compilation of Qemu on armv7l due > > to libsecomp current lack of support for this arch. This problem is > > already fixed on libseccomp upstream but no release date for that is > > scheduled to far. This patch disables support for seccomp on armv7l > > temporarily until libseccomp does a new release. Then I'll remove the > > hack and update libseccomp dependency on configure script. > > > > Related bug: https://bugs.launchpad.net/qemu/+bug/1363641 > > > > Signed-off-by: Eduardo Otubo > > --- > > configure | 20 +++++++++++--------- > > 1 file changed, 11 insertions(+), 9 deletions(-) > > > > diff --git a/configure b/configure > > index 2f17bf3..16fd7f5 100755 > > --- a/configure > > +++ b/configure > > @@ -1823,15 +1823,17 @@ fi > > # libseccomp check > > > > if test "$seccomp" != "no" ; then > > - if $pkg_config --atleast-version=2.1.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.0" - fi > > - seccomp="no" > > + if test "$cpu" = "i386" || test "$cpu" = "x86_64"; then > > + if $pkg_config --atleast-version=2.1.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.0" + fi > > + seccomp="no" > > + fi > > fi > > fi > > ########################################## > > Also, note the current release of libseccomp is v2.1.1 which has a number of > bug fixes on top of v2.1.0. > Does that applies to the distros package version? I'm running Ubuntu 14.04 and it's still 2.1.0. A regular user would have to download and install from scratch in order to build Qemu, then. -- Eduardo Otubo ProfitBricks GmbH