diff mbox

[PULL,06/14] linux-user/elfload: ensure mmap_lock() held while setting up

Message ID 1477923215-30323-7-git-send-email-pbonzini@redhat.com
State Accepted
Commit 98c1076cc9a3ccebd1316fecf65149f6052a24fc
Headers show

Commit Message

Paolo Bonzini Oct. 31, 2016, 2:13 p.m. UTC 
From: Alex Bennée <alex.bennee@linaro.org>


Future patches will enforce the holding of mmap_lock() when we are
manipulating internal memory structures. Technically it doesn't matter
in the case of elfload as we haven't started executing yet. However it
is easier to grab the lock when required than special case the
translate-all API.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: Richard Henderson <rth@twiddle.net>

Message-Id: <20161027151030.20863-8-alex.bennee@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

---
 linux-user/elfload.c | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.7.4
diff mbox

Patch

diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index 816272a..547053c 100644
--- a/linux-user/elfload.c
+++ b/linux-user/elfload.c
@@ -1842,6 +1842,8 @@  static void load_elf_image(const char *image_name, int image_fd,
     info->pt_dynamic_addr = 0;
 #endif
 
+    mmap_lock();
+
     /* Find the maximum size of the image and allocate an appropriate
        amount of memory to handle that.  */
     loaddr = -1, hiaddr = 0;
@@ -2002,6 +2004,8 @@  static void load_elf_image(const char *image_name, int image_fd,
         load_symbols(ehdr, image_fd, load_bias);
     }
 
+    mmap_unlock();
+
     close(image_fd);
     return;