From patchwork Wed Mar 18 13:19:10 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Aditya Vidyadhar Kamath X-Patchwork-Id: 131901 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from vm01.sourceware.org (localhost [127.0.0.1]) by sourceware.org (Postfix) with ESMTP id A79574B196B9 for ; Wed, 18 Mar 2026 13:20:59 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org A79574B196B9 Authentication-Results: sourceware.org; dkim=pass (2048-bit key, unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20230601 header.b=ewscJvgm X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mail-pf1-x42f.google.com (mail-pf1-x42f.google.com [IPv6:2607:f8b0:4864:20::42f]) by sourceware.org (Postfix) with ESMTPS id C4DE34BBC0E2 for ; Wed, 18 Mar 2026 13:19:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org C4DE34BBC0E2 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org C4DE34BBC0E2 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2607:f8b0:4864:20::42f ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1773839988; cv=none; b=AiMCSsnDJnTGe9611h9oOcLD8OaQfiBdCztRFneqPv/RJL6xSy8jIrOLIb1bVcNGR+cH2NLps8y8FsdKBSFAVeWB+3aWMeCssjtyjNpJKSvPxrslK9g2e1Y0RJKfINYm7OhoWk6hE8I6be1qsi7RRafURV0blHYdouSN33fSy+M= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1773839988; c=relaxed/simple; bh=oezt/ZwLvBZz8KsAs7pwsuHk/1TqWCQvEYG+zDnL+U8=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=DcPhi9auEKK/wswddOorpzwWzNv9NoOQUPWhy2vQeKQTericMoJHnNEwJ5brAsXAV26cUCZYKlU9P69+Ell+OB/43DOHhs3Kpg0akGQ4U5ltCNwYbMSAMy4DMTZfHpLowjwJMK/bQ9FZJ9O4kbyYMBmJmhDYVLQ0qP/QG5d8wBY= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C4DE34BBC0E2 Received: by mail-pf1-x42f.google.com with SMTP id d2e1a72fcca58-82a124f3a5bso3960756b3a.0 for ; Wed, 18 Mar 2026 06:19:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773839987; x=1774444787; darn=sourceware.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=p/INHPmUKIzVyWmy8AtJTj7SV+gF8lLs5wfbBZvgHBU=; b=ewscJvgmSSP0bUtoQb6KMMvsrAglcttpQ6BPwKZpQtKv18tIhi17p7k1hJYSjeaC+0 INHpYYYTS5wDKQZgh7GjwLmTBGARknlr74prWCUMUYus7cFABijiAQE3S8PvBw1Wys0n 1aN992teJgRxFZZqkFTooZOV9DaZklhoqJWvwzGSn5hk0bwjeQZ7TgeTq6n+OMIhjCVz Mw13wK+gp6+J/Sw1Xay940zt0gNgfrW1CxK2aJszPFPDpvxGGi9lYCqQXUn7Hmk3++h/ 8PTmTuf7H3c2X5B0ZiQXcAv2Q2/v5O6HEQQ3SUS0YI5yQPjWu/AeK3GXQbWRt6FZdXDJ /hbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773839987; x=1774444787; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=p/INHPmUKIzVyWmy8AtJTj7SV+gF8lLs5wfbBZvgHBU=; b=JTyawXIFz3/1xAGA1rs2UiIHE+uVi/OtWVAjt/+X0/kOuUHLRw1R+fpYgmHKP1RzqF ztndkRsYEHcrQl15cat9iwB2Z1MiNH3HAxAve62/Sm6lOscghW76lcQgXVDv1oyMKnvB Nx/6EgqOHbqUnxIsXTGFkvRrIfL3LuqI36S8gEJp3rE8chaeaeEX4tOOWgrVndF83V+j U+IQkKFd1jH9IKJy2bjZIBYQq06SlwoLrHmjsQauR/cs+sKhpt41MlgfyVHt0HlvCRUQ pw9VlSzdGD3xWAAmYLRrp13QughdukSBfI+JiCo5F093R5DmgihCpK6OfaxdPSblEWXX C6Ig== X-Gm-Message-State: AOJu0YztK7ZbsDTwRd1aYzhwLesizsgLeZuCYOl/7irQxHWAJsJIlHEb wuggnQDo/CNSaWIZGmA9pt2aRRfK1TvKIWJk6wqkvxuc5ij8IAYZrZf2 X-Gm-Gg: ATEYQzyEN3o0jIED/PalB25rKeGT3meXAzKqBZ4fzm0HyxuG36rxiDqPourHhDyAe+W sCo8Ff2qA5Ed7jt3fVHeDRZrV7nBQ+tU0wZaM59/Nx8q2McB++ypMP8kQm3Sc34XrW0RaX1dXTX 27gzrKHpreQF/J/AejAKZjSvpO9p4RG5foPLA1y1thyNR9J84IJ0AxK6gLAEW9xtWoo68nnm08i jG/SjaS+t5NFsnFanMXbj75+Kk0IYWz8+DTniLmbI4x01mlnl7EzfSPxsbq5giayBzl0ALMM1Wg mtQT9Ap1X/xVNyKVxiCkVw5g6QpPaAAiTIVTliaYjKiSnNOieMe0OE/yp1VbmWF04HHCXxp+ald aogwkPGPxc4CvJLytIuHjgIiMUud1dmL54+RJQ2UbCuqGbXJg5VA4DDJfLdcsvtnkLjPqi0EHp5 /CUlpzN3VRvReuFHJav63p0nFflx1I+Vreg3NzqjtU8/l7hBuaMESJYw== X-Received: by 2002:a05:6a00:18a9:b0:7e8:4398:b34f with SMTP id d2e1a72fcca58-82a6aee1883mr3225739b3a.34.1773839986550; Wed, 18 Mar 2026 06:19:46 -0700 (PDT) Received: from localhost.localdomain ([129.41.58.2]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82a6bbb2f41sm2608832b3a.33.2026.03.18.06.19.43 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Wed, 18 Mar 2026 06:19:45 -0700 (PDT) From: Aditya Vidyadhar Kamath To: ulrich.weigand@de.ibm.com, simon.marchi@polymtl.ca, tom@tromey.com Cc: gdb-patches@sourceware.org, Aditya.Kamath1@ibm.com, sangamesh.swamy@in.ibm.com, Aditya Vidyadhar Kamath Subject: [PATCH] Add support for recording minimal symbols in AIX for non DWARF compiled library function record. Date: Wed, 18 Mar 2026 18:49:10 +0530 Message-ID: <20260318131909.6066-2-akamath996@gmail.com> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-Spam-Status: No, score=-11.1 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, GB_FREEMAIL_NUM, GIT_PATCH_0, KAM_ASCII_DIVIDERS, KAM_SHORT, RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE, SPF_PASS, TXREP, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on sourceware.org 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~patchwork=sourceware.org@sourceware.org From: Aditya Vidyadhar Kamath Commit 1dfd89c739 (gdb: Remove stabs support from XCOFF inferiors) removed STABS handling from xcoffread.c. While removing full symtab and compunit_symtab construction is appropriate when DWARF is available, the change also removed recording of minimal symbols, which still provide value for XCOFF binaries that lack DWARF. On AIX, system libraries such as libc and libpthread are built with STABS debug information up to AIX 7.3 TL4. As a result, current GDB master is unable to record even minimal symbols for these libraries, leading to missing function names (e.g. printf) in common debugging scenarios. Consider the example below, # cat ~/gdb_tests/main.c #include #include "libx.h" int main() { printf ("lib_func() = %d \n", lib_func()); return 0; } # cat ~/gdb_tests/libx.c int g_in_lib = 777; int lib_func() { return g_in_lib + 1; } # cat ~/gdb_tests/libx.h int lib_func(); We build libx shared library and then link it to main. When we compile this in GDB in AIX today in master branch we get, Reading symbols from //gdb_tests/main... (gdb) b main Breakpoint 1 at 0x10000530: file //gdb_tests/main.c, line 5. (gdb) r Starting program: /gdb_tests/main Breakpoint 1, main () at //gdb_tests/main.c:5 5 printf ("lib_func() = %d \n", lib_func()); (gdb) call printf() No symbol "printf" in current context. (gdb) The issue is Without minimal symbols: - Functions from system libraries are not visible in backtraces. - Calls such as call printf() fail with no symbol in current context. - This affects not only stepping, but also basic usability of backtraces on AIX systems without DWARF-enabled system libraries. - Even when full debug information is unavailable, minimal symbols are still sufficient to: - Identify function entry/exit points. - Display meaningful backtraces involving system libraries. This patch restores recording of minimal symbols in xcoffread.c, while keeping the removal of: - STABS-based symtabs - compunit_symtabs The below function (read_xcoff_minimal_symbols) ensures that: - record_minimal_symbol() is still invoked where appropriate. - No attempt is made to rebuild full debug information from STABS. - DWARF remains the sole source for full debugging information when available. This keeps the original intent of the STABS removal intact, while restoring essential functionality for non-DWARF XCOFF binaries. After applying the patch we get, GNU gdb (GDB) 18.0.50.20260316-git Copyright (C) 2026 Free Software Foundation, Inc. License GPLv3+: GNU GPL version 3 or later This is free software: you are free to change and redistribute it. There is NO WARRANTY, to the extent permitted by law. Type "show copying" and "show warranty" for details. This GDB was configured as "powerpc64-ibm-aix7.2.0.0". Type "show configuration" for configuration details. For bug reporting instructions, please see: . +------------------------------------------------------------------------------+ | Find the GDB manual online at: | | http://www.gnu.org/software/gdb/documentation/. | | For help, type "help". | | Type "apropos word" to search for commands related to "word". | +------------------------------------------------------------------------------+ Reading symbols from //gdb_tests/main... (gdb) b main Breakpoint 1 at 0x10000530: file //gdb_tests/main.c, line 5. (gdb) n The program is not being run. (gdb) r Starting program: /gdb_tests/main Breakpoint 1, main () at //gdb_tests/main.c:5 5 printf ("lib_func() = %d \n", lib_func()); (gdb) call (int)printf ("lib_func() = %d \n", lib_func()) lib_func() = 778 $1 = 18 (gdb) --- gdb/xcoffread.c | 97 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 97 insertions(+) diff --git a/gdb/xcoffread.c b/gdb/xcoffread.c index aec959e1375..848d1d39be7 100644 --- a/gdb/xcoffread.c +++ b/gdb/xcoffread.c @@ -28,6 +28,7 @@ #include "xcoffread.h" #include "symtab.h" +#include "minsyms.h" #include "gdbtypes.h" #include "symfile.h" #include "objfiles.h" @@ -232,6 +233,97 @@ xcoff_get_toc_offset (objfile *objfile) return 0; } +/* This function is used to read minimal sysmbols from the XCOFF STABS + symbol table. This function needs to be called when a library (like + libc) in AIX is compiled using xlc whose debug format is STABS. + When STABS debug information is not available, then in the backtrace + or other GDB features the STABS compiled functions like (printf) + will not be seen. Though we no longer support STABS debug, this + function records minimal symbol or information needed to show + such functions in backtrace. */ + +static void +read_xcoff_minimal_symbols (objfile *objfile) +{ + bfd *abfd = objfile->obfd.get (); + long do_we_need_storage = bfd_get_symtab_upper_bound (abfd); + asymbol **sym_table = NULL; + int number_of_symbols; + struct minimal_symbol_reader reader (objfile); + + do_we_need_storage = bfd_get_symtab_upper_bound (abfd); + if (do_we_need_storage <= 0) + return; + + sym_table = (asymbol **) xmalloc (do_we_need_storage); + /* Get the number of symbols we need. */ + number_of_symbols = bfd_canonicalize_symtab (abfd, sym_table); + + /* Return on error. */ + if (number_of_symbols < 0) + { + xfree (sym_table); + return; + } + + for (int i = 0; i < number_of_symbols; i++) + { + asymbol *sym = sym_table[i]; + flagword sym_flags = sym->flags; + CORE_ADDR sym_value = bfd_asymbol_value (sym); + asection *sym_section = bfd_asymbol_section (sym); + enum minimal_symbol_type ms_type = mst_unknown; + const char *sym_name = bfd_asymbol_name (sym); + + /* Skip undefined sections if any. */ + if (sym_section == bfd_und_section_ptr) + continue; + + /* Skip undefined, special symbols and debugging symbols. */ + if (sym_flags & (BSF_DEBUGGING | BSF_SECTION_SYM)) + continue; + + /* We need to pass ms_type when we record. So find the symbol type. */ + + /* If this is an object file. */ + if (sym_flags & BSF_OBJECT) + { + /* Code section. */ + if (sym_section && (bfd_section_flags (sym_section) & SEC_CODE)) + ms_type = mst_text; + /* Data section. */ + else if (sym_section && (bfd_section_flags (sym_section) & SEC_DATA)) + ms_type = mst_data; + /* BSS section. */ + else if (sym_section && (bfd_section_flags (sym_section) & SEC_ALLOC)) + ms_type = mst_bss; + else + ms_type = mst_unknown; + } + /* Check for symbols marked as functions explicitly. */ + else if (sym_flags & BSF_FUNCTION) + ms_type = mst_text; + /* Section Based Guess. Code or Data or BSS. */ + else if (sym_section) + { + if (bfd_section_flags (sym_section) & SEC_CODE) + ms_type = mst_text; + else if (bfd_section_flags (sym_section) & SEC_DATA) + ms_type = mst_data; + else if (bfd_section_flags (sym_section) & SEC_ALLOC) + ms_type = mst_bss; + } + + /* Add the symbol if it's global or weak */ + if ((sym_flags & (BSF_GLOBAL | BSF_WEAK)) && sym_name && *sym_name) + reader.record_with_info (sym_name, unrelocated_addr (sym_value), ms_type, + gdb_bfd_section_index (abfd, sym_section)); + } + + reader.install (); + xfree (sym_table); +} + /* Read the XCOFF symbol table. The only thing we are interested in is the TOC offset value. */ @@ -240,6 +332,11 @@ xcoff_symfile_read (objfile *objfile, symfile_add_flags symfile_flags) { xcoff_find_toc_offset (objfile); + /* Read minimal symbols from the STABS symbol table of an XCOFF binary + This makes sure we see functions from libc given libraries link to + it and libc in AIX may not be DWARF compiled. . */ + read_xcoff_minimal_symbols (objfile); + /* DWARF2 sections. */ dwarf2_initialize_objfile (objfile, &dwarf2_xcoff_names); }