[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 09/14] linux-user: Put the stack guard page at the t
From: |
Richard Henderson |
Subject: |
[Qemu-devel] [PATCH 09/14] linux-user: Put the stack guard page at the top. |
Date: |
Wed, 28 Apr 2010 19:39:53 -0000 |
There are no supported stack-grows-up targets. We were putting
the guard page at the highest address, i.e. the bottom of the stack.
Use the maximum of host and guest page size for the guard size.
Signed-off-by: Richard Henderson <address@hidden>
---
linux-user/elfload.c | 29 ++++++++++++++++-------------
1 files changed, 16 insertions(+), 13 deletions(-)
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index 5853a30..464160a 100644
--- a/linux-user/elfload.c
+++ b/linux-user/elfload.c
@@ -1002,28 +1002,31 @@ static abi_ulong copy_elf_strings(int argc,char **
argv, void **page,
static abi_ulong setup_arg_pages(abi_ulong p, struct linux_binprm *bprm,
struct image_info *info)
{
- abi_ulong stack_base, size, error;
+ abi_ulong stack_base, size, error, guard;
int i;
/* Create enough stack to hold everything. If we don't use
- * it for args, we'll use it for something else...
- */
+ it for args, we'll use it for something else. */
size = guest_stack_size;
- if (size < MAX_ARG_PAGES*TARGET_PAGE_SIZE)
+ if (size < MAX_ARG_PAGES*TARGET_PAGE_SIZE) {
size = MAX_ARG_PAGES*TARGET_PAGE_SIZE;
- error = target_mmap(0,
- size + qemu_host_page_size,
- PROT_READ | PROT_WRITE,
- MAP_PRIVATE | MAP_ANONYMOUS,
- -1, 0);
+ }
+ guard = TARGET_PAGE_SIZE;
+ if (guard < qemu_real_host_page_size) {
+ guard = qemu_real_host_page_size;
+ }
+
+ error = target_mmap(0, size + guard, PROT_READ | PROT_WRITE,
+ MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
if (error == -1) {
- perror("stk mmap");
+ perror("mmap stack");
exit(-1);
}
- /* we reserve one extra page at the top of the stack as guard */
- target_mprotect(error + size, qemu_host_page_size, PROT_NONE);
- stack_base = error + size - MAX_ARG_PAGES*TARGET_PAGE_SIZE;
+ /* We reserve one extra page at the top of the stack as guard. */
+ target_mprotect(error, guard, PROT_NONE);
+
+ stack_base = error + guard + size - MAX_ARG_PAGES*TARGET_PAGE_SIZE;
p += stack_base;
for (i = 0 ; i < MAX_ARG_PAGES ; i++) {
--
1.6.6.1
- [Qemu-devel] [PATCH 11/14] linux-user: Extract load_elf_image from load_elf_interp., (continued)
- [Qemu-devel] [PATCH 11/14] linux-user: Extract load_elf_image from load_elf_interp., Richard Henderson, 2010/04/05
- [Qemu-devel] [PATCH 09/14] linux-user: Put the stack guard page at the top., Richard Henderson, 2010/04/05
- [Qemu-devel] [PATCH 07/14] linux-user: Load symbols from the interpreter., Richard Henderson, 2010/04/05
- [Qemu-devel] [PATCH 12/14] linux-user: Re-use load_elf_image for the main binary., Richard Henderson, 2010/04/05
- [Qemu-devel] [PATCH 00/14] Implement VDSO for x86-64-linux-user, v2, Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 04/14] linux-user: Reduce lseek+reads while loading elf files., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 01/14] linux-user: Handle filesz < memsz for any PT_LOAD segment., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 02/14] Add more DT_* and AT_* constants to qemu's copy of elf.h., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 09/14] linux-user: Put the stack guard page at the top.,
Richard Henderson <=
- [Qemu-devel] [PATCH 05/14] linux-user: Fix ELF_DATA for Alpha., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 06/14] linux-user: Clean up byte-swapping in elfload.c., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 10/14] linux-user: Remove partial support for a.out interpreters., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 08/14] linux-user: Improve consistency checking in elf headers., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 11/14] linux-user: Extract load_elf_image from load_elf_interp., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 13/14] linux-user: Build vdso for x64., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 14/14] linux-user: Load a VDSO for x86-64., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 12/14] linux-user: Re-use load_elf_image for the main binary., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 07/14] linux-user: Load symbols from the interpreter., Richard Henderson, 2010/04/28
- [Qemu-devel] [PATCH 03/14] linux-user: Reindent elfload.c., Richard Henderson, 2010/04/28