From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by sourceware.org (Postfix) with ESMTPS id 85867383F85E for ; Tue, 16 Jun 2020 09:51:07 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 85867383F85E IronPort-SDR: i+LW0bQNdzKIOX6xvpnV56LYGOUHkV4VTXcPgoObhfGx0DRkvvyl8GRc0ScNID4P378oacjGIF +Ijo5dfBZO6Q== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Jun 2020 02:51:05 -0700 IronPort-SDR: CvkRs9IMpVNV+g1r/ekjKgnTApDkKLMPpLFor+yavOOMSTydt7+o89TrR5leiuwiYiriJWW2Xe gDrdLgfxDhgg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,518,1583222400"; d="scan'208";a="298847389" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by fmsmga004.fm.intel.com with ESMTP; 16 Jun 2020 02:51:04 -0700 Received: from ulvlx001.iul.intel.com (ulvlx001.iul.intel.com [172.28.207.17]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id 05G9p3pA020492; Tue, 16 Jun 2020 10:51:03 +0100 Received: from ulvlx001.iul.intel.com (localhost [127.0.0.1]) by ulvlx001.iul.intel.com with ESMTP id 05G9p3CC004966; Tue, 16 Jun 2020 11:51:03 +0200 Received: (from taktemur@localhost) by ulvlx001.iul.intel.com with LOCAL id 05G9p3t9004962; Tue, 16 Jun 2020 11:51:03 +0200 From: Tankut Baris Aktemur To: gdb-patches@sourceware.org Cc: simark@simark.ca Subject: [PATCH v2 1/3] gdb/jit: pass the jiter objfile as an argument to jit_event_handler Date: Tue, 16 Jun 2020 11:49:43 +0200 Message-Id: X-Mailer: git-send-email 1.7.0.7 In-Reply-To: References: In-Reply-To: References: X-Spam-Status: No, score=-23.3 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_NONE, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KAM_STOCKGEN, SPF_HELO_NONE, 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: Tue, 16 Jun 2020 09:51:09 -0000 This is a refactoring that adds a new parameter to the `jit_event_handler` function: the JITer objfile. The goal is to distinguish which JITer triggered the JIT event, in case there are multiple JITers -- a capability that is added in a subsequent patch. gdb/ChangeLog: 2020-06-15 Tankut Baris Aktemur * jit.h: Include "objfiles.h". (jit_event_handler): Add a second parameter, the JITer objfile. * breakpoint.c (handle_jit_event): Update the call to jit_event_handler to pass the JITer objfile. * jit.c (jit_read_descriptor): Change the signature to take the JITer objfile as an argument instead of the jit_program_space_data. (jit_inferior_init): Update the call to jit_read_descriptor. (jit_event_handler): Use the new JITer objfile argument when calling jit_read_descriptor. --- gdb/breakpoint.c | 3 ++- gdb/jit.c | 27 +++++++++++++++------------ gdb/jit.h | 4 +++- 3 files changed, 20 insertions(+), 14 deletions(-) diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index aead882acd8..da7e910dfc2 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -5448,8 +5448,9 @@ handle_jit_event (void) frame = get_current_frame (); gdbarch = get_frame_arch (frame); + objfile *jiter = symbol_objfile (get_frame_function (frame)); - jit_event_handler (gdbarch); + jit_event_handler (gdbarch, jiter); target_terminal::inferior (); } diff --git a/gdb/jit.c b/gdb/jit.c index 1b5ef46469e..1bfc80455b9 100644 --- a/gdb/jit.c +++ b/gdb/jit.c @@ -332,9 +332,9 @@ get_jit_program_space_data () memory. Returns 1 if all went well, 0 otherwise. */ static int -jit_read_descriptor (struct gdbarch *gdbarch, - struct jit_descriptor *descriptor, - struct jit_program_space_data *ps_data) +jit_read_descriptor (gdbarch *gdbarch, + jit_descriptor *descriptor, + objfile *jiter) { int err; struct type *ptr_type; @@ -344,16 +344,16 @@ jit_read_descriptor (struct gdbarch *gdbarch, enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); struct jit_objfile_data *objf_data; - if (ps_data->objfile == NULL) - return 0; - objf_data = get_jit_objfile_data (ps_data->objfile); + gdb_assert (jiter != nullptr); + + objf_data = get_jit_objfile_data (jiter); if (objf_data->descriptor == NULL) return 0; if (jit_debug) fprintf_unfiltered (gdb_stdlog, "jit_read_descriptor, descriptor_addr = %s\n", - paddress (gdbarch, MSYMBOL_VALUE_ADDRESS (ps_data->objfile, + paddress (gdbarch, MSYMBOL_VALUE_ADDRESS (jiter, objf_data->descriptor))); /* Figure out how big the descriptor is on the remote and how to read it. */ @@ -363,7 +363,7 @@ jit_read_descriptor (struct gdbarch *gdbarch, desc_buf = (gdb_byte *) alloca (desc_size); /* Read the descriptor. */ - err = target_read_memory (MSYMBOL_VALUE_ADDRESS (ps_data->objfile, + err = target_read_memory (MSYMBOL_VALUE_ADDRESS (jiter, objf_data->descriptor), desc_buf, desc_size); if (err) @@ -1255,9 +1255,13 @@ jit_inferior_init (struct gdbarch *gdbarch) if (jit_breakpoint_re_set_internal (gdbarch, ps_data) != 0) return; + /* There must be a JITer registered, otherwise we would exit early + above. */ + objfile *jiter = ps_data->objfile; + /* Read the descriptor so we can check the version number and load any already JITed functions. */ - if (!jit_read_descriptor (gdbarch, &descriptor, ps_data)) + if (!jit_read_descriptor (gdbarch, &descriptor, jiter)) return; /* Check that the version number agrees with that we support. */ @@ -1330,7 +1334,7 @@ jit_inferior_exit_hook (struct inferior *inf) } void -jit_event_handler (struct gdbarch *gdbarch) +jit_event_handler (gdbarch *gdbarch, objfile *jiter) { struct jit_descriptor descriptor; struct jit_code_entry code_entry; @@ -1338,8 +1342,7 @@ jit_event_handler (struct gdbarch *gdbarch) struct objfile *objf; /* Read the descriptor from remote memory. */ - if (!jit_read_descriptor (gdbarch, &descriptor, - get_jit_program_space_data ())) + if (!jit_read_descriptor (gdbarch, &descriptor, jiter)) return; entry_addr = descriptor.relevant_entry; diff --git a/gdb/jit.h b/gdb/jit.h index cc135037812..dddded0236f 100644 --- a/gdb/jit.h +++ b/gdb/jit.h @@ -20,6 +20,8 @@ #ifndef JIT_H #define JIT_H +#include "objfiles.h" + /* When the JIT breakpoint fires, the inferior wants us to take one of these actions. These values are used by the inferior, so the values of these enums cannot be changed. */ @@ -78,6 +80,6 @@ extern void jit_breakpoint_re_set (void); /* This function is called by handle_inferior_event when it decides that the JIT event breakpoint has fired. */ -extern void jit_event_handler (struct gdbarch *gdbarch); +extern void jit_event_handler (gdbarch *gdbarch, objfile *jiter); #endif /* JIT_H */ -- 2.17.1