From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id xH6jF9kBLmO+7D4AWB0awg (envelope-from ) for ; Fri, 23 Sep 2022 14:58:33 -0400 Received: by simark.ca (Postfix, from userid 112) id 541791E112; Fri, 23 Sep 2022 14:58:33 -0400 (EDT) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=PXthjSnM; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-4.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,NICE_REPLY_A,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id A85361E07B for ; Fri, 23 Sep 2022 14:58:29 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 316733857BBB for ; Fri, 23 Sep 2022 18:58:28 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 316733857BBB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1663959508; bh=jsxQuyzzWUNYHLdbOxLHrrOtrtrZWPm//mKdC5OuhLo=; h=Date:Subject:To:References:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=PXthjSnMZeByV7rxT/M6+nDb1XODFjt8q+s3lg6/wZUrBDLEa4B1RYnvvdY5K66xz UhlUhCkihWQvs6eohO3vW4XvLEe2Mnn/NpXDroj+y5e8yhpvIWfXI/lnpi1vT1JgBm c9ynWTrU5VVHLOUx9u2xlCypeP3TcxHicnS+drFs= Received: from smtp.polymtl.ca (smtp.polymtl.ca [132.207.4.11]) by sourceware.org (Postfix) with ESMTPS id 0E0643857C6F; Fri, 23 Sep 2022 18:58:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0E0643857C6F Received: from simark.ca (simark.ca [158.69.221.121]) (authenticated bits=0) by smtp.polymtl.ca (8.14.7/8.14.7) with ESMTP id 28NIvsUR005794 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 23 Sep 2022 14:57:59 -0400 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp.polymtl.ca 28NIvsUR005794 Received: from [10.0.0.11] (unknown [217.28.27.60]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPSA id 10C471E07B; Fri, 23 Sep 2022 14:57:54 -0400 (EDT) Message-ID: Date: Fri, 23 Sep 2022 14:57:53 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH v3] binutils, gdb: support zstd compressed debug sections Content-Language: en-US To: Fangrui Song , Nick Clifton References: <20220923040837.550160-1-maskray@google.com> <20220923182038.ilg6lh235f6zzdmr@google.com> In-Reply-To: <20220923182038.ilg6lh235f6zzdmr@google.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Poly-FromMTA: (simark.ca [158.69.221.121]) at Fri, 23 Sep 2022 18:57:54 +0000 X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Simon Marchi via Gdb-patches Reply-To: Simon Marchi Cc: gdb-patches@sourceware.org, binutils@sourceware.org, Jan Beulich , Alan Modra Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" On 2022-09-23 14:20, Fangrui Song wrote: > On 2022-09-23, Simon Marchi wrote: >> >> >> On 2022-09-23 11:45, Nick Clifton wrote: >>> Hi Fangrui, >>> >>>> PR29397 PR29563: The new configure option --with-zstd defaults to auto. >>>> If pkgconfig/libzstd.pc is found, define HAVE_ZSTD and support zstd >>>> compressed debug sections for most tools. > > Hi Simon and Nick, > > I have checked that `make all-gdb` works in a --enable-gdb --without-debuginfod build. > > I then inspected the 2020 commits adding AC_DEBUGINFOD to > gdb/configure.ac and binutils/configure.ac. They come with no > PKG_PROG_PKG_CONFIG and config/debuginfod.m4 does not have > PKG_PROG_PKG_CONFIG, but they still work.  So I think AC_ZSTD doesn't > need PKG_PROG_PKG_CONFIG, either. Though I have no insight why it > behaves that way:) Ok, I am a bit confused. Based on my comprehension, since the first call to PKG_CHECK_MODULES may not happen do to a condition: if test "x$with_debuginfod" != xno; then PKG_CHECK_MODULES([DEBUGINFOD], [libdebuginfod >= 0.179], ... fi ... the probing for pkg-config should not happen if --without-debuginfod is used. But in gdb/configure, I see that the probing for pkg-config actually appears outside the condition. Anyway, good if it works. > +diff --git a/gdb/NEWS b/gdb/NEWS > +index 9619842bc03..1457c99ff04 100644 > +--- a/gdb/NEWS > ++++ b/gdb/NEWS > +@@ -57,6 +57,8 @@ > + + * The Windows native target now supports target async. > + ++* gdb now supports zstd compressed debug sections (ELFCOMPRESS_ZSTD) for ELF. > ++ I'm not sure I understand the diff above, but I think it's missing an empty line between the previous bullet and the new bullet. Otherwise, that is OK for GDB. Simon