[commit] [patchv6] compile: Use also inferior munmap

[Jan Kratochvil]

On Sat, 16 May 2015 15:18:50 +0200, Jan Kratochvil wrote:
> gdb/ChangeLog
> 2015-04-28  Jan Kratochvil  <jan.kratochvil@redhat.com>
> 
> 	* arch-utils.c (default_infcall_munmap): New.
> 	* arch-utils.h (default_infcall_munmap): New declaration.
> 	* compile/compile-object-load.c (struct munmap_list, munmap_list_add)
> 	(munmap_list_free, munmap_listp_free_cleanup): New.
> 	(struct setup_sections_data): Add field munmap_list_headp.
> 	(setup_sections): Call munmap_list_add.
> 	(compile_object_load): New variable munmap_list_head, initialize
> 	setup_sections_data.munmap_list_headp, return munmap_list_head.
> 	* compile/compile-object-load.h (struct munmap_list): New declaration.
> 	(struct compile_module): Add field munmap_list_head.
> 	(munmap_list_free): New declaration.
> 	* compile/compile-object-run.c (struct do_module_cleanup): Add field
> 	munmap_list_head.
> 	(do_module_cleanup): Call munmap_list_free.
> 	(compile_object_run): Pass munmap_list_head to do_module_cleanup.
> 	* gdbarch.c: Regenerate.
> 	* gdbarch.h: Regenerate.
> 	* gdbarch.sh (infcall_munmap): New.
> 	* linux-tdep.c (linux_infcall_munmap): New.
> 	(linux_init_abi): Install it.
>     
> gdb/testsuite/ChangeLog
> 2015-04-28  Jan Kratochvil  <jan.kratochvil@redhat.com>
> 
> 	* gdb.compile/compile.exp (keep jit in memory): Rename to ...
> 	(do not keep jit in memory): ... this.
> 	(expect 5): Change it to ...
> 	(expect no 5): ... this.

Checked in.

This last post was incorrectly diffed containing bogus gdb_stdout->gdb_stdlog
changes from a different (already checked-in patch).

But otherwise the patch was exactly like:
	[PATCH v4 11/11] RFC only: compile: Use also inferior munmap
	https://sourceware.org/ml/gdb-patches/2015-05/msg00037.html
	Message-ID: <20150503140815.18583.29612.stgit@host1.jankratochvil.net>
already IMO approved by Pedro by:
	https://sourceware.org/ml/gdb-patches/2015-05/msg00107.html
	That said, I skimmed it and it overall looks good.
except for the 'stop_registers' part 
	The stop_registers bits looked surprising.
which has been solved in a different patchset already checked in.  So I have
removed that 'stop_registers' part in this checked-in patch.


Jan
-------------- next part --------------
An embedded message was scrubbed...
From: Jan Kratochvil <jan.kratochvil@redhat.com>
Subject: [PATCH] compile: Use also inferior munmap
Date: Wed, 3 Jun 2015 21:22:56 +0200
Size: 19780
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20150603/0d64bde8/attachment.eml>