From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22616 invoked by alias); 20 Nov 2013 04:46:42 -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 22603 invoked by uid 89); 20 Nov 2013 04:46:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=3.1 required=5.0 tests=AWL,BAYES_50,GARBLED_BODY,RDNS_NONE,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: relay1.mentorg.com Received: from Unknown (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 20 Nov 2013 04:46:40 +0000 Received: from svr-orw-exc-10.mgc.mentorg.com ([147.34.98.58]) by relay1.mentorg.com with esmtp id 1VizgG-0002qv-QV from Yao_Qi@mentor.com ; Tue, 19 Nov 2013 20:46:24 -0800 Received: from SVR-ORW-FEM-02.mgc.mentorg.com ([147.34.96.206]) by SVR-ORW-EXC-10.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.4675); Tue, 19 Nov 2013 20:46:24 -0800 Received: from qiyao.dyndns.org (147.34.91.1) by svr-orw-fem-02.mgc.mentorg.com (147.34.96.168) with Microsoft SMTP Server id 14.2.247.3; Tue, 19 Nov 2013 20:46:24 -0800 Message-ID: <528C3E40.2070203@codesourcery.com> Date: Wed, 20 Nov 2013 07:54:00 -0000 From: Yao Qi User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 MIME-Version: 1.0 To: Doug Evans CC: gdb-patches Subject: Re: [PATCH 07/10] Associate target_dcache to address_space. References: <1383458049-20893-1-git-send-email-yao@codesourcery.com> <1383458049-20893-8-git-send-email-yao@codesourcery.com> In-Reply-To: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2013-11/txt/msg00585.txt.bz2 On 11/18/2013 05:09 AM, Doug Evans wrote: > Ok. > > [I realize I've asked for changes that will affect this patch, but > beyond that it's ok.] Update the patch, and fix the doc pointed out by Eli. Patch below is pushed. -- Yao (齐尧) gdb/doc: 2013-11-20 Yao Qi * gdb.texinfo (Caching Target Data): Update doc for per-address-space dcache. gdb: 2013-11-20 Yao Qi * progspace.h (struct address_space_data): Declare. * target-dcache.c: Include "progspace.h". (target_dache): Remove. (target_dcache_aspace_key): New. (target_dcache_cleanup): New function. (target_dcache_init_p): Get data through target_dcache_aspace_key. (target_dcache_invalidate): Likewise. (target_dcache_get): Likewise. (target_dcache_get_or_init): Likewise. (_initialize_target_dcache): Initialize target_dcache_aspace_key. --- gdb/ChangeLog | 15 +++++++++++++++ gdb/doc/ChangeLog | 5 +++++ gdb/doc/gdb.texinfo | 12 +++++++----- gdb/progspace.h | 1 + gdb/target-dcache.c | 46 +++++++++++++++++++++++++++++++++++++--------- 5 files changed, 65 insertions(+), 14 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e8df902..dadea24 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,20 @@ 2013-11-20 Yao Qi + * progspace.h (struct address_space_data): Declare. + * target-dcache.c: Include "progspace.h". + (target_dache): Remove. + (target_dcache_aspace_key): New. + (target_dcache_cleanup): New function. + (target_dcache_init_p): Get data through + target_dcache_aspace_key. + (target_dcache_invalidate): Likewise. + (target_dcache_get): Likewise. + (target_dcache_get_or_init): Likewise. + (_initialize_target_dcache): Initialize + target_dcache_aspace_key. + +2013-11-20 Yao Qi + * progspace.c (struct address_space): Update comments. : New fields. DEFINE_REGISTRY for address_space. diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index 9b641e6..207b845 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,5 +1,10 @@ 2013-11-20 Yao Qi + * gdb.texinfo (Caching Target Data): Update doc for + per-address-space dcache. + +2013-11-20 Yao Qi + * gdb.texinfo (Data): Rename menu item. (Caching Remote Data): Rename to ... (Caching Target Data): ... it. Update. diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index acad48d..d1fa6c8 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -10828,6 +10828,8 @@ character. @cindex caching data of targets @value{GDBN} caches data exchanged between the debugger and a target. +Each cache is associated with the address space of the inferior. +@xref{Inferiors and Programs}, about inferior and address space. Such caching generally improves performance in remote debugging (@pxref{Remote Debugging}), because it reduces the overhead of the remote protocol by bundling memory reads and writes into large chunks. @@ -10867,11 +10869,11 @@ Show the current state of data caching for memory accesses. @kindex info dcache @item info dcache @r{[}line@r{]} -Print the information about the data cache performance. The -information displayed includes the dcache width and depth, and for -each cache line, its number, address, and how many times it was -referenced. This command is useful for debugging the data cache -operation. +Print the information about the performance of data cache of the +current inferior's address space. The information displayed +includes the dcache width and depth, and for each cache line, its +number, address, and how many times it was referenced. This +command is useful for debugging the data cache operation. If a line number is specified, the contents of that line will be printed in hex. diff --git a/gdb/progspace.h b/gdb/progspace.h index 04ac374..e2786ec 100644 --- a/gdb/progspace.h +++ b/gdb/progspace.h @@ -33,6 +33,7 @@ struct inferior; struct exec; struct address_space; struct program_space_data; +struct address_space_data; typedef struct so_list *so_list_ptr; DEF_VEC_P (so_list_ptr); diff --git a/gdb/target-dcache.c b/gdb/target-dcache.c index bf04679..eb7d478 100644 --- a/gdb/target-dcache.c +++ b/gdb/target-dcache.c @@ -18,16 +18,31 @@ #include "defs.h" #include "target-dcache.h" #include "gdbcmd.h" +#include "progspace.h" -/* Cache of memory operations, to speed up remote access. */ -static DCACHE *target_dcache; +/* The target dcache is kept per-address-space. This key lets us + associate the cache with the address space. */ + +static const struct address_space_data *target_dcache_aspace_key; + +/* Clean up dcache, represented by ARG, which is associated with + ASPACE. */ + +static void +target_dcache_cleanup (struct address_space *aspace, void *arg) +{ + dcache_free (arg); +} /* Target dcache is initialized or not. */ int target_dcache_init_p (void) { - return (target_dcache != NULL); + DCACHE *dcache = address_space_data (current_program_space->aspace, + target_dcache_aspace_key); + + return (dcache != NULL); } /* Invalidate the target dcache. */ @@ -35,8 +50,11 @@ target_dcache_init_p (void) void target_dcache_invalidate (void) { - if (target_dcache_init_p ()) - dcache_invalidate (target_dcache); + DCACHE *dcache = address_space_data (current_program_space->aspace, + target_dcache_aspace_key); + + if (dcache != NULL) + dcache_invalidate (dcache); } /* Return the target dcache. Return NULL if target dcache is not @@ -45,7 +63,10 @@ target_dcache_invalidate (void) DCACHE * target_dcache_get (void) { - return target_dcache; + DCACHE *dcache = address_space_data (current_program_space->aspace, + target_dcache_aspace_key); + + return dcache; } /* Return the target dcache. If it is not initialized yet, initialize @@ -54,10 +75,13 @@ target_dcache_get (void) DCACHE * target_dcache_get_or_init (void) { - if (!target_dcache_init_p ()) - target_dcache = dcache_init (); + DCACHE *dcache = address_space_data (current_program_space->aspace, + target_dcache_aspace_key); - return target_dcache; + if (dcache == NULL) + dcache = dcache_init (); + + return dcache; } /* The option sets this. */ @@ -113,4 +137,8 @@ By default, caching for stack access is on."), set_stack_cache_enabled_p, show_stack_cache_enabled_p, &setlist, &showlist); + + target_dcache_aspace_key + = register_address_space_data_with_cleanup (NULL, + target_dcache_cleanup); } -- 1.7.7.6