From nobody Sat May 4 12:04: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=1636027456; cv=none; d=zohomail.com; s=zohoarc; b=Hm8CAp0EUUlFL0UdZEz0YPQrkuZZSlqO6roppVaCiodYG52d+shxUPPlZvbHQcjo9q8Qb/OSC5adcmeDu5J/hqSTz2PTNQjpDZn2Rb/XYwbUqyg0zYINnkaAa88Uud8tspeKnVL9tRp4cp8SO4qsWl7uw/xSO7AmDbxhNY29MGo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1636027456; 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=4W8QFCSZ6YUbqfKweIjCXKAp41fzIgy106byLf3XNR4=; b=mTWm+cxlPH/ju7hcUV5o3PhYW5wXqlYNbH+dLXHrwFE3E/gSZaoepwnAGf6oLkFoqY6K46IaXLln1PcbvixYBfhjEVsETC9a7jh95YJcWFxMIP38eIY21EjJcuAMMqrk0DYOOrOxbbJHqhM1PvULZjN9vb6bF4BcxuUjCDbTqc8= 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 1636027456167823.6361373572835; Thu, 4 Nov 2021 05:04:16 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.221460.383180 (Exim 4.92) (envelope-from ) id 1mibSu-0001i6-Lt; Thu, 04 Nov 2021 12:03:32 +0000 Received: by outflank-mailman (output) from mailman id 221460.383180; Thu, 04 Nov 2021 12:03:32 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mibSu-0001gs-Dn; Thu, 04 Nov 2021 12:03:32 +0000 Received: by outflank-mailman (input) for mailman id 221460; Thu, 04 Nov 2021 12:03:31 +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 1mibSs-0001eM-W1 for xen-devel@lists.xenproject.org; Thu, 04 Nov 2021 12:03:31 +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 38d9c73c-3d67-11ec-a9d2-d9f7a1cc8784; Thu, 04 Nov 2021 13:03:29 +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 75F361FD47; Thu, 4 Nov 2021 12:03:28 +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 51CA613C68; Thu, 4 Nov 2021 12:03:28 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id AKnaEhDMg2HJfQAAMHmgww (envelope-from ); Thu, 04 Nov 2021 12:03:28 +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: 38d9c73c-3d67-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1636027408; 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=4W8QFCSZ6YUbqfKweIjCXKAp41fzIgy106byLf3XNR4=; b=Wexz9RccvlB9oSkK2FMZzMLj23F8Cz9U1nkKCutgOHML5s/V/+SbznnubuTlw2L90GMBWj Lv3ZA46aRlhto0HZjBnlnGBhc5s4t/ALVMezuRx55AWwG8ktQ4r7wzsh+/OO+vIOUFPQaa 0nZ5EO5BEzRNMnURbT/EMbxt0prFABM= From: Juergen Gross To: xen-devel@lists.xenproject.org Cc: Juergen Gross , Ian Jackson , Wei Liu Subject: [PATCH-for-4.16 1/2] configure: modify default of building rombios Date: Thu, 4 Nov 2021 13:03:23 +0100 Message-Id: <20211104120324.9668-2-jgross@suse.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20211104120324.9668-1-jgross@suse.com> References: <20211104120324.9668-1-jgross@suse.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1636027457773100001 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 per default when disabling qemu traditional. Fix that be rearranging the dependencies: - build ipxe per default - build rombios per default if either ipxe or qemu traditional are being built This modification prepares not building qemu traditional per default without affecting build of rombios and ipxe. Signed-off-by: Juergen Gross Reviewed-by: Ian Jackson --- tools/configure | 116 +++++++++++++++++++++------------------------ tools/configure.ac | 67 +++++++++++++------------- 2 files changed, 86 insertions(+), 97 deletions(-) diff --git a/tools/configure b/tools/configure index 33814b24b3..7b62b3cdd8 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,12 @@ 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 and is bound by the + presence of rombios, --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 +1557,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 +4314,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 +4676,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..4be3138cb3 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 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" ], [ + 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 Sat May 4 12:04: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=1636027457; cv=none; d=zohomail.com; s=zohoarc; b=QwiySSb8jqyHLtWiHb8uzx8rCWiNu/aX1trt56u51MxrL3RagYQMg2gPcc2QEC7jpcms9f0BhR0GEB5sV5TkHWGQXtL4KM2DP65AQwq2M70ZSea3o2WOMn0dkBFSwV+berMbK9s3CHSRLLBFGH7lwF2uv/VuYcXvuaBuKL1OhHA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1636027457; 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=k/v5OEykdgPDYAbYvSHlTvD3M2pDaoCL1y7adw+gtcU=; b=SjNJJOABWB8zvib0tKP8G7fTgMVZA4ZcCemKvbAMxMCsPL2LsS+0uqo5aPdnznWh86amb4gc0pr0QtXFM4XO7mb9OKmv5IkceDpviiWVJKGngF07tcugFxESWNE2kQj7/czlkH1MHU+n13kjZoVjkc5JymhwfxqYYWPR7jyHl3I= 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 163602745723657.343247334250236; Thu, 4 Nov 2021 05:04:17 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.221461.383188 (Exim 4.92) (envelope-from ) id 1mibSv-0001oq-0c; Thu, 04 Nov 2021 12:03:33 +0000 Received: by outflank-mailman (output) from mailman id 221461.383188; Thu, 04 Nov 2021 12:03:32 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mibSu-0001lD-OC; Thu, 04 Nov 2021 12:03:32 +0000 Received: by outflank-mailman (input) for mailman id 221461; Thu, 04 Nov 2021 12:03:31 +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 1mibSt-0001eN-Ic for xen-devel@lists.xenproject.org; Thu, 04 Nov 2021 12:03:31 +0000 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 38eaa456-3d67-11ec-9787-a32c541c8605; Thu, 04 Nov 2021 13:03:29 +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-out1.suse.de (Postfix) with ESMTPS id A7E96218D5; Thu, 4 Nov 2021 12:03:28 +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 7D40D13C68; Thu, 4 Nov 2021 12:03:28 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id uMJsHRDMg2HJfQAAMHmgww (envelope-from ); Thu, 04 Nov 2021 12:03:28 +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: 38eaa456-3d67-11ec-9787-a32c541c8605 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1636027408; 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=k/v5OEykdgPDYAbYvSHlTvD3M2pDaoCL1y7adw+gtcU=; b=Lilj/uFphU6V6ULoQp+bU8PUtpxmviLaCKSaeDfqNdLLdicbBhy6jdUbx4Bp5wUSTrkNdp x+5jRDbFv4iJbEOFE36EaOgCmWDPAd8SpYwwdLKTv3+pYd6/o1T14Z46/IQ8M8IFHrpN6M EgpK1Ac5LEAnAzv3ax25lK3xcICM1P8= 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 2/2] tools: disable building qemu-trad per default Date: Thu, 4 Nov 2021 13:03:24 +0100 Message-Id: <20211104120324.9668-3-jgross@suse.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20211104120324.9668-1-jgross@suse.com> References: <20211104120324.9668-1-jgross@suse.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1636027459594100003 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 7b62b3cdd8..eb4ab9d723 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 @@ -4287,19 +4287,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 4be3138cb3..02f310ce6c 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