From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from re-prd-fep-042.btinternet.com (mailomta26-re.btinternet.com [213.120.69.119]) by sourceware.org (Postfix) with ESMTPS id 0774C384A023 for ; Wed, 12 Aug 2020 19:18:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 0774C384A023 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=dronecode.org.uk Authentication-Results: sourceware.org; spf=none smtp.mailfrom=jon.turney@dronecode.org.uk Received: from re-prd-rgout-002.btmx-prd.synchronoss.net ([10.2.54.5]) by re-prd-fep-042.btinternet.com with ESMTP id <20200812191847.FABA13627.re-prd-fep-042.btinternet.com@re-prd-rgout-002.btmx-prd.synchronoss.net>; Wed, 12 Aug 2020 20:18:47 +0100 Authentication-Results: btinternet.com; none X-Originating-IP: [31.51.206.146] X-OWM-Source-IP: 31.51.206.146 (GB) X-OWM-Env-Sender: jonturney@btinternet.com X-VadeSecure-score: verdict=clean score=0/300, class=clean X-RazorGate-Vade: gggruggvucftvghtrhhoucdtuddrgeduiedrledvgddufeehucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuueftkffvkffujffvgffngfevqffopdfqfgfvnecuuegrihhlohhuthemuceftddunecunecujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpeflohhnucfvuhhrnhgvhicuoehjohhnrdhtuhhrnhgvhiesughrohhnvggtohguvgdrohhrghdruhhkqeenucggtffrrghtthgvrhhnpeefieduveehgfffffeuueehleefgeevfedvffeljeefheduteelteelvdettefhvdenucfkphepfedurdehuddrvddtiedrudegieenucevlhhushhtvghrufhiiigvpedunecurfgrrhgrmhephhgvlhhopehlohgtrghlhhhoshhtrdhlohgtrghlughomhgrihhnpdhinhgvthepfedurdehuddrvddtiedrudegiedpmhgrihhlfhhrohhmpeeojhhonhdrthhurhhnvgihsegurhhonhgvtghouggvrdhorhhgrdhukheqpdhrtghpthhtohepoehguggsqdhprghttghhvghssehsohhurhgtvgifrghrvgdrohhrgheqpdhrtghpthhtohepoehjohhnrdhtuhhrnhgvhiesughrohhnvggtohguvgdrohhrghdruhhkqe X-RazorGate-Vade-Verdict: clean 0 X-RazorGate-Vade-Classification: clean Received: from localhost.localdomain (31.51.206.146) by re-prd-rgout-002.btmx-prd.synchronoss.net (5.8.340) (authenticated as jonturney@btinternet.com) id 5ED9C0CC0B80065A; Wed, 12 Aug 2020 20:18:47 +0100 From: Jon Turney To: gdb-patches@sourceware.org Cc: Jon Turney Subject: [PATCH 3/4] Promote windows_core_xfer_shared_libraries and windows_core_pid_to_str Date: Wed, 12 Aug 2020 20:18:15 +0100 Message-Id: <20200812191816.23246-4-jon.turney@dronecode.org.uk> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20200812191816.23246-1-jon.turney@dronecode.org.uk> References: <20200812191816.23246-1-jon.turney@dronecode.org.uk> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-9.5 required=5.0 tests=BAYES_00, FORGED_SPF_HELO, GIT_PATCH_0, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_PASS, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org 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: , X-List-Received-Date: Wed, 12 Aug 2020 19:18:49 -0000 Move windows_core_xfer_shared_libraries() and windows_core_pid_to_str() to windows-tdep, and use in amd64-windows-tdep.c to handle Cygwin x86_64 core dumps. v2: Keep _initialize function at the bottom of the file. gdb/ChangeLog: 2020-07-01 Jon Turney * windows-tdep.h: Add prototypes. * i386-windows-tdep.c(windows_core_xfer_shared_libraries): Move. (i386_windows_core_pid_to_str): Move and rename ... * windows-tdep.c (windows_core_xfer_shared_libraries): ... to here (windows_core_pid_to_str): ... and here. * amd64-windows-tdep.c (amd64_windows_init_abi_common): Register here. --- gdb/ChangeLog | 8 ++++ gdb/amd64-windows-tdep.c | 5 ++ gdb/i386-windows-tdep.c | 100 +-------------------------------------- gdb/windows-tdep.c | 98 ++++++++++++++++++++++++++++++++++++++ gdb/windows-tdep.h | 8 ++++ 5 files changed, 120 insertions(+), 99 deletions(-) diff --git a/gdb/amd64-windows-tdep.c b/gdb/amd64-windows-tdep.c index 7bb98bd2bd2..e427c205389 100644 --- a/gdb/amd64-windows-tdep.c +++ b/gdb/amd64-windows-tdep.c @@ -1310,6 +1310,11 @@ amd64_windows_init_abi_common (gdbarch_info info, struct gdbarch *gdbarch) tdep->sizeof_gregset = AMD64_WINDOWS_SIZEOF_GREGSET; tdep->sizeof_fpregset = 0; + /* Core file support. */ + set_gdbarch_core_xfer_shared_libraries + (gdbarch, windows_core_xfer_shared_libraries); + set_gdbarch_core_pid_to_str (gdbarch, windows_core_pid_to_str); + set_gdbarch_auto_wide_charset (gdbarch, amd64_windows_auto_wide_charset); } diff --git a/gdb/i386-windows-tdep.c b/gdb/i386-windows-tdep.c index ae61ed8291c..1477e54b4d5 100644 --- a/gdb/i386-windows-tdep.c +++ b/gdb/i386-windows-tdep.c @@ -89,104 +89,6 @@ static int i386_windows_gregset_reg_offset[] = #define I386_WINDOWS_SIZEOF_GREGSET 716 -struct cpms_data -{ - struct gdbarch *gdbarch; - struct obstack *obstack; - int module_count; -}; - -static void -core_process_module_section (bfd *abfd, asection *sect, void *obj) -{ - struct cpms_data *data = (struct cpms_data *) obj; - enum bfd_endian byte_order = gdbarch_byte_order (data->gdbarch); - - char *module_name; - size_t module_name_size; - CORE_ADDR base_addr; - - gdb_byte *buf = NULL; - - if (!startswith (sect->name, ".module")) - return; - - buf = (gdb_byte *) xmalloc (bfd_section_size (sect) + 1); - if (!buf) - { - printf_unfiltered ("memory allocation failed for %s\n", sect->name); - goto out; - } - if (!bfd_get_section_contents (abfd, sect, - buf, 0, bfd_section_size (sect))) - goto out; - - - - /* A DWORD (data_type) followed by struct windows_core_module_info. */ - - base_addr = - extract_unsigned_integer (buf + 4, 4, byte_order); - - module_name_size = - extract_unsigned_integer (buf + 8, 4, byte_order); - - if (12 + module_name_size > bfd_section_size (sect)) - goto out; - module_name = (char *) buf + 12; - - /* The first module is the .exe itself. */ - if (data->module_count != 0) - windows_xfer_shared_library (module_name, base_addr, - NULL, data->gdbarch, data->obstack); - data->module_count++; - -out: - xfree (buf); - return; -} - -static ULONGEST -windows_core_xfer_shared_libraries (struct gdbarch *gdbarch, - gdb_byte *readbuf, - ULONGEST offset, ULONGEST len) -{ - struct obstack obstack; - const char *buf; - ULONGEST len_avail; - struct cpms_data data = { gdbarch, &obstack, 0 }; - - obstack_init (&obstack); - obstack_grow_str (&obstack, "\n"); - bfd_map_over_sections (core_bfd, - core_process_module_section, - &data); - obstack_grow_str0 (&obstack, "\n"); - - buf = (const char *) obstack_finish (&obstack); - len_avail = strlen (buf); - if (offset >= len_avail) - return 0; - - if (len > len_avail - offset) - len = len_avail - offset; - memcpy (readbuf, buf + offset, len); - - obstack_free (&obstack, NULL); - return len; -} - -/* This is how we want PTIDs from core files to be printed. */ - -static std::string -i386_windows_core_pid_to_str (struct gdbarch *gdbarch, ptid_t ptid) -{ - if (ptid.lwp () != 0) - return string_printf ("Thread 0x%lx", ptid.lwp ()); - - return normal_pid_to_str (ptid); -} - static CORE_ADDR i386_windows_skip_trampoline_code (struct frame_info *frame, CORE_ADDR pc) { @@ -251,7 +153,7 @@ i386_windows_init_abi_common (struct gdbarch_info info, struct gdbarch *gdbarch) /* Core file support. */ set_gdbarch_core_xfer_shared_libraries (gdbarch, windows_core_xfer_shared_libraries); - set_gdbarch_core_pid_to_str (gdbarch, i386_windows_core_pid_to_str); + set_gdbarch_core_pid_to_str (gdbarch, windows_core_pid_to_str); set_gdbarch_auto_wide_charset (gdbarch, i386_windows_auto_wide_charset); } diff --git a/gdb/windows-tdep.c b/gdb/windows-tdep.c index aa0adeba99b..037958a4bf5 100644 --- a/gdb/windows-tdep.c +++ b/gdb/windows-tdep.c @@ -1088,6 +1088,104 @@ range [0x%" BFD_VMA_FMT "x, 0x%" BFD_VMA_FMT "x[."), return false; } +struct cpms_data +{ + struct gdbarch *gdbarch; + struct obstack *obstack; + int module_count; +}; + +static void +core_process_module_section (bfd *abfd, asection *sect, void *obj) +{ + struct cpms_data *data = (struct cpms_data *) obj; + enum bfd_endian byte_order = gdbarch_byte_order (data->gdbarch); + + char *module_name; + size_t module_name_size; + CORE_ADDR base_addr; + + gdb_byte *buf = NULL; + + if (!startswith (sect->name, ".module")) + return; + + buf = (gdb_byte *) xmalloc (bfd_section_size (sect) + 1); + if (!buf) + { + printf_unfiltered ("memory allocation failed for %s\n", sect->name); + goto out; + } + if (!bfd_get_section_contents (abfd, sect, + buf, 0, bfd_section_size (sect))) + goto out; + + + + /* A DWORD (data_type) followed by struct windows_core_module_info. */ + + base_addr = + extract_unsigned_integer (buf + 4, 4, byte_order); + + module_name_size = + extract_unsigned_integer (buf + 8, 4, byte_order); + + if (12 + module_name_size > bfd_section_size (sect)) + goto out; + module_name = (char *) buf + 12; + + /* The first module is the .exe itself. */ + if (data->module_count != 0) + windows_xfer_shared_library (module_name, base_addr, + NULL, data->gdbarch, data->obstack); + data->module_count++; + +out: + xfree (buf); + return; +} + +ULONGEST +windows_core_xfer_shared_libraries (struct gdbarch *gdbarch, + gdb_byte *readbuf, + ULONGEST offset, ULONGEST len) +{ + struct obstack obstack; + const char *buf; + ULONGEST len_avail; + struct cpms_data data = { gdbarch, &obstack, 0 }; + + obstack_init (&obstack); + obstack_grow_str (&obstack, "\n"); + bfd_map_over_sections (core_bfd, + core_process_module_section, + &data); + obstack_grow_str0 (&obstack, "\n"); + + buf = (const char *) obstack_finish (&obstack); + len_avail = strlen (buf); + if (offset >= len_avail) + return 0; + + if (len > len_avail - offset) + len = len_avail - offset; + memcpy (readbuf, buf + offset, len); + + obstack_free (&obstack, NULL); + return len; +} + +/* This is how we want PTIDs from core files to be printed. */ + +std::string +windows_core_pid_to_str (struct gdbarch *gdbarch, ptid_t ptid) +{ + if (ptid.lwp () != 0) + return string_printf ("Thread 0x%lx", ptid.lwp ()); + + return normal_pid_to_str (ptid); +} + void _initialize_windows_tdep (); void _initialize_windows_tdep () diff --git a/gdb/windows-tdep.h b/gdb/windows-tdep.h index cd7717bd917..ec677cbdd46 100644 --- a/gdb/windows-tdep.h +++ b/gdb/windows-tdep.h @@ -31,6 +31,14 @@ extern void windows_xfer_shared_library (const char* so_name, struct gdbarch *gdbarch, struct obstack *obstack); +extern ULONGEST windows_core_xfer_shared_libraries (struct gdbarch *gdbarch, + gdb_byte *readbuf, + ULONGEST offset, + ULONGEST len); + +extern std::string windows_core_pid_to_str (struct gdbarch *gdbarch, + ptid_t ptid); + /* To be called from the various GDB_OSABI_WINDOWS handlers for the various Windows architectures and machine types. */ -- 2.28.0