From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26892 invoked by alias); 18 Jun 2013 14:47:46 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 26849 invoked by uid 89); 18 Jun 2013 14:47:45 -0000 X-Spam-SWARE-Status: No, score=-6.1 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD,TW_BJ autolearn=ham version=3.3.1 Received: from mga09.intel.com (HELO mga09.intel.com) (134.134.136.24) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Tue, 18 Jun 2013 14:47:44 +0000 Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga102.jf.intel.com with ESMTP; 18 Jun 2013 07:45:23 -0700 X-ExtLoop1: 1 Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga002.jf.intel.com with ESMTP; 18 Jun 2013 07:47:38 -0700 Received: from ulslx001.iul.intel.com (ulslx001.iul.intel.com [172.28.207.63]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id r5IElbJP027602; Tue, 18 Jun 2013 15:47:38 +0100 Received: from ulslx001.iul.intel.com (localhost [127.0.0.1]) by ulslx001.iul.intel.com with ESMTP id r5IElbSs004784; Tue, 18 Jun 2013 16:47:37 +0200 Received: (from nblanc@localhost) by ulslx001.iul.intel.com with id r5IElbZE004780; Tue, 18 Jun 2013 16:47:37 +0200 From: Nicolas Blanc To: gdb-patches@sourceware.org, Hafiz_Abid@mentor.com, palves@redhat.com, tromey@redhat.com, eliz@gnu.org, yao@codesourcery.com, lgustavo@codesourcery.com, dje@google.com Cc: nicolas.blanc@intel.com Subject: [patch v9 4/5] 'add-symbol-file' should update the current target sections. Date: Tue, 18 Jun 2013 15:04:00 -0000 Message-Id: <1371566833-4713-5-git-send-email-nicolas.blanc@intel.com> In-Reply-To: <1371566833-4713-1-git-send-email-nicolas.blanc@intel.com> References: <1371566833-4713-1-git-send-email-nicolas.blanc@intel.com> X-SW-Source: 2013-06/txt/msg00420.txt.bz2 2013-17-06 Nicolas Blanc * symfile.c (add_target_sections_of_objfile): New function. (add_symbol_file_command): Update the current target sections. (symfile_free_objfile): New function. (_initialize_symfile): Attach observer for free_objfile. Signed-off-by: Nicolas Blanc --- gdb/symfile.c | 69 +++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 67 insertions(+), 2 deletions(-) diff --git a/gdb/symfile.c b/gdb/symfile.c index f7ad268..33d0eda 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -57,6 +57,7 @@ #include "stack.h" #include "gdb_bfd.h" #include "cli/cli-utils.h" +#include "target.h" #include #include @@ -2162,6 +2163,52 @@ print_transfer_performance (struct ui_file *stream, ui_out_text (uiout, ".\n"); } +/* Add the sections of OBJFILE to the current set of target sections. */ + +static void +add_target_sections_of_objfile (struct objfile* objfile) +{ + struct target_section_table *table = current_target_sections; + struct obj_section *osect; + int space; + unsigned count = 0; + struct target_section* ts; + + if (objfile == NULL) + return; + + /* Compute the number of sections to add. */ + ALL_OBJFILE_OSECTIONS (objfile, osect) + { + if (bfd_get_section_size (osect->the_bfd_section) == 0) + continue; + count++; + } + + if (count == 0) + return; + + space = resize_section_table (table, count); + + ts = table->sections + space; + + ALL_OBJFILE_OSECTIONS (objfile, osect) + { + if (bfd_get_section_size (osect->the_bfd_section) == 0) + continue; + + gdb_assert (ts < table->sections + space + count); + + ts->addr = obj_section_addr (osect); + ts->endaddr = obj_section_endaddr (osect); + ts->the_bfd_section = osect->the_bfd_section; + ts->key = (void*) objfile; + ts->bfd = objfile->obfd; + + ts++; + } +} + /* This function allows the addition of incrementally linked object files. It does not modify any state in the target, only in the debugger. */ /* Note: ezannoni 2000-04-13 This function/command used to have a @@ -2185,6 +2232,7 @@ add_symbol_file_command (char *args, int from_tty) int expecting_sec_name = 0; int expecting_sec_addr = 0; char **argv; + struct objfile *objf; struct sect_opt { @@ -2320,8 +2368,10 @@ add_symbol_file_command (char *args, int from_tty) if (from_tty && (!query ("%s", ""))) error (_("Not confirmed.")); - symbol_file_add (filename, from_tty ? SYMFILE_VERBOSE : 0, - section_addrs, flags); + objf = symbol_file_add (filename, from_tty ? SYMFILE_VERBOSE : 0, + section_addrs, flags); + + add_target_sections_of_objfile (objf); /* Getting new symbols may change our opinion about what is frameless. */ @@ -3798,11 +3848,26 @@ symfile_find_segment_sections (struct objfile *objfile) free_symfile_segment_data (data); } +/* Listen for free_objfile events. */ + +static void +symfile_free_objfile (struct objfile *objfile) +{ + if (objfile == NULL) + return; + + /* Remove the target sections of user-added objfiles. */ + if (objfile->flags & OBJF_USERLOADED && objfile->obfd) + remove_target_sections ((void*)objfile, objfile->obfd); +} + void _initialize_symfile (void) { struct cmd_list_element *c; + observer_attach_free_objfile (symfile_free_objfile); + c = add_cmd ("symbol-file", class_files, symbol_file_command, _("\ Load symbol table from executable file FILE.\n\ The `file' command can also load symbol tables, as well as setting the file\n\ -- 1.7.6.5