From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sa-prd-fep-041.btinternet.com (mailomta6-sa.btinternet.com [213.120.69.12]) by sourceware.org (Postfix) with ESMTPS id F0BDE3857007 for ; Wed, 1 Jul 2020 21:34:15 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org F0BDE3857007 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 sa-prd-rgout-001.btmx-prd.synchronoss.net ([10.2.38.4]) by sa-prd-fep-041.btinternet.com with ESMTP id <20200701213415.WVJU7327.sa-prd-fep-041.btinternet.com@sa-prd-rgout-001.btmx-prd.synchronoss.net>; Wed, 1 Jul 2020 22:34:15 +0100 Authentication-Results: btinternet.com; none X-Originating-IP: [31.51.206.31] X-OWM-Source-IP: 31.51.206.31 (GB) X-OWM-Env-Sender: jonturney@btinternet.com X-VadeSecure-score: verdict=clean score=0/300, class=clean X-RazorGate-Vade: gggruggvucftvghtrhhoucdtuddrgeduiedrtddvgdduieehucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuueftkffvkffujffvgffngfevqffopdfqfgfvnecuuegrihhlohhuthemuceftddunecunecujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpeflohhnucfvuhhrnhgvhicuoehjohhnrdhtuhhrnhgvhiesughrohhnvggtohguvgdrohhrghdruhhkqeenucggtffrrghtthgvrhhnpeefieduveehgfffffeuueehleefgeevfedvffeljeefheduteelteelvdettefhvdenucfkphepfedurdehuddrvddtiedrfedunecuvehluhhsthgvrhfuihiivgepheenucfrrghrrghmpehhvghloheplhhotggrlhhhohhsthdrlhhotggrlhguohhmrghinhdpihhnvghtpeefuddrhedurddvtdeirdefuddpmhgrihhlfhhrohhmpeeojhhonhdrthhurhhnvgihsegurhhonhgvtghouggvrdhorhhgrdhukheqpdhrtghpthhtohepoehguggsqdhprghttghhvghssehsohhurhgtvgifrghrvgdrohhrgheqpdhrtghpthhtohepoehjohhnrdhtuhhrnhgvhiesughrohhnvggtohguvgdrohhrghdruhhkqe X-RazorGate-Vade-Verdict: clean 0 X-RazorGate-Vade-Classification: clean Received: from localhost.localdomain (31.51.206.31) by sa-prd-rgout-001.btmx-prd.synchronoss.net (5.8.340) (authenticated as jonturney@btinternet.com) id 5ED99EC904C00995; Wed, 1 Jul 2020 22:34:15 +0100 From: Jon Turney To: gdb-patches@sourceware.org Cc: Jon Turney Subject: [PATCH 6/7] Promote windows_core_xfer_shared_libraries and windows_core_pid_to_str Date: Wed, 1 Jul 2020 22:32:24 +0100 Message-Id: <20200701213225.14144-7-jon.turney@dronecode.org.uk> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20200701213225.14144-1-jon.turney@dronecode.org.uk> References: <20200701213225.14144-1-jon.turney@dronecode.org.uk> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-10.2 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_H3, 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, 01 Jul 2020 21:34:17 -0000 Move windows_core_xfer_shared_libraries() and windows_core_pid_to_str() to windows-tdep, use in amd64-windows-tdep.c to handle Cygwin x86_64 core dumps. 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 f1526283f2f..89b3de15bce 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..9dae287554c 100644 --- a/gdb/windows-tdep.c +++ b/gdb/windows-tdep.c @@ -1120,3 +1120,101 @@ even if their meaning is unknown."), isn't another convenience variable of the same name. */ create_internalvar_type_lazy ("_tlb", &tlb_funcs, NULL); } + +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); +} 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.27.0