From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47629) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TsWo8-00051Y-I0 for qemu-devel@nongnu.org; Tue, 08 Jan 2013 05:53:26 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TsWo7-0007w3-DC for qemu-devel@nongnu.org; Tue, 08 Jan 2013 05:53:24 -0500 Received: from relay1.mentorg.com ([192.94.38.131]:42561) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TsWo7-0007vy-77 for qemu-devel@nongnu.org; Tue, 08 Jan 2013 05:53:23 -0500 Message-ID: <50EBFA9B.1010701@mentor.com> Date: Tue, 8 Jan 2013 12:53:15 +0200 From: Alexander Barabash MIME-Version: 1.0 References: <1357560425-15950-1-git-send-email-alexander_barabash@mentor.com> <50EAC810.2060201@suse.de> In-Reply-To: <50EAC810.2060201@suse.de> Content-Type: text/plain; charset="ISO-8859-15"; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 1/1] Added address_space_init2(). List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-15?Q?Andreas_F=E4rber?= Cc: qemu-devel@nongnu.org Hi, On 01/07/2013 03:05 PM, Andreas Färber wrote: > A "memory: " prefix in the subject would've been nice for filtering. OK > > Am 07.01.2013 13:07, schrieb Alexander Barabash: >> address_space_init2: initializes a named address space. > What for? There are no users in this patch that justify its utility over > setting the field manually. Names of memory regions and address spaces are used for debugging purposes. Currently, there is no way for address spaces other than "memory" and "io" to appear in debugging output. This is used in some code to be posted later. > > The name is really awful, maybe ..._init_with_name or add a name > argument to the existing function and let the existing callers pass NULL? Changing the existing callers of address_space_init() is OK with me. I did not do that primarily because that would be an obvious way to do to start with. And since this was not implemented this way, there should have been some some reason. However, I'll send a PATCH along these lines. > > Regards, > Andreas Regards, Alex > >> Signed-off-by: Alexander Barabash >> --- >> include/exec/memory.h | 9 +++++++++ >> memory.c | 6 ++++++ >> 2 files changed, 15 insertions(+) >> >> diff --git a/include/exec/memory.h b/include/exec/memory.h >> index 2322732..8f8a31d 100644 >> --- a/include/exec/memory.h >> +++ b/include/exec/memory.h >> @@ -820,6 +820,15 @@ void mtree_info(fprintf_function mon_printf, void *f); >> */ >> void address_space_init(AddressSpace *as, MemoryRegion *root); >> >> +/** >> + * address_space_init2: initializes a named address space >> + * >> + * @as: an uninitialized #AddressSpace >> + * @root: a #MemoryRegion that routes addesses for the address space >> + * @name: used for debugging >> + */ >> +void address_space_init2(AddressSpace *as, MemoryRegion *root, >> + const char *name); >> >> /** >> * address_space_destroy: destroy an address space >> diff --git a/memory.c b/memory.c >> index 410c5f8..1652c10 100644 >> --- a/memory.c >> +++ b/memory.c >> @@ -1574,6 +1574,12 @@ void address_space_init(AddressSpace *as, MemoryRegion *root) >> address_space_init_dispatch(as); >> } >> >> +void address_space_init2(AddressSpace *as, MemoryRegion *root, const char *name) >> +{ >> + address_space_init(as, root); >> + as->name = g_strdup(name); >> +} >> + >> void address_space_destroy(AddressSpace *as) >> { >> /* Flush out anything from MemoryListeners listening in on this */