From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id mOKUNgvKmGKI6wkAWB0awg (envelope-from ) for ; Thu, 02 Jun 2022 10:32:43 -0400 Received: by simark.ca (Postfix, from userid 112) id DB5D51E221; Thu, 2 Jun 2022 10:32:43 -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=FDx5fb3/; 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=-3.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,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 0BDC91E01D for ; Thu, 2 Jun 2022 10:32:43 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id B08A4396E06A for ; Thu, 2 Jun 2022 14:32:42 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org B08A4396E06A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1654180362; bh=/yvglLMj46XSYvrHkUnr5kV+5gvusuS1dAG/x9IItCg=; h=To:Subject:Date:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=FDx5fb3/ijXiLvsCvSKLVfPHposORDxKhTKBuS9B1Ol10HnnCyQ9xbxemAtshmK2z ZkbMelmaGk+n7RUq+24A73Z+vk+eLcFvtDrPdj3vsx4HJdyon+Smzt6PGqYlutI7F4 cweqBS9js/+ivO+Wk1ZvGkoAJ2vljrEIqdqt7yFg= Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by sourceware.org (Postfix) with ESMTPS id 24B4B384B833 for ; Thu, 2 Jun 2022 14:29:11 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 24B4B384B833 X-IronPort-AV: E=McAfee;i="6400,9594,10365"; a="362337247" X-IronPort-AV: E=Sophos;i="5.91,271,1647327600"; d="scan'208";a="362337247" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2022 07:29:00 -0700 X-IronPort-AV: E=Sophos;i="5.91,271,1647327600"; d="scan'208";a="563331194" Received: from labpc2407.iul.intel.com (HELO localhost) ([172.28.50.61]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2022 07:28:59 -0700 To: gdb-patches@sourceware.org Subject: [PATCH v5 05/15] gdb, gdbserver: extend RSP to support namespaces Date: Thu, 2 Jun 2022 15:25:04 +0200 Message-Id: <20220602132514.957983-6-markus.t.metzger@intel.com> X-Mailer: git-send-email 2.35.3 In-Reply-To: <20220602132514.957983-1-markus.t.metzger@intel.com> References: <20220602132514.957983-1-markus.t.metzger@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit 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: Markus Metzger via Gdb-patches Reply-To: Markus Metzger Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" Introduce a new qXfer:libraries-svr4:read annex key/value pair lmid= to be used together with start and prev to provide the namespace of start and prev to gdbserver. Unknown key/value pairs are ignored by gdbserver so no new supports check is needed. Introduce a new library-list-svr4 library attribute lmid to provide the namespace of a library entry to GDB. This implementation uses the address of a namespace's r_debug object as namespace identifier. This should have incremented the minor version but since unknown XML attributes are ignored, anyway, and since changing the version results in a warning from GDB, the version is left at 1.0. --- gdb/doc/gdb.texinfo | 16 +++++++++++-- gdb/features/library-list-svr4.dtd | 4 ++++ gdb/solib-svr4.c | 38 +++++++++++++++++++++++++++--- gdbserver/linux-low.cc | 26 +++++++++++++------- 4 files changed, 70 insertions(+), 14 deletions(-) diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 5f09f3a1433..e2f430feb7e 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -43334,6 +43334,12 @@ specified by the @samp{start} argument. If unset or zero then the remote stub will expect that no @samp{struct link_map} exists prior to the starting point. +@item lmid=@var{lmid} +A hexadecimal number specifying a namespace identifier. This is +currently only used together with @samp{start} to provide the +namespace identifier back to @value{GDBN} in the response. +@value{GDBN} will only provide values that were previously reported to +it. If unset, the response will include @samp{lmid="0x0"}. @end table Arguments that are not understood by the remote stub will be silently @@ -45872,6 +45878,11 @@ memory address. It is a displacement of absolute memory address against address the file was prelinked to during the library load. @item @code{l_ld}, which is memory address of the @code{PT_DYNAMIC} segment +@item +@code{lmid}, which is an identifier for a linker namespace, such as +the memory address of the @code{r_debug} object that contains this +namespace's load map or the namespace identifier returned by +@code{dlinfo (3)}. @end itemize Additionally the single @code{main-lm} attribute specifies address of @@ -45887,9 +45898,9 @@ looks like this: @smallexample + l_ld="0xe4eefc" lmid="0xfffe0"/> + l_ld="0x152350" lmid="0xfffe0"/> @end smallexample @@ -45905,6 +45916,7 @@ The format of an SVR4 library list is described by this DTD: + @end smallexample @node Memory Map Format diff --git a/gdb/features/library-list-svr4.dtd b/gdb/features/library-list-svr4.dtd index 7d62354c34c..e869ff1349e 100644 --- a/gdb/features/library-list-svr4.dtd +++ b/gdb/features/library-list-svr4.dtd @@ -14,3 +14,7 @@ + + diff --git a/gdb/solib-svr4.c b/gdb/solib-svr4.c index 1f2abcf16ef..ca396acf24f 100644 --- a/gdb/solib-svr4.c +++ b/gdb/solib-svr4.c @@ -1068,8 +1068,37 @@ library_list_start_library (struct gdb_xml_parser *parser, new_elem->so_name[sizeof (new_elem->so_name) - 1] = 0; strcpy (new_elem->so_original_name, new_elem->so_name); - *list->tailp = new_elem; - list->tailp = &new_elem->next; + /* Older versions did not supply lmid. Put the element into the flat + list of the special namespace zero in that case. */ + gdb_xml_value *at_lmid = xml_find_attribute (attributes, "lmid"); + if (at_lmid == nullptr) + { + *list->tailp = new_elem; + list->tailp = &new_elem->next; + } + else + { + ULONGEST lmid = *(ULONGEST *) at_lmid->value.get (); + + /* Ensure that the element is actually initialized. */ + if (list->solib_lists.find (lmid) == list->solib_lists.end ()) + list->solib_lists[lmid] = nullptr; + + so_list **psolist = &list->solib_lists[lmid]; + so_list **pnext = psolist; + + /* Walk to the end of the list if we have one. */ + so_list *solist = *psolist; + if (solist != nullptr) + { + for (; solist->next != nullptr; solist = solist->next) + /* Nothing. */; + + pnext = &solist->next; + } + + *pnext = new_elem; + } } /* Handle the start of a element. */ @@ -1109,6 +1138,7 @@ static const struct gdb_xml_attribute svr4_library_attributes[] = { "lm", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL }, { "l_addr", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL }, { "l_ld", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL }, + { "lmid", GDB_XML_AF_NONE, gdb_xml_parse_attr_ulongest, NULL }, { NULL, GDB_XML_AF_NONE, NULL, NULL } }; @@ -1866,7 +1896,9 @@ solist_update_incremental (svr4_info *info, CORE_ADDR debug_base, struct svr4_library_list library_list; char annex[64]; - xsnprintf (annex, sizeof (annex), "start=%s;prev=%s", + /* Unknown key=value pairs are ignored by the gdbstub. */ + xsnprintf (annex, sizeof (annex), "lmid=%s;start=%s;prev=%s", + phex_nz (debug_base, sizeof (debug_base)), phex_nz (lm, sizeof (lm)), phex_nz (prev_lm, sizeof (prev_lm))); if (!svr4_current_sos_via_xfer_libraries (&library_list, annex)) diff --git a/gdbserver/linux-low.cc b/gdbserver/linux-low.cc index 4b4d38b75bc..f54551f2eca 100644 --- a/gdbserver/linux-low.cc +++ b/gdbserver/linux-low.cc @@ -6481,8 +6481,8 @@ static const link_map_offsets lmo_64bit_offsets = /* Get the loaded shared libraries from one namespace. */ static void -read_link_map (std::string &document, CORE_ADDR lm_addr, CORE_ADDR lm_prev, - int ptr_size, const link_map_offsets *lmo) +read_link_map (std::string &document, CORE_ADDR lmid, CORE_ADDR lm_addr, + CORE_ADDR lm_prev, int ptr_size, const link_map_offsets *lmo) { CORE_ADDR l_name, l_addr, l_ld, l_next, l_prev; @@ -6517,9 +6517,9 @@ read_link_map (std::string &document, CORE_ADDR lm_addr, CORE_ADDR lm_prev, string_appendf (document, "", + "l_ld=\"0x%s\" lmid=\"0x%s\"/>", paddress (lm_addr), paddress (l_addr), - paddress (l_ld)); + paddress (l_ld), paddress (lmid)); } lm_prev = lm_addr; @@ -6539,7 +6539,7 @@ linux_process_target::qxfer_libraries_svr4 (const char *annex, char filename[PATH_MAX]; int pid, is_elf64; unsigned int machine; - CORE_ADDR lm_addr = 0, lm_prev = 0; + CORE_ADDR lmid = 0, lm_addr = 0, lm_prev = 0; if (writebuf != NULL) return -2; @@ -6573,7 +6573,9 @@ linux_process_target::qxfer_libraries_svr4 (const char *annex, break; name_len = sep - annex; - if (name_len == 5 && startswith (annex, "start")) + if (name_len == 4 && startswith (annex, "lmid")) + addrp = &lmid; + else if (name_len == 5 && startswith (annex, "start")) addrp = &lm_addr; else if (name_len == 4 && startswith (annex, "prev")) addrp = &lm_prev; @@ -6592,19 +6594,25 @@ linux_process_target::qxfer_libraries_svr4 (const char *annex, std::string document = "r_debug; if (r_debug == 0) r_debug = priv->r_debug = get_r_debug (pid, is_elf64); @@ -6670,7 +6678,7 @@ linux_process_target::qxfer_libraries_svr4 (const char *annex, } } - read_link_map (document, lm_addr, lm_prev, ptr_size, lmo); + read_link_map (document, r_debug, lm_addr, lm_prev, ptr_size, lmo); if (r_version < 2) break; -- 2.35.3 Intel Deutschland GmbH Registered Address: Am Campeon 10, 85579 Neubiberg, Germany Tel: +49 89 99 8853-0, www.intel.de Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva Chairperson of the Supervisory Board: Nicole Lau Registered Office: Munich Commercial Register: Amtsgericht Muenchen HRB 186928