From nobody Mon May 6 01:49:22 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=1; a=rsa-sha256; t=1636042322; cv=none; d=zohomail.com; s=zohoarc; b=cmlLL1x4mEFiy8XCFwlIOE2nXX8RDJjy3XsPVuJ1/m3pO5/W5NkOwIILNA8Fnl4/JYXoLBiqapNFqzrInINXxBB/URVK7dSsg2JNubzxnCWkDlgXu/js+0jASuG8c6zjhsExvU3kngYrwfB+Vj86ONHUCYODFhl3R65tkaRFQ3w= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1636042322; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=HepRV8+VsNCo8NAgipaKy04UoWqLrUAfUhmyvWOwJkY=; b=CUw1fv/R3gI+Cjhz26qLUmBXjxe7kN3SpdxBd73FJAFZmlh5zcfL4VgcY0zF8OKQhsJ6G8MHW7VYf2tca3GGECvfuR1Rj62/yuNQYqMdpw6nOVCQA85M2hytI1C8UWAKMqYFZIFpiK44b5mY89LEWXjHGNaBbTLrCw87jnl4WuE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1636042322378796.9064103132872; Thu, 4 Nov 2021 09:12:02 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.221687.383531 (Exim 4.92) (envelope-from ) id 1mifKq-00087C-Iw; Thu, 04 Nov 2021 16:11:28 +0000 Received: by outflank-mailman (output) from mailman id 221687.383531; Thu, 04 Nov 2021 16:11:28 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mifKq-00086p-F8; Thu, 04 Nov 2021 16:11:28 +0000 Received: by outflank-mailman (input) for mailman id 221687; Thu, 04 Nov 2021 16:11:26 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mifKo-0007c5-Hv for xen-devel@lists.xenproject.org; Thu, 04 Nov 2021 16:11:26 +0000 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id dba8dc0b-3d89-11ec-a9d2-d9f7a1cc8784; Thu, 04 Nov 2021 17:11:24 +0100 (CET) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 92FC01FD3F; Thu, 4 Nov 2021 16:11:24 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 6B01313EEA; Thu, 4 Nov 2021 16:11:24 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id 8Mv9GCwGhGEABgAAMHmgww (envelope-from ); Thu, 04 Nov 2021 16:11:24 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: dba8dc0b-3d89-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1636042284; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=HepRV8+VsNCo8NAgipaKy04UoWqLrUAfUhmyvWOwJkY=; b=CTifw9LtiYIimJf50J1IFq4HheblJMNfpPGpztKClkLgV4vCEtf8bo2yI50OYDqk7DyEwx v5WiYfRZU3PjTHilcf10qY2GNttgHNNZEnwlBbeIw9NJkvvTLwTF5lqaMGkj502kk4ZmTq gxNRr68DJHKTovliSz5vaxfNJP8onig= From: Juergen Gross To: xen-devel@lists.xenproject.org Cc: Juergen Gross , Ian Jackson , Wei Liu Subject: [PATCH-for-4.16 v2 1/2] configure: modify default of building rombios Date: Thu, 4 Nov 2021 17:11:20 +0100 Message-Id: <20211104161121.18995-2-jgross@suse.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20211104161121.18995-1-jgross@suse.com> References: <20211104161121.18995-1-jgross@suse.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1636042324554100001 Content-Type: text/plain; charset="utf-8" The tools/configure script will default to build rombios if qemu traditional is enabled. If rombios is being built, ipxe will be built per default, too. This results in rombios and ipxe no longer being built by default when disabling qemu traditional. Fix that be rearranging the dependencies: - build ipxe by default - build rombios by default if either ipxe or qemu traditional are being built This modification prepares not building qemu traditional by default without affecting build of rombios and ipxe. Signed-off-by: Juergen Gross Reviewed-by: Ian Jackson --- V2: - correct help text (Andrew Cooper) - correct commit message (Andrew Cooper) --- tools/configure | 115 +++++++++++++++++++++------------------------ tools/configure.ac | 67 +++++++++++++------------- 2 files changed, 85 insertions(+), 97 deletions(-) diff --git a/tools/configure b/tools/configure index 33814b24b3..d980b3ffc9 100755 --- a/tools/configure +++ b/tools/configure @@ -698,7 +698,6 @@ APPEND_INCLUDES PREPEND_LIB PREPEND_INCLUDES EXTRA_QEMUU_CONFIGURE_ARGS -ipxe qemu_xen_systemd qemu_xen_path qemu_xen @@ -706,6 +705,7 @@ rombios BCC LD86 AS86 +ipxe qemu_traditional LINUX_BACKEND_MODULES golang @@ -815,13 +815,13 @@ enable_seabios enable_golang with_linux_backend_modules enable_qemu_traditional +enable_ipxe +with_system_ipxe enable_rombios with_system_qemu with_stubdom_qmp_proxy with_system_seabios with_system_ovmf -enable_ipxe -with_system_ipxe with_extra_qemuu_configure_args with_xenstored enable_systemd @@ -1504,10 +1504,10 @@ Optional Features: --enable-qemu-traditional Enable qemu traditional device model, (DEFAULT i= s on for Linux or NetBSD x86, otherwise off) + --enable-ipxe Enable in-tree IPXE, (DEFAULT is on for x86, + otherwise off, see also --with-system-ipxe) --enable-rombios Enable ROMBIOS, (DEFAULT is on if qemu-tradition= al - is enabled, otherwise off) - --disable-ipxe Enable in-tree IPXE, (DEFAULT is on if rombios is - enabled, otherwise off, see also --with-system-i= pxe) + or ipxe is enabled, otherwise off) --enable-systemd Enable systemd support (default is DISABLED) --enable-9pfs Explicitly enable 9pfs support in QEMU build (default is to defer to QEMU configure default) @@ -1538,6 +1538,11 @@ Optional Packages: --with-linux-backend-modules=3D"mod1 mod2" List of Linux backend module or modalias names t= o be autoloaded on startup. + --with-system-ipxe[=3DPATH] + Use system supplied IPXE PATH instead of building + and installing our own version, it takes precede= nce + over --{en,dis}able-ipxe, --without-system-ipxe = is + an error --with-system-qemu[=3DPATH] Use system supplied qemu PATH or qemu (taken from $PATH) as qemu-xen device model instead of build= ing @@ -1551,12 +1556,6 @@ Optional Packages: --with-system-ovmf[=3DPATH] Use system supplied OVMF PATH instead of building and installing our own version - --with-system-ipxe[=3DPATH] - Use system supplied IPXE PATH instead of building - and installing our own version, it takes precede= nce - over --{en,dis}able-ipxe and is bound by the - presence of rombios, --without-system-ipxe is an - error --with-extra-qemuu-configure-args[=3D"--ARG1 ..."] List of additional configure options for upstream qemu @@ -4314,13 +4313,54 @@ else =20 fi =20 +if test "x$enable_ipxe" =3D "xno"; then : + ipxe=3Dn +else + ipxe=3Dy +fi + +# Check whether --enable-ipxe was given. +if test "${enable_ipxe+set}" =3D set; then : + enableval=3D$enable_ipxe; +else + + case "$host_cpu" in + i[3456]86|x86_64) + enable_ipxe=3D"yes";; + *) enable_ipxe=3D"no";; + esac + +fi + + +# Check whether --with-system-ipxe was given. +if test "${with_system_ipxe+set}" =3D set; then : + withval=3D$with_system_ipxe; + case $withval in + no) as_fn_error $? "--without-system-ipxe has no effect" "$LINENO"= 5 ;; + /*) ipxe_path=3D$withval; ipxe=3Dn ;; + *) as_fn_error $? "IPXE specified, but is not an absolute path" "$= LINENO" 5 ;; + esac + +fi + +if test "x$ipxe" =3D "xy" -o -n "$ipxe_path" ; then : + + +cat >>confdefs.h <<_ACEOF +#define IPXE_PATH "${ipxe_path:-$XENFIRMWAREDIR/ipxe.bin}" +_ACEOF + + +fi + =20 # Check whether --enable-rombios was given. if test "${enable_rombios+set}" =3D set; then : enableval=3D$enable_rombios; else =20 - if test "x$enable_qemu_traditional" =3D "xyes"; then : + if test "x$enable_qemu_traditional" =3D "xyes" -o "x$enable_ipxe" =3D = "xyes"; then : =20 enable_rombios=3D"yes" =20 @@ -4635,55 +4675,6 @@ _ACEOF =20 fi =20 -# Check whether --enable-ipxe was given. -if test "${enable_ipxe+set}" =3D set; then : - enableval=3D$enable_ipxe; - if test "x$enable_ipxe" =3D "xno"; then : - ipxe=3Dn -else - ipxe=3Dy -fi - -else - - if test "x$enable_rombios" =3D "xno"; then : - ipxe=3Dn -else - ipxe=3Dy -fi - -fi - - -# Check whether --with-system-ipxe was given. -if test "${with_system_ipxe+set}" =3D set; then : - withval=3D$with_system_ipxe; - case $withval in - no) as_fn_error $? "--without-system-ipxe has no effect" "$LINENO"= 5 ;; - /*) ipxe_path=3D$withval; ipxe=3Dn ;; - *) as_fn_error $? "IPXE specified, but is not an absolute path" "$= LINENO" 5 ;; - esac - -fi - -if test "x$ipxe" =3D "xy" -o -n "$ipxe_path" ; then : - - - if test "x$enable_rombios" =3D "xno"; then : - - as_fn_error $? "Rombios is required to use IPXE" "$LINENO" 5 - -fi - - -cat >>confdefs.h <<_ACEOF -#define IPXE_PATH "${ipxe_path:-$XENFIRMWAREDIR/ipxe.bin}" -_ACEOF - - -fi - - =20 # Check whether --with-extra-qemuu-configure-args was given. if test "${with_extra_qemuu_configure_args+set}" =3D set; then : diff --git a/tools/configure.ac b/tools/configure.ac index 6414fcbb44..97582951c8 100644 --- a/tools/configure.ac +++ b/tools/configure.ac @@ -139,10 +139,40 @@ AC_DEFINE([HAVE_QEMU_TRADITIONAL], [1], [Qemu traditi= onal enabled]) ]) AC_SUBST(qemu_traditional) =20 +AC_ARG_ENABLE([ipxe], + AS_HELP_STRING([--enable-ipxe], + [Enable in-tree IPXE, (DEFAULT is on for x86, + otherwise off, see also --with-system-ipxe)]),,[ + case "$host_cpu" in + i[[3456]]86|x86_64) + enable_ipxe=3D"yes";; + *) enable_ipxe=3D"no";; + esac +]) +AS_IF([test "x$enable_ipxe" =3D "xno"], [ipxe=3Dn], [ipxe=3Dy]) +AC_ARG_WITH([system-ipxe], + AS_HELP_STRING([--with-system-ipxe@<:@=3DPATH@:>@], + [Use system supplied IPXE PATH instead of building and installing + our own version, it takes precedence over --{en,dis}able-ipxe, + --without-system-ipxe is an error]),[ + case $withval in + no) AC_MSG_ERROR([--without-system-ipxe has no effect]) ;; + /*) ipxe_path=3D$withval; ipxe=3Dn ;; + *) AC_MSG_ERROR([IPXE specified, but is not an absolute path]) ;; + esac +],[]) +AS_IF([test "x$ipxe" =3D "xy" -o -n "$ipxe_path" ], [ + AC_DEFINE_UNQUOTED([IPXE_PATH], + ["${ipxe_path:-$XENFIRMWAREDIR/ipxe.bin}"], + [IPXE path]) +]) +AC_SUBST(ipxe) + AC_ARG_ENABLE([rombios], AS_HELP_STRING([--enable-rombios], - [Enable ROMBIOS, (DEFAULT is on if qemu-traditional is = enabled, otherwise off)]),,[ - AS_IF([test "x$enable_qemu_traditional" =3D "xyes"], [ + [Enable ROMBIOS, (DEFAULT is on if qemu-traditional or = ipxe is enabled, + otherwise off)]),,[ + AS_IF([test "x$enable_qemu_traditional" =3D "xyes" -o "x$enable_ipxe" = =3D "xyes"], [ enable_rombios=3D"yes" ], [ enable_rombios=3D"no" @@ -240,39 +270,6 @@ AS_IF([test "x$ovmf" =3D "xy" -o -n "$ovmf_path" ], [ [OVMF path]) ]) =20 -AC_ARG_ENABLE([ipxe], - AS_HELP_STRING([--disable-ipxe], - [Enable in-tree IPXE, (DEFAULT is on if rombios is enab= led, - otherwise off, see also --with-system-ipxe)]), - [ - AS_IF([test "x$enable_ipxe" =3D "xno"], [ipxe=3Dn], [ipxe=3Dy]) - ], - [ - AS_IF([test "x$enable_rombios" =3D "xno"], [ipxe=3Dn], [ipxe=3Dy]) -]) -AC_ARG_WITH([system-ipxe], - AS_HELP_STRING([--with-system-ipxe@<:@=3DPATH@:>@], - [Use system supplied IPXE PATH instead of building and installing - our own version, it takes precedence over --{en,dis}able-ipxe and = is - bound by the presence of rombios, --without-system-ipxe is an erro= r]),[ - case $withval in - no) AC_MSG_ERROR([--without-system-ipxe has no effect]) ;; - /*) ipxe_path=3D$withval; ipxe=3Dn ;; - *) AC_MSG_ERROR([IPXE specified, but is not an absolute path]) ;; - esac -],[]) -AS_IF([test "x$ipxe" =3D "xy" -o -n "$ipxe_path" ], [ - - AS_IF([test "x$enable_rombios" =3D "xno"], [ - AC_MSG_ERROR([Rombios is required to use IPXE]) - ], []) - - AC_DEFINE_UNQUOTED([IPXE_PATH], - ["${ipxe_path:-$XENFIRMWAREDIR/ipxe.bin}"], - [IPXE path]) -]) -AC_SUBST(ipxe) - AC_ARG_WITH([extra-qemuu-configure-args], AS_HELP_STRING([--with-extra-qemuu-configure-args@<:@=3D"--ARG1 ..."@:= >@], [List of additional configure options for upstream qemu]),[ --=20 2.26.2 From nobody Mon May 6 01:49:22 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=1; a=rsa-sha256; t=1636042328; cv=none; d=zohomail.com; s=zohoarc; b=EYqiY4NPSnJdUXxlH+Z1bGv2qTAtANbQnun1YOrLDRfJclpcJDPYWgrQVrscvUd3rb31MCq4i6yV+OyadCQVucWIICH3viYCHI3gI51f5QVJp4Ks+afDpPW/HN83bhmKzYjQU4r3csdMe1k25q2jDK8jRxbAjwTCK2UBleOWsss= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1636042328; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=uT7wIMCexC5sOksHtDS/6G5hw1BD1mbW59xbOMQ15QY=; b=U4G639VI9HY05mxd7H5KInV0QwuTcQdjzHBp5pmCJ7mdRHU5aYElmkhCQhrzI1UscQoFa2Ms4VPX922hOQUW3qNZO2iGR93ykHSRmYl/Q+Zk697iFxsaNMszCz6nz0jOY+/GTiLmf/fb5syNIeexwBGI3AzyKFMT9qZPfE5CkPw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1636042328043249.53076157475095; Thu, 4 Nov 2021 09:12:08 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.221686.383514 (Exim 4.92) (envelope-from ) id 1mifKp-0007fe-Cg; Thu, 04 Nov 2021 16:11:27 +0000 Received: by outflank-mailman (output) from mailman id 221686.383514; Thu, 04 Nov 2021 16:11:27 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mifKp-0007fE-73; Thu, 04 Nov 2021 16:11:27 +0000 Received: by outflank-mailman (input) for mailman id 221686; Thu, 04 Nov 2021 16:11:26 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mifKo-0007c6-5M for xen-devel@lists.xenproject.org; Thu, 04 Nov 2021 16:11:26 +0000 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id dbc9262b-3d89-11ec-9787-a32c541c8605; Thu, 04 Nov 2021 17:11:25 +0100 (CET) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id C946E1FD5B; Thu, 4 Nov 2021 16:11:24 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 9948B13DA2; Thu, 4 Nov 2021 16:11:24 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id SKwtJCwGhGEABgAAMHmgww (envelope-from ); Thu, 04 Nov 2021 16:11:24 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: dbc9262b-3d89-11ec-9787-a32c541c8605 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1636042284; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=uT7wIMCexC5sOksHtDS/6G5hw1BD1mbW59xbOMQ15QY=; b=XmQ1nttShV9z6E3QHHsrq0TS/8vUYESiP9Kcp73wVN71spyRY+a0B6bvZ6LrVu3n8f3Igr P5+Y/dW6LT969xS0yZnYPIETkK2cFUlncNv2+1JuZKMhSOI8gX8DNkXGC8jCaLC73CROTK 6OFyhm8kRGmc3clj2OvSsPyMQsJT53g= From: Juergen Gross To: xen-devel@lists.xenproject.org Cc: Juergen Gross , Ian Jackson , Community Manager , Samuel Thibault , Wei Liu Subject: [PATCH-for-4.16 v2 2/2] tools: disable building qemu-trad per default Date: Thu, 4 Nov 2021 17:11:21 +0100 Message-Id: <20211104161121.18995-3-jgross@suse.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20211104161121.18995-1-jgross@suse.com> References: <20211104161121.18995-1-jgross@suse.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1636042330284100001 Content-Type: text/plain; charset="utf-8" Using qemu-traditional as device model is deprecated for some time now. So change the default for building it to "disable". This will affect ioemu-stubdom, too, as there is a direct dependency between the two. Today it is possible to use a PVH/HVM Linux-based stubdom as device model. Additionally using ioemu-stubdom isn't really helping for security, as it requires to run a very old and potentially buggy qemu version in a PV domain. This is adding probably more security problems than it is removing by using a stubdom. Signed-off-by: Juergen Gross Reviewed-by: Samuel Thibault Acked-by: Ian Jackson --- This is a resend of my original patch after that has been reverted due to dependencies to ipxe and rombios. Those have been dealt with in the first patch of this series. --- CHANGELOG.md | 3 +++ stubdom/configure | 8 -------- stubdom/configure.ac | 8 +------- tools/configure | 17 ++--------------- tools/configure.ac | 13 +------------ 5 files changed, 7 insertions(+), 42 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e7107ac3de..e5ab49e779 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,9 @@ The format is based on [Keep a Changelog](https://keepach= angelog.com/en/1.0.0/) or by passing "iommu=3Dquarantine=3Dscratch-page" on the hypervisor com= mand line. - pv-grub stubdoms will no longer be built per default. In order to be ab= le to use pv-grub configure needs to be called with "--enable-pv-grub" as parameter. + - qemu-traditional based device models (both, qemu-traditional and ioemu-= stubdom) will + no longer be built per default. In order to be able to use those, confi= gure needs to + be called with "--enable-qemu-traditional" as parameter. =20 ## [4.15.0 UNRELEASED](https://xenbits.xen.org/gitweb/?p=3Dxen.git;a=3Dsho= rtlog;h=3DRELEASE-4.15.0) - TBD =20 diff --git a/stubdom/configure b/stubdom/configure index df31532abb..07b709f998 100755 --- a/stubdom/configure +++ b/stubdom/configure @@ -2286,14 +2286,6 @@ fi # Check whether --enable-qemu-traditional was given. if test "${enable_qemu_traditional+set}" =3D set; then : enableval=3D$enable_qemu_traditional; -else - - case "$host_cpu" in - i[3456]86|x86_64) - enable_qemu_traditional=3D"yes";; - *) enable_qemu_traditional=3D"no";; - esac - fi =20 if test "x$enable_qemu_traditional" =3D "xyes"; then : diff --git a/stubdom/configure.ac b/stubdom/configure.ac index a07a1edae5..e20d99edac 100644 --- a/stubdom/configure.ac +++ b/stubdom/configure.ac @@ -27,13 +27,7 @@ AX_STUBDOM_DEFAULT_ENABLE([xenstorepvh-stubdom], [xensto= repvh]) AX_STUBDOM_CONDITIONAL([vtpm-stubdom], [vtpm]) AX_STUBDOM_CONDITIONAL([vtpmmgr-stubdom], [vtpmmgr]) =20 -AC_ARG_ENABLE([qemu-traditional],,,[ - case "$host_cpu" in - i[[3456]]86|x86_64) - enable_qemu_traditional=3D"yes";; - *) enable_qemu_traditional=3D"no";; - esac -]) +AC_ARG_ENABLE([qemu-traditional]) AS_IF([test "x$enable_qemu_traditional" =3D "xyes"], [ qemu_traditional=3Dy],[ qemu_traditional=3Dn diff --git a/tools/configure b/tools/configure index d980b3ffc9..a3d33eb907 100755 --- a/tools/configure +++ b/tools/configure @@ -1502,8 +1502,8 @@ Optional Features: --disable-seabios Disable SeaBIOS (default is ENABLED) --disable-golang Disable Go tools (default is ENABLED) --enable-qemu-traditional - Enable qemu traditional device model, (DEFAULT i= s on - for Linux or NetBSD x86, otherwise off) + Enable qemu traditional device model, (DEFAULT is + off) --enable-ipxe Enable in-tree IPXE, (DEFAULT is on for x86, otherwise off, see also --with-system-ipxe) --enable-rombios Enable ROMBIOS, (DEFAULT is on if qemu-tradition= al @@ -4286,19 +4286,6 @@ LINUX_BACKEND_MODULES=3D"`eval echo $LINUX_BACKEND_M= ODULES`" # Check whether --enable-qemu-traditional was given. if test "${enable_qemu_traditional+set}" =3D set; then : enableval=3D$enable_qemu_traditional; -else - - case "$host_cpu" in - i[3456]86|x86_64) - enable_qemu_traditional=3D"yes";; - *) enable_qemu_traditional=3D"no";; - esac - case "$host_os" in - freebsd*) - enable_qemu_traditional=3D"no";; - esac - - fi =20 if test "x$enable_qemu_traditional" =3D "xyes"; then : diff --git a/tools/configure.ac b/tools/configure.ac index 97582951c8..03eb7cf146 100644 --- a/tools/configure.ac +++ b/tools/configure.ac @@ -120,18 +120,7 @@ AC_SUBST(LINUX_BACKEND_MODULES) =20 AC_ARG_ENABLE([qemu-traditional], AS_HELP_STRING([--enable-qemu-traditional], - [Enable qemu traditional device model, (DEFAULT is on f= or Linux or NetBSD x86, otherwise off)]),,[ - case "$host_cpu" in - i[[3456]]86|x86_64) - enable_qemu_traditional=3D"yes";; - *) enable_qemu_traditional=3D"no";; - esac - case "$host_os" in - freebsd*) - enable_qemu_traditional=3D"no";; - esac - -]) + [Enable qemu traditional device model, (DEFAULT is off)= ])) AS_IF([test "x$enable_qemu_traditional" =3D "xyes"], [ AC_DEFINE([HAVE_QEMU_TRADITIONAL], [1], [Qemu traditional enabled]) qemu_traditional=3Dy],[ --=20 2.26.2