From patchwork Fri May 24 15:40:24 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Ijaz, Abdul B" X-Patchwork-Id: 90824 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 86C743857C4F for ; Fri, 24 May 2024 15:41:43 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) by sourceware.org (Postfix) with ESMTPS id BE851384609A for ; Fri, 24 May 2024 15:41:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org BE851384609A Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=intel.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org BE851384609A Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=192.198.163.9 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1716565267; cv=none; b=kcfwNKGulGagnka51K18EKedCytGxDtIM0Xr5VGl6WyK+pTo33+L1kltls9Mhuycf18g7Sqco1zPHgdppM84iz4VrajyfxHepJiHn+7TZBvRPAUIYsvKfxh2CPeBr9C0DZdnIr6evqtpEfXQRaI5nIvqgbZM+K3xE9aTSllSt6o= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1716565267; c=relaxed/simple; bh=0wPTDcxnFGXAJnm4xY33FAyqrLrlV8V4CyRpK6bnLtg=; h=DKIM-Signature:From:To:Subject:Date:Message-Id:MIME-Version; b=Gl3DdUOcfedcCmS5+5D/MhCJrgNEYgLk8ided8pqggvGGXtBgvKNeFwwBpBc5u7pg9Gn/2WwXtDheSYVYvDv4U57K+G4RbWlwaBy5crJm6QXGPP5HwRcbQ+dGOxxh+jyrPM/MSBCr2pXrgPlDp2gEm8RUedAWU70AwtwGQUlTc4= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1716565265; x=1748101265; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=0wPTDcxnFGXAJnm4xY33FAyqrLrlV8V4CyRpK6bnLtg=; b=OiN+VbLKoeKvJJ4w7K2PfReVCs2UQi5usUyJZ8eCVliXvBztQ/vDwg18 S8MEvS9+j9l7vr1i0Xy3r0KP+b97LvHjP0pjC7vYhMier6IuyARnaTYNO W1pos0KWKBj+73wIe59pEIpcERh0Uq1uG3/5Du9vN3XRJaCLk879KimsU B3M6IoxNjzRKYdF7tCaW6t/5b7j7oHpDKVjshedwIxdbCF7MxB5RlUMvs UxI2+Hy9Fhkso31b0WFVegGo58EuGnTP9EryufHyMRKwJXP3/3aLrQP5K ghsamrht/KkV0u9VNo5aT/dnFnf/WuKye4sRQMypJ2thgLLPsYAIXt/+3 Q==; X-CSE-ConnectionGUID: xmX58GllQ9aiJGO6HtbWQQ== X-CSE-MsgGUID: r26B3E7XQRenNTV93s2Q/g== X-IronPort-AV: E=McAfee;i="6600,9927,11082"; a="23612584" X-IronPort-AV: E=Sophos;i="6.08,185,1712646000"; d="scan'208";a="23612584" Received: from fmviesa010.fm.intel.com ([10.60.135.150]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 May 2024 08:41:04 -0700 X-CSE-ConnectionGUID: tEUDSaDAR8qXz5nOE17e2Q== X-CSE-MsgGUID: gSKXa6M0SIiNq5cjm/NVYg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,185,1712646000"; d="scan'208";a="34164747" Received: from abijaz-mobl2.ger.corp.intel.com (HELO localhost) ([10.94.253.168]) by fmviesa010-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 May 2024 08:41:01 -0700 From: Abdul Basit Ijaz To: gdb-patches@sourceware.org Cc: abdul.b.ijaz@intel.com, JiniSusan.George@amd.com, tom@tromey.com, eliz@gnu.org, Nils-Christian Kempke Subject: [PATCH v8 02/10] gdb/symtab: add lookup for trampoline functions Date: Fri, 24 May 2024 17:40:24 +0200 Message-Id: <20240524154032.12652-3-abdul.b.ijaz@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240524154032.12652-1-abdul.b.ijaz@intel.com> References: <20240524154032.12652-1-abdul.b.ijaz@intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-10.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.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: Nils-Christian Kempke In order to query information about the DW_AT_trampoline tag for subroutines and inlined subroutines, two function were added to symtab. First, a routine for querying whether the given pc belongs to a block that is associated with a function (maybe inlined) marked DW_AT_trampoline. Second, a routine for querying a trampoline function's target. Subroutines and inlined subroutines marked with DW_AT_trampoline usually contain information about the target subroutine they are 'wrapping'/ passing control to. 2024-05-24 Nils-Christian Kempke --- gdb/symtab.c | 69 ++++++++++++++++++++++++++++++++++++++++++++++++++++ gdb/symtab.h | 14 +++++++++++ 2 files changed, 83 insertions(+) diff --git a/gdb/symtab.c b/gdb/symtab.c index 58648a8779d..5eac2ee8535 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -73,6 +73,7 @@ #include "gdbsupport/pathstuff.h" #include "gdbsupport/common-utils.h" #include +#include "gdbsupport/symbol.h" /* Forward declarations for local functions. */ @@ -4286,6 +4287,74 @@ find_function_alias_target (bound_minimal_symbol msymbol) return NULL; } +/* See symtab.h. */ + +bool +in_trampoline_function (CORE_ADDR pc) +{ + /* Find the innermost function containing pc. This might be an inlined + function. */ + symbol *sym = find_pc_sect_containing_function (pc, + find_pc_mapped_section (pc)); + return sym != nullptr && TYPE_IS_TRAMPOLINE (sym->type ()); +} + +/* See symtab.h. */ + +CORE_ADDR +find_function_trampoline_target (CORE_ADDR pc) +{ + /* Find the innermost function containing pc. This might be an inlined + function. */ + symbol *sym = find_pc_sect_containing_function (pc, + find_pc_mapped_section (pc)); + CORE_ADDR target_address = 0; + + if (sym != nullptr && TYPE_IS_TRAMPOLINE (sym->type ())) + { + trampoline_target *trampoline = TYPE_TRAMPOLINE_TARGET (sym->type ()); + + /* DW_AT_trampoline can be given as an address, name, or flag here (die + references have been resolved as names at this point. In the case + where DW_AT_trampoline contains a flag we do not know the target + address and return 0. */ + if (trampoline->target_kind () == TRAMPOLINE_TARGET_NAME) + { + /* Handle both the mangled and demangled PHYSNAME. */ + const char *physname = trampoline->target_name (); + + /* First, check whether there exists a symbol matching the + physname. If we cannot find one also check for minimal + symbols. */ + const block *blk = block_for_pc (pc); + struct block_symbol bs = \ + lookup_symbol (physname, blk, SEARCH_VAR_DOMAIN, 0); + if (bs.symbol != nullptr) + { + const struct block *block = bs.symbol->value_block (); + gdb_assert (block != nullptr); + target_address = block->start (); + } + else + { + if (find_minimal_symbol_address (physname, &target_address, + nullptr) != 0) + target_address = 0; + } + } + else if (trampoline->target_kind () == TRAMPOLINE_TARGET_ADDR) + { + /* If the function symbol containing this trampoline target has + been relocated we assume the target_address also needs relocation. + If it has not been relocated the offset should be zero. */ + target_address = \ + ( (CORE_ADDR) trampoline->target_addr () + + sym->objfile ()->section_offsets[sym->section_index ()]); + } + } + + return target_address; +} /* If P is of the form "operator[ \t]+..." where `...' is some legitimate operator text, return a pointer to the diff --git a/gdb/symtab.h b/gdb/symtab.h index bf9a3cfb79f..c60485dc6d3 100644 --- a/gdb/symtab.h +++ b/gdb/symtab.h @@ -2354,6 +2354,20 @@ extern const struct gnu_ifunc_fns *gnu_ifunc_fns_p; extern CORE_ADDR find_solib_trampoline_target (const frame_info_ptr &, CORE_ADDR); +/* Return whether or not the current pc is within a block that belongs to a + function that is marked as a trampoline by the compiler. */ + +extern bool in_trampoline_function (CORE_ADDR pc); + +/* Find the target of a trampoline function marked via the DW_AT_trampoline + attribute and return its address. Returns 0 if the pc is not contained + in a trampoline function (inlined or not). If DW_AT_trampoline + is given as a flag, the target is unknown and the function will still return + 0. One has to additionally query in_trampoline_function to cover this + case. */ + +extern CORE_ADDR find_function_trampoline_target (CORE_ADDR pc); + struct symtab_and_line { /* The program space of this sal. */