On 28 December 2017 at 18:08, Luke Shumaker <luke...@lukeshu.com> wrote: > From: Luke Shumaker <luke...@parabola.nu> > > There are 3 parts to this change: > - Add a comment showing the relative sizes and positions of the blocks of > memory > - introduce and use new aligned_{start,size} instead of adjusting > real_{start_size} > - When we clean up (on failure), munmap(real_start, real_size) instead of > munmap(aligned_start, aligned_size). It *shouldn't* make any > difference, but I will admit that this does mean we are making the > syscall with different values, so this isn't quite a no-op patch. > > Signed-off-by: Luke Shumaker <luke...@parabola.nu> > --- > linux-user/elfload.c | 43 +++++++++++++++++++++++++++++++++---------- > 1 file changed, 33 insertions(+), 10 deletions(-)
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM