This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v4 11/11] RFC only: compile: Use also inferior munmap
- From: Pedro Alves <palves at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>, gdb-patches at sourceware dot org
- Cc: Phil Muldoon <pmuldoon at redhat dot com>
- Date: Wed, 06 May 2015 16:21:37 +0100
- Subject: Re: [PATCH v4 11/11] RFC only: compile: Use also inferior munmap
- Authentication-results: sourceware.org; auth=none
- References: <20150503140647 dot 18583 dot 2012 dot stgit at host1 dot jankratochvil dot net> <20150503140815 dot 18583 dot 29612 dot stgit at host1 dot jankratochvil dot net>
On 05/03/2015 03:08 PM, Jan Kratochvil wrote:
> Hi,
>
> currently inferior memory is allocated by inferior mmap() but it is never
> deallocated; despite the injected objfile incl. its symbols is freed. This was
> intentional so that one can do for example:
> inferior:
> char *str = "foo";
> GDB:
> (gdb) compile code str = "bar";
>
> I believe later patches will be needed to introduce full control over keeping
> vs. discarding the injected module as being discussed in:
> compile: objfiles lifetime UI
> https://sourceware.org/ml/gdb/2015-04/msg00051.html
> Message-ID: <20150429135735.GA16974@host1.jankratochvil.net>
> This patch at least introduces code which will be needed for the part/cases of
> really freeing all the resources of an injected module.
>
> It is "RFC only" as given the patch as is it regresses GDB functionality.
I don't which parts of this would cause regressions; it's helpful
when an RFC points out what is known to be hacky or patch bits that are known
to need to change. It's not obvious, at least to me.
That said, I skimmed it and it overall looks good. The stop_registers
bits looked surprising. My reaction was that I'd think that would be
something handled around save_infcall_suspend_state/restore_infcall_suspend_state,
though it's not clear to me.
Thanks,
Pedro Alves