From nobody Mon Feb 9 16:18:53 2026 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=reject dis=none) header.from=citrix.com ARC-Seal: i=1; a=rsa-sha256; t=1638810471; cv=none; d=zohomail.com; s=zohoarc; b=nMEMP+Wtv4KXcpJV3d6Sn+WtZr4cBIfBTex8TMXLpRpRxB9fv+S/5M8Vk75ZZ3vg2vy0BPtiBtry450xoER0OfS9jPwwrRJqrE7njeni+40HpayY/cLcw1DUrxGHL/49zf4KjkFHOkeSlHHeZfSoWsYO+UsGroFOOFC1iZS+brI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1638810471; h=Content-Type: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=wZJ90c/MFvhmzt2/lDpi5dxEjIH5+Xlp8OAfmtQ0HOE=; b=MnFFLeUZ54XqN2zRQQPu3/jYlDY54D8kgoljwcXEqiD9Rf2oS/SJMlzMRwnmGlNb8bHgGlGyp8OcTQ2NNkuh0/AfLCuFH9Ll7FN0SYSkxp7jul8c3heE1p2WdRtDtuVIyTo5LwEI3N8aQowCTZMs22AKaLCj+gMGjYvkH9wJCT4= 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=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1638810471307847.1138603442977; Mon, 6 Dec 2021 09:07:51 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.239476.415187 (Exim 4.92) (envelope-from ) id 1muHSj-0000rY-DF; Mon, 06 Dec 2021 17:07:37 +0000 Received: by outflank-mailman (output) from mailman id 239476.415187; Mon, 06 Dec 2021 17:07:37 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1muHSi-0000py-Vb; Mon, 06 Dec 2021 17:07:36 +0000 Received: by outflank-mailman (input) for mailman id 239476; Mon, 06 Dec 2021 17:07:35 +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 1muHOq-0005ti-Ho for xen-devel@lists.xenproject.org; Mon, 06 Dec 2021 17:03:36 +0000 Received: from esa6.hc3370-68.iphmx.com (esa6.hc3370-68.iphmx.com [216.71.155.175]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 714dd024-56b6-11ec-8a4d-196798b21f7b; Mon, 06 Dec 2021 18:03:34 +0100 (CET) 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: 714dd024-56b6-11ec-8a4d-196798b21f7b DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=citrix.com; s=securemail; t=1638810214; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=87Jwg3dUCW2JRgwzDCH17RKTaiNypi0W5MI+j9zb2Dw=; b=LQZ1fSRynnjoshix7PuQJUgu4lwgFfIx5cXQl47wdlSPOeFSCWkQQFCU 6hy+1H35iV3yaZ2rlwXE2i+KdjFZVVy0kDROboZgan1IyARYSwBKH28C9 FWXsqcLPqeqFhx5NYmtBU02VEtWOm0P6HoH+25dss2LULOtyp35kbOcM/ k=; Authentication-Results: esa6.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none IronPort-SDR: aRgDtDW9+i7arDAhUgwN9Wt3ZP+K9bzn48v7bdjUxgCDVK4pTw3RyGQ/3ZpDYguReP0VIyqa8g P+AMN+z2PsPWPaBKj2RNfhnMiKc/q+YpE9wEJ0eCYZYcrEwvC8D6evICfZaAcSWZvUfl+UAgpj tiesAr6vB1vHb2CtvL5UEVbgBJDFERpA/7T8s47WHyug4QkBleYKfDj4npuUULkHrMIxoCy/oF X37WQBIXSp5FNrSOo7AGmwf49WYlWBZnI8xZicEgk5R4VqII/lvsMxu0s+CTnlOtribpmLBRrn 4sy7WHlXmCM8U2pSY/0RXVZ5 X-SBRS: 5.1 X-MesageID: 59324028 X-Ironport-Server: esa6.hc3370-68.iphmx.com X-Remote-IP: 162.221.156.83 X-Policy: $RELAYED IronPort-Data: A9a23:ejcUyqNr2ibBIcDvrR1UkMFynXyQoLVcMsEvi/4bfWQNrUoj0TEHz mRJDTvTP/vYMDHzftF3Odu1801TupPRn4dlQAto+SlhQUwRpJueD7x1DKtR0wB+jCHnZBg6h ynLQoCYdKjYdpJYz/uUGuCJQUNUjMlkfZKhTr6bUsxNbVU8En5400o9w7RRbrNA2rBVPSvc4 bsenOWHULOV82Yc3rU8sv/rRLtH5ZweiRtA1rAMTakjUGz2zhH5OKk3N6CpR0YUd6EPdgKMq 0Qv+5nilo/R109F5tpICd8XeGVSKlLZFVDmZna7x8FOK/WNz8A/+v9TCRYSVatYoyi1rfVSw tdOjMyTdwk1bojjg88dbSANRkmSPYUekFPGCX22sMjVxEzaaXr8hf5pCSnaP6VBpLwxWzsXs 6VFdnZdNXhvhMrvqF6/YuBqmsQkKtitJI4Fs2ts5TrYEewnUdbIRKCiCdpwgWZr3JwWR6a2i 8wxWSo/MyrQWkJzCnwSJ8wQhPy5mGDfWmgNwL6SjfVuuDWCpOBr65DyNPLFd9rMQt9a9m6aq XjB5H/RGQwBOZqUzj/t2nOzhMffkCXjQoUQGbaksPlwjzW71mEVTREbS1a/if24kVKlHcJSL VQO/SgjprR081akJvH0WQG4p3iV+AYRQdNKH8U17QiMzuzf5APxO4QfZmcfMpp87pZwHGF0k A/S9z/0OdBxmKCpe1e+/LWXlGORByYJfUotfgQEczJQtrEPv7oPph7IS99iFou8gdv0BSz8z li2kcQuu1kApZVVjvvmpDgrlxrp/8GUFVBtum07S0r8tlshDLNJcbBE/rQyARxoCI+CBmeMs 3Ef8yR1xLBfVMrd/MBhrQhkIV1I2xpnGGCF6bKMN8N4n9hIx5JFVdoKiN2ZDB02WvvogRezP CfuVfp5vfe/xkeCY65teJ6WAM8316XmHtmNfqmKNYoROMksLlXZo3EGiausM4bFyhJEfUYXY 8jzTCpRJSxCVfQPIMSeGY/xLoPHNghhnDiOFPgXPjys0KaEZW79dFv2GADmUwzN14vd+F+92 48Gb6OikkwDOMWjMni/2dNCdjgicClkba0aXuQKL4Zv1CI9Qzp/YxIQqJt8E7FYc1N9yr2Vo yrjAxAAkzISRxTvcG23V5yqU5u3Nb4XkJ7xFXZE0Y+A1ydxbICxwr0YcpdrL7Ar+PY6lax/T uUfetXGCfNKE2yV9zMYZJj7jYpjaBX02l7eY3v7OGAyL8x6WgjE2t74ZQ+zpiMAOTW66Jklq Lq62wKFHZdaH1Z+DNzbYe6Exk+quSRPg/p7WkbFe4EBeEjl/IVwBTb2i/s7f5MFJRnZn2PI3 AeKGxYI4+LKptZtotXOgKmFqaavEvd/QRUGTzWKs+7uOHCDrGS5wIJGXOKZRhznVTv5qPe4e OFY7/DgK/lbzlxEhJVxTuRwxqUk6tqx+7IDllZ4HG/GZkiAA697JiXUxtFGs6BAy+MLuQayX U7TqNBWNa/QZZHgGV8VYgEkcv6CxbcfnTyLtaY5J0Dz5SlW+rubUBoNY0nQ2XIFdLYlYpk4x eoBudIN713tgxUnBd+KkyRI+jneNXcHSagm6skXDYKDZtDHEb2ejUgw0hPL3aw= IronPort-HdrOrdr: A9a23:K8bvzKEIbgwv9TqvpLqE4seALOsnbusQ8zAXP0AYc3Nom6uj5q WTdZUgpHjJYVkqOU3I9ersBEDiewK/yXcW2+ks1N6ZNWGM0ldARLsSibcKqAePJ8SRzIJgPN 9bAstDNOE= X-IronPort-AV: E=Sophos;i="5.87,292,1631592000"; d="scan'208";a="59324028" From: Anthony PERARD To: CC: Anthony PERARD , Anthony PERARD , Ian Jackson , Wei Liu , Juergen Gross Subject: [XEN PATCH 19/57] tools/configure.ac: Create ZLIB_LIBS and ZLIB_CFLAGS Date: Mon, 6 Dec 2021 17:02:02 +0000 Message-ID: <20211206170241.13165-20-anthony.perard@citrix.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20211206170241.13165-1-anthony.perard@citrix.com> References: <20211206170241.13165-1-anthony.perard@citrix.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1638810471746100001 Content-Type: text/plain; charset="utf-8" Use both ZLIB_CFLAGS and ZLIB_LIBS instead of cherry-picking flags from a single "ZLIB" variable. Signed-off-by: Anthony PERARD Reviewed-by: Juergen Gross --- tools/configure.ac | 19 ++++++++++++++----- config/Tools.mk.in | 3 ++- tools/configure | 22 ++++++++++++++-------- tools/libs/guest/Makefile | 12 +++++------- 4 files changed, 35 insertions(+), 21 deletions(-) diff --git a/tools/configure.ac b/tools/configure.ac index 5a4fb9022d..56afe7930d 100644 --- a/tools/configure.ac +++ b/tools/configure.ac @@ -391,17 +391,26 @@ AX_CHECK_FETCHER =20 # Checks for libraries. AC_CHECK_HEADER([bzlib.h], [ -AC_CHECK_LIB([bz2], [BZ2_bzDecompressInit], [zlib=3D"$zlib -DHAVE_BZLIB -l= bz2"]) + AC_CHECK_LIB([bz2], [BZ2_bzDecompressInit], + [ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_BZLIB" + ZLIB_LIBS=3D"$ZLIB_LIBS -lbz2"]) ]) AC_CHECK_HEADER([lzma.h], [ -AC_CHECK_LIB([lzma], [lzma_stream_decoder], [zlib=3D"$zlib -DHAVE_LZMA -ll= zma"]) + AC_CHECK_LIB([lzma], [lzma_stream_decoder], + [ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_LZMA" + ZLIB_LIBS=3D"$ZLIB_LIBS -llzma"]) ]) AC_CHECK_HEADER([lzo/lzo1x.h], [ -AC_CHECK_LIB([lzo2], [lzo1x_decompress], [zlib=3D"$zlib -DHAVE_LZO1X -llzo= 2"]) + AC_CHECK_LIB([lzo2], [lzo1x_decompress], + [ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_LZO1X" + ZLIB_LIBS=3D"$ZLIB_LIBS -llzo2"]) ]) PKG_CHECK_MODULES([libzstd], [libzstd], - [zlib=3D"$zlib -DHAVE_ZSTD $libzstd_CFLAGS $libzstd_LIBS"], [true]) -AC_SUBST(zlib) + [ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_ZSTD $libzstd_CFLAGS" + ZLIB_LIBS=3D"$ZLIB_LIBS $libzstd_LIBS"], + [true]) +AC_SUBST([ZLIB_CFLAGS]) +AC_SUBST([ZLIB_LIBS]) AX_CHECK_EXTFS AX_CHECK_PTHREAD AX_CHECK_PTYFUNCS diff --git a/config/Tools.mk.in b/config/Tools.mk.in index 934d899967..6c1a0a676f 100644 --- a/config/Tools.mk.in +++ b/config/Tools.mk.in @@ -66,7 +66,8 @@ CONFIG_9PFS :=3D @ninepfs@ LINUX_BACKEND_MODULES :=3D @LINUX_BACKEND_MODULES@ =20 #System options -ZLIB :=3D @zlib@ +ZLIB_CFLAGS :=3D @ZLIB_CFLAGS@ +ZLIB_LIBS :=3D @ZLIB_LIBS@ CONFIG_LIBICONV :=3D @libiconv@ EXTFS_LIBS :=3D @EXTFS_LIBS@ CURSES_LIBS :=3D @CURSES_LIBS@ diff --git a/tools/configure b/tools/configure index 829753b5dd..a052c186a5 100755 --- a/tools/configure +++ b/tools/configure @@ -641,7 +641,8 @@ PTHREAD_LIBS PTHREAD_LDFLAGS PTHREAD_CFLAGS EXTFS_LIBS -zlib +ZLIB_LIBS +ZLIB_CFLAGS libzstd_LIBS libzstd_CFLAGS FETCHER @@ -8605,7 +8606,7 @@ fi ac_fn_c_check_header_mongrel "$LINENO" "bzlib.h" "ac_cv_header_bzlib_h" "$= ac_includes_default" if test "x$ac_cv_header_bzlib_h" =3D xyes; then : =20 -{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for BZ2_bzDecompressInit= in -lbz2" >&5 + { $as_echo "$as_me:${as_lineno-$LINENO}: checking for BZ2_bzDecompress= Init in -lbz2" >&5 $as_echo_n "checking for BZ2_bzDecompressInit in -lbz2... " >&6; } if ${ac_cv_lib_bz2_BZ2_bzDecompressInit+:} false; then : $as_echo_n "(cached) " >&6 @@ -8642,7 +8643,8 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_lib_bz2_BZ2_bzDeco= mpressInit" >&5 $as_echo "$ac_cv_lib_bz2_BZ2_bzDecompressInit" >&6; } if test "x$ac_cv_lib_bz2_BZ2_bzDecompressInit" =3D xyes; then : - zlib=3D"$zlib -DHAVE_BZLIB -lbz2" + ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_BZLIB" + ZLIB_LIBS=3D"$ZLIB_LIBS -lbz2" fi =20 =20 @@ -8652,7 +8654,7 @@ fi ac_fn_c_check_header_mongrel "$LINENO" "lzma.h" "ac_cv_header_lzma_h" "$ac= _includes_default" if test "x$ac_cv_header_lzma_h" =3D xyes; then : =20 -{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for lzma_stream_decoder = in -llzma" >&5 + { $as_echo "$as_me:${as_lineno-$LINENO}: checking for lzma_stream_deco= der in -llzma" >&5 $as_echo_n "checking for lzma_stream_decoder in -llzma... " >&6; } if ${ac_cv_lib_lzma_lzma_stream_decoder+:} false; then : $as_echo_n "(cached) " >&6 @@ -8689,7 +8691,8 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_lib_lzma_lzma_stre= am_decoder" >&5 $as_echo "$ac_cv_lib_lzma_lzma_stream_decoder" >&6; } if test "x$ac_cv_lib_lzma_lzma_stream_decoder" =3D xyes; then : - zlib=3D"$zlib -DHAVE_LZMA -llzma" + ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_LZMA" + ZLIB_LIBS=3D"$ZLIB_LIBS -llzma" fi =20 =20 @@ -8699,7 +8702,7 @@ fi ac_fn_c_check_header_mongrel "$LINENO" "lzo/lzo1x.h" "ac_cv_header_lzo_lzo= 1x_h" "$ac_includes_default" if test "x$ac_cv_header_lzo_lzo1x_h" =3D xyes; then : =20 -{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for lzo1x_decompress in = -llzo2" >&5 + { $as_echo "$as_me:${as_lineno-$LINENO}: checking for lzo1x_decompress= in -llzo2" >&5 $as_echo_n "checking for lzo1x_decompress in -llzo2... " >&6; } if ${ac_cv_lib_lzo2_lzo1x_decompress+:} false; then : $as_echo_n "(cached) " >&6 @@ -8736,7 +8739,8 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_lib_lzo2_lzo1x_dec= ompress" >&5 $as_echo "$ac_cv_lib_lzo2_lzo1x_decompress" >&6; } if test "x$ac_cv_lib_lzo2_lzo1x_decompress" =3D xyes; then : - zlib=3D"$zlib -DHAVE_LZO1X -llzo2" + ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_LZO1X" + ZLIB_LIBS=3D"$ZLIB_LIBS -llzo2" fi =20 =20 @@ -8812,10 +8816,12 @@ else libzstd_LIBS=3D$pkg_cv_libzstd_LIBS { $as_echo "$as_me:${as_lineno-$LINENO}: result: yes" >&5 $as_echo "yes" >&6; } - zlib=3D"$zlib -DHAVE_ZSTD $libzstd_CFLAGS $libzstd_LIBS" + ZLIB_CFLAGS=3D"$ZLIB_CFLAGS -DHAVE_ZSTD $libzstd_CFLAGS" + ZLIB_LIBS=3D"$ZLIB_LIBS $libzstd_LIBS" fi =20 =20 + ac_fn_c_check_header_mongrel "$LINENO" "ext2fs/ext2fs.h" "ac_cv_header_ext= 2fs_ext2fs_h" "$ac_includes_default" if test "x$ac_cv_header_ext2fs_ext2fs_h" =3D xyes; then : =20 diff --git a/tools/libs/guest/Makefile b/tools/libs/guest/Makefile index 8f5f3acd21..1f4b7f7c58 100644 --- a/tools/libs/guest/Makefile +++ b/tools/libs/guest/Makefile @@ -89,13 +89,12 @@ CFLAGS +=3D $(CFLAGS_libxendevicemodel) CFLAGS +=3D $(CFLAGS_libxencall) $(CFLAGS_libxenforeignmemory) =20 ifeq ($(CONFIG_MiniOS),y) -zlib-options =3D -else -zlib-options =3D $(ZLIB) +ZLIB_CFLAGS :=3D +ZLIB_LIBS :=3D endif =20 -xg_dom_bzimageloader.o: CFLAGS +=3D $(filter -D%,$(zlib-options)) -xg_dom_bzimageloader.opic: CFLAGS +=3D $(filter -D%,$(zlib-options)) +xg_dom_bzimageloader.o: CFLAGS +=3D $(ZLIB_CFLAGS) +xg_dom_bzimageloader.opic: CFLAGS +=3D $(ZLIB_CFLAGS) =20 LIBHEADER :=3D xenguest.h =20 @@ -103,8 +102,7 @@ NO_HEADERS_CHK :=3D y =20 include $(XEN_ROOT)/tools/libs/libs.mk =20 -libxenguest.so.$(MAJOR).$(MINOR): COMPRESSION_LIBS =3D $(filter -l%,$(zlib= -options)) -libxenguest.so.$(MAJOR).$(MINOR): APPEND_LDFLAGS +=3D $(COMPRESSION_LIBS) = -lz +libxenguest.so.$(MAJOR).$(MINOR): APPEND_LDFLAGS +=3D $(ZLIB_LIBS) -lz =20 -include $(DEPS_INCLUDE) =20 --=20 Anthony PERARD