| Message ID | 20200221214614.165338-1-yuanzi@google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | linux-user: Add an argument QEMU_MMAP_BASE to set custom mmap base address in qemu user mode | expand |
Patchew URL: https://patchew.org/QEMU/20200221214614.165338-1-yuanzi@google.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH] linux-user: Add an argument QEMU_MMAP_BASE to set custom mmap base address in qemu user mode Message-id: 20200221214614.165338-1-yuanzi@google.com Type: series === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 Switched to a new branch 'test' 38a7b04 linux-user: Add an argument QEMU_MMAP_BASE to set custom mmap base address in qemu user mode === OUTPUT BEGIN === ERROR: consider using qemu_strtoul in preference to strtoul #23: FILE: linux-user/main.c:341: + mmap_base = strtoul(arg, NULL, 0); total: 1 errors, 0 warnings, 50 lines checked Commit 38a7b049a7f8 (linux-user: Add an argument QEMU_MMAP_BASE to set custom mmap base address in qemu user mode) has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200221214614.165338-1-yuanzi@google.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
diff --git a/linux-user/main.c b/linux-user/main.c index fba833aac9..dfcd867399 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -336,6 +336,16 @@ static void handle_arg_guest_base(const char *arg) have_guest_base = 1; } +static void handle_arg_mmap_base(const char *arg) +{ + mmap_base = strtoul(arg, NULL, 0); + if (mmap_base == 0) { + fprintf(stderr, "Invalid mmap_base: %s\n", arg); + exit(EXIT_FAILURE); + } + mmap_next_start = mmap_base; +} + static void handle_arg_reserved_va(const char *arg) { char *p; @@ -440,6 +450,8 @@ static const struct qemu_argument arg_table[] = { "uname", "set qemu uname release string to 'uname'"}, {"B", "QEMU_GUEST_BASE", true, handle_arg_guest_base, "address", "set guest_base address to 'address'"}, + {"mmap_base", "QEMU_MMAP_BASE", true, handle_arg_mmap_base, + "", "begin allocating guest pages at this host address"}, {"R", "QEMU_RESERVED_VA", true, handle_arg_reserved_va, "size", "reserve 'size' bytes for guest virtual address space"}, {"d", "QEMU_LOG", true, handle_arg_log, diff --git a/linux-user/mmap.c b/linux-user/mmap.c index 8685f02e7e..3f35543acf 100644 --- a/linux-user/mmap.c +++ b/linux-user/mmap.c @@ -189,6 +189,7 @@ static int mmap_frag(abi_ulong real_start, # define TASK_UNMAPPED_BASE 0x40000000 #endif abi_ulong mmap_next_start = TASK_UNMAPPED_BASE; +abi_ulong mmap_base = TASK_UNMAPPED_BASE; unsigned long last_brk; @@ -299,7 +300,7 @@ abi_ulong mmap_find_vma(abi_ulong start, abi_ulong size, abi_ulong align) if ((addr & (align - 1)) == 0) { /* Success. */ - if (start == mmap_next_start && addr >= TASK_UNMAPPED_BASE) { + if (start == mmap_next_start && addr >= mmap_base) { mmap_next_start = addr + size; } return addr; diff --git a/linux-user/qemu.h b/linux-user/qemu.h index 560a68090e..83c00cfea2 100644 --- a/linux-user/qemu.h +++ b/linux-user/qemu.h @@ -161,6 +161,11 @@ void task_settid(TaskState *); void stop_all_tasks(void); extern const char *qemu_uname_release; extern unsigned long mmap_min_addr; +/* + * mmap_base is minimum address to use when allocating guest pages. All guest + * pages will be allocated at this (guest) address or higher addresses. + */ +extern abi_ulong mmap_base; /* ??? See if we can avoid exposing so much of the loader internals. */
This change allows us to set custom base address for guest programs. It is needed to allow qemu to work with Thread Sanitizer (TSan), which has specific boundary definitions for memory mappings on different platforms: https://github.com/llvm/llvm-project/blob/master/compiler-rt/lib/tsan/rtl/tsan_platform.h Signed-off-by: Lirong Yuan <yuanzi@google.com> --- linux-user/main.c | 12 ++++++++++++ linux-user/mmap.c | 3 ++- linux-user/qemu.h | 5 +++++ 3 files changed, 19 insertions(+), 1 deletion(-)