From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id uWAZARYm5WhoDSMAWB0awg (envelope-from ) for ; Tue, 07 Oct 2025 10:39:18 -0400 Authentication-Results: simark.ca; dkim=pass (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=QAcetXNT; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id E771A1E0B6; Tue, 07 Oct 2025 10:39:17 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-3.4 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,RCVD_IN_VALIDITY_CERTIFIED_BLOCKED, RCVD_IN_VALIDITY_RPBL_BLOCKED,RCVD_IN_VALIDITY_SAFE_BLOCKED autolearn=ham autolearn_force=no version=4.0.1 Received: from server2.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 ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id D504C1E047 for ; Tue, 07 Oct 2025 10:39:16 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 4F0EB3858C60 for ; Tue, 7 Oct 2025 14:39:16 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4F0EB3858C60 Authentication-Results: sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=QAcetXNT Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTP id 1CD403858D1E for ; Tue, 7 Oct 2025 14:38:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 1CD403858D1E Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 1CD403858D1E Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=170.10.129.124 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1759847915; cv=none; b=vKTFTIACr3Fa/yBeG4S8yFYOaT5DzURIjwVGbi+83VW4UleEEFdPPsmqm/FRJi9tCMNjOE9a5zjh8t5RvskNt+MZ5DQFhTWZi36+q9K8RfzJM/R1nN/eYsa7bYlSRv1ihpZwTK36wDWehvN/BFIiuru/2R5Sb4xcmC0ke304muk= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1759847915; c=relaxed/simple; bh=lyaqh42DvNqBMEek4iqClrF+WlIXlDG8MbyaLDqqEVU=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=TY8OLDd3K0m6VmoA+fFwAqaWzG1AqoeBWmBWfQp9fwQDYx0l72y9RWxHX604lbE2ZTmTVTzyIeyBWaOzMWd2176Oo6T6bf1rpOQlF+aWdTX0w/gBwZGKfgrTIVJY7O/BPv911B1MGOluwi9VRW9EfHM80HQlU/ngOTmBOkv6Vok= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 1CD403858D1E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1759847914; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kME3mZd6tVtyxr7jpTWZN/OUo7G5ZJHTpEVMB7TVBRI=; b=QAcetXNTByQeSrIt4T/QMbucxvV2ROOp6VRNDJs5H+J+r/wGD2vS6eSlJjy07GlVu9pY9D 8pw8Drx0GOUKlQo+0DEhu4zroPq2Famvzl+n2uZEg2WhV6aDKLKire54jkweJYivMapD+e a30qVmCeZw0TGvuPR7q9iBl/JBjHCnU= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-77-2Pqlp7ivMSyU0Ahr2V9rmg-1; Tue, 07 Oct 2025 10:38:33 -0400 X-MC-Unique: 2Pqlp7ivMSyU0Ahr2V9rmg-1 X-Mimecast-MFC-AGG-ID: 2Pqlp7ivMSyU0Ahr2V9rmg_1759847912 Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-401dbafbcfaso3883363f8f.1 for ; Tue, 07 Oct 2025 07:38:33 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1759847912; x=1760452712; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kME3mZd6tVtyxr7jpTWZN/OUo7G5ZJHTpEVMB7TVBRI=; b=d75Kgr9cYcI2dCXy3TKbePqsaSjBgN5KT8iXCeLKnDil2NYNCBm3x7fErCGanVC2Ht fDou+dSh/w5x1X63NYDiRtWUb6+bbFPJ2jR/zrYAnhG/o9voyEOkHGqVyfNgjqX/UnpO fiZ3P7psM0mxaiJbTQTvMVQBWxbh0HXmw/HdOHj9Wyn/bh93UG14lcCDtmZPErroZVeg oyMttQnTpNyox0hmw7dQYT7LkH8Rks4xjvF13n4xuvKwHCBbLm3KZWiLQwwyxXgEGJFK Icy48mn/PF9BQ02K0+WL1lS79cv1vQkZxt/JS3SOfzvJzmYlAXqJVroQhRKUN7Dk47kW SFaw== X-Gm-Message-State: AOJu0YzDap+YYVEcHUJeEykCLi1IjryiRRdM+hmImu7dvzfbHDTa+PJs C4Z7OD6Zt+GPSb/iwM73yyLGWTFGAXXknexhM4Rgu5nBwCD75m24FJneWo+eQP1twwOnM6+Zvrt iNnGrhNkITz0B3efqa6TaWuUe4+KQrrxgEg68WpFkfuWIUrXmvcMtJq2VmIlOHak= X-Gm-Gg: ASbGncuG+PVX5pgpcLOe/1lFTvWbpi3eJlg7YhEtXnUBT/2X3J14bnAfFSKpwagDCzj QPrnRB1LPoJ4xxIybwiYqPeruE7e+4RBM7uoePYB1H9zN39d1vr3He7Zfc7wF4PsaWv+tx0CcJw 05ilhC361GIEgbwtv46Roh1oRJqNh4wZ0B5kYiiH0sb5dFmM1ThlKuUSvRTXjvzWKQ86qrMRnwT O1WDG8PCrxU7vHfBXUxaoiM0ZT2DAHk2jIWvxQE0pkSO7TFEFr8QOZDAFV+ybdTQRBWTPSJgM/Y jctGnAWQcwi0BbqmG/HReGo7rIrM+tmxRSgAN6tr X-Received: by 2002:a05:6000:2506:b0:414:6fe6:8fbd with SMTP id ffacd0b85a97d-425671c10f5mr10912666f8f.54.1759847912198; Tue, 07 Oct 2025 07:38:32 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHVSS19OK1WVHe1vfC8B8ZNaKo3HcRitWwdCEfJ/B4mSuOQX2fmZZG5JZKUuHtibwh/vwXEfg== X-Received: by 2002:a05:6000:2506:b0:414:6fe6:8fbd with SMTP id ffacd0b85a97d-425671c10f5mr10912637f8f.54.1759847911665; Tue, 07 Oct 2025 07:38:31 -0700 (PDT) Received: from localhost ([31.111.84.207]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4255d8a6b77sm25743081f8f.6.2025.10.07.07.38.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Oct 2025 07:38:31 -0700 (PDT) From: Andrew Burgess To: Tom de Vries , Tom Tromey Cc: gdb-patches@sourceware.org Subject: Re: [PATCHv2 3/3] gdb/python: add Corefile.mapped_files method In-Reply-To: <87o6qi6dwn.fsf@redhat.com> References: <9ab589510f784da2752b72d0b1385afa33aca406.1758634958.git.aburgess@redhat.com> <87cy737q50.fsf@tromey.com> <87zfa26gxp.fsf@redhat.com> <87o6qi6dwn.fsf@redhat.com> Date: Tue, 07 Oct 2025 15:38:30 +0100 Message-ID: <87jz166al5.fsf@redhat.com> MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: ESpq0vt-kMeRGM_WXkv9ZMwdOi65lj2iqAswkahtGt8_1759847912 X-Mimecast-Originator: redhat.com Content-Type: text/plain X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org Andrew Burgess writes: > > Thanks for the diagnostic help. I'll work on improving the test to try > and catch these cases. If that works, then we might be able to adopt > this to help with other tests. Keeping this in mind, I'll add helper > functions to lib/gdb.exp. Hi Tom, At your leisure, could you check that the patch below resolves the FAIL you reported please. Thanks, Andrew --- commit 6d6ec8321fcbbd12dc1544487367e2adefdc8ac9 Author: Andrew Burgess Date: Tue Oct 7 15:24:01 2025 +0100 gdb/testsuite: fix failure from gdb.python/py-corefile.exp After commit: commit f69c1d03c4d6c68ae3f90facd63245426c028047 Date: Mon Aug 25 16:48:22 2025 +0100 gdb/python: add Corefile.mapped_files method Tom reported a failure: (gdb) check-build-ids Python Exception : build-id mismatch for /lib64/libc.so.6 Error occurred in Python: build-id mismatch for /lib64/libc.so.6 (gdb) FAIL: gdb.python/py-corefile.exp: test mapped files data: check-build-ids The discussion thread can be found here: https://inbox.sourceware.org/gdb-patches/de21b43c-e3bd-4354-aace-bd3f50c1c64c@suse.de The problem is an issue where some versions of the linker didn't place the build-id within the first page of an ELF. As a result, the Linux kernel would not include the build-id in the generated core file, and so GDB cannot to find the build-id. In this patch I've added mitigation for this problem. The check-build-ids command was defined in Python, and previously did all the checks, printing "PASS" if everything was fine, otherwise raising a Python exception. This has now changed to a show-build-ids Python command. This extracts the build-ids from the inferior's objfiles and core file, and prints a table. The .exp test script can then process this table, and decide if a difference is bad or not. When processing the show-build-ids table, if the build-id from the objfile object matches the build-id from the core file, then this is fine. If we have an objfile build-id and a different core file build-id, then this is an error, GDB managed to find the _wrong_ build-id. Very bad. If we have an objfile build-id, and no core file build-id, then we use a new helper function added to lib/gdb.exp, expect_build_id_in_core_file. This helper uses readelf to check where the build-id is located within the file. If the build-id is within the first page of the ELF then we expect the build-id to appear in the core file, and GDB should be able to find it, in this case we have found a GDB bug, and the test will FAIL. If however, readelf says that the build-id is outside the first page of the ELF, then we don't expect GDB to be able to find the build-id (as it likely isn't in the core file). In this case we don't count this as a test failure. I'm only aware of GNU/Linux forcing the first page of every mapped ELF into the generated core files, and so, I've made the part of the test that relies on this Linux only. This change should resolve the FAIL that Tom reported. diff --git a/gdb/testsuite/gdb.python/py-corefile.exp b/gdb/testsuite/gdb.python/py-corefile.exp index 3b57cc0e250..866b60a0dbe 100644 --- a/gdb/testsuite/gdb.python/py-corefile.exp +++ b/gdb/testsuite/gdb.python/py-corefile.exp @@ -211,8 +211,53 @@ with_test_prefix "test mapped files data" { "Files \[^\r\n\]+-out-1.txt and \[^\r\n\]+-out-2.txt are identical" \ "diff input and output one" - # Check build-ids within the core file mapping data. - gdb_test "check-build-ids" "^PASS" + # Check build-ids within the core file mapping data. I'm only + # aware of GNU/Linux placing the first page of each mapped ELF + # into the generated core file so that the build-id can be found. + if {[istarget *-*-linux*]} { + set results [list] + gdb_test_multiple "show-build-ids" "" { + -re "^show-build-ids\r\n" { + exp_continue + } + -re "^Objfile Build-Id\\s+Core File Build-Id\\s+File Name\\s*\r\n" { + exp_continue + } + -re "^(\\S+)\\s+(\\S+)\\s+(\[^\r\n\]+)\r\n" { + set objfile_build_id $expect_out(1,string) + set core_file_build_id $expect_out(2,string) + set file_name $expect_out(3,string) + lappend results [list $objfile_build_id \ + $core_file_build_id \ + $file_name] + exp_continue + } + -re "^$gdb_prompt " { + pass $gdb_test_name + } + } + + set bad_count 0 + foreach entry $results { + set objfile_build_id [lindex $entry 0] + set core_file_build_id [lindex $entry 1] + set file_name [lindex $entry 2] + if { $objfile_build_id ne $core_file_build_id } { + if { $core_file_build_id ne "None" } { + verbose -log "Mismatched build-ids $objfile_build_id vs $core_file_build_id for $file_name" + incr bad_count + } elseif { [expect_build_id_in_core_file $file_name] } { + verbose -log "Failed to find build-id for $file_name" + incr bad_count + } else { + verbose -log "This build-id was likely not in the core file" + } + } + } + + gdb_assert { $bad_count == 0 } \ + "found expected build-ids in core file" + } # Check the is_main_executable flag in the mapping data. gdb_test "check-main-executable" "^PASS" diff --git a/gdb/testsuite/gdb.python/py-corefile.py b/gdb/testsuite/gdb.python/py-corefile.py index cffd037a23b..5c8ee7b2f0c 100644 --- a/gdb/testsuite/gdb.python/py-corefile.py +++ b/gdb/testsuite/gdb.python/py-corefile.py @@ -78,9 +78,23 @@ class InfoProcPyMappings(gdb.Command): InfoProcPyMappings() -class CheckBuildIds(gdb.Command): +# Assume that a core file is currently loaded. +# +# Look through all the objfiles for the current inferior, and record +# any that have a build-id. +# +# Then look through the core file mapped files. Look for entries that +# correspond with the loaded objfiles. For these matching entries, +# capture the build-id extracted from the core file. +# +# Finally, print a table with the build-id from the objfile, the +# build-id from the core file, and the file name. +# +# This is then processed from the test script to check the build-ids +# match. +class ShowBuildIds(gdb.Command): def __init__(self): - gdb.Command.__init__(self, "check-build-ids", gdb.COMMAND_DATA) + gdb.Command.__init__(self, "show-build-ids", gdb.COMMAND_DATA) def invoke(self, args, from_tty): inf = gdb.selected_inferior() @@ -88,12 +102,17 @@ class CheckBuildIds(gdb.Command): path_to_build_id = {} + # Initial length based on column headings. + longest_build_id = 18 + for o in objfiles: if not o.is_file or o.build_id is None: continue p = pathlib.Path(o.filename).resolve() b = o.build_id - path_to_build_id[p] = b + path_to_build_id[p] = {"objfile": b} + if len(b) > longest_build_id: + longest_build_id = len(b) count = 0 core_mapped_files = inf.corefile.mapped_files() @@ -101,16 +120,39 @@ class CheckBuildIds(gdb.Command): p = pathlib.Path(m.filename).resolve() b = m.build_id + if b is not None and len(b) > longest_build_id: + longest_build_id = len(b) + if p in path_to_build_id: - count += 1 - assert path_to_build_id[p] == b, "build-id mismatch for %s" % p + path_to_build_id[p]["corefile"] = b - assert count > 0, "no mapped files checked" + format_str = ( + "%-" + str(longest_build_id) + "s %-" + str(longest_build_id) + "s %s" + ) - print("PASS") + def make_title(string, length=0): + if length > 0: + padding_len = length - len(string) + else: + padding_len = 0 + + padding = " " * padding_len + style = gdb.Style("title") + return style.apply(string) + padding + + print( + "%s %s %s" + % ( + make_title("Objfile Build-Id", longest_build_id), + make_title("Core File Build-Id", longest_build_id), + make_title("File Name"), + ) + ) + for p, b in path_to_build_id.items(): + print(format_str % (b["objfile"], b["corefile"], p)) -CheckBuildIds() +ShowBuildIds() class CheckMainExec(gdb.Command): diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index e07aab4cdd8..5dfcfefd7c1 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -11596,6 +11596,63 @@ proc section_get {exec section} { return $retval } +# Return true if we expect the build-id from FILENAME to be included +# in a core file. +# +# On GNU/Linux, when creating a core file, the kernel places the first +# page of an ELF into the core file. If the build-id is within that +# page then GDB can find the build-id from the core file. +# +# This proc checks that the target is GNU/Linux, and then uses readelf +# to find the offset of the build-id within the ELF. If there is a +# build-id, and it is within the first page, then return true. +# Otherwise, return false. + +proc expect_build_id_in_core_file { filename } { + # I'm not sure if other kernels take care to add the first page of + # each ELF into the core file. If they do then this test can be + # relaxed. + if {![istarget *-*-linux*]} { + return false + } + + # Use readelf to find the build-id note in FILENAME. + set readelf_program [gdb_find_readelf] + set cmd [list $readelf_program -WS $filename | grep ".note.gnu.build-id"] + set res [catch {exec {*}$cmd} output] + verbose -log "running: $cmd" + verbose -log "result: $res" + verbose -log "output: $output" + if { $res != 0 } { + return false + } + + # Extract the OFFSET from the readelf output. + set res [regexp {NOTE[ \t]+([0-9a-f]+)[ \t]+([0-9a-f]+)} \ + $output dummy addr offset] + if { $res != 1 } { + return false + } + + # Convert OFFSET to decimal. + set offset [expr 0x$offset + 0] + + # Now figure out the page size. This should be fine for Linux + # hosts, see the istarget check above. + if {[catch {exec getconf PAGESIZE} page_size]} { + # Failed to fetch page size. + return false + } + + # If the build-id is within the first page, then we expect the + # kernel to include it in the core file. There is actually a + # kernel setting (see coredump_filter) that could prevent this, + # but the default behaviour is to include the first page of the + # ELF, so for now, we just assume this is on. + verbose -log "Page size is $page_size, Offset is $offset" + return [expr $offset < $page_size] +} + # Return 1 if the compiler supports __builtin_trap, else return 0. gdb_caching_proc have_builtin_trap {} {