[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V7 0/6] coroutine: mmap stack memory and stack s
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH V7 0/6] coroutine: mmap stack memory and stack size |
Date: |
Tue, 30 Aug 2016 14:08:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 23/08/2016 16:21, Peter Lieven wrote:
> I decided to split this from the rest of the Qemu RSS usage series as
> it contains the more or less non contentious patches.
>
> I omitted the MAP_GROWSDOWN flag in mmap as we are not 100% sure which
> side effects it has.
>
> I kept the guard page which is now nicely makes the stacks visible in
> smaps. The old version of the relevent patch lacked the MAP_FIXED flag
> in the second call to mmap.
>
> The last patch which reduces the stack size of coroutines to 64kB
> may be omitted if its found to risky.
>
> v6->v6:
> - Patch 1: avoid multiple calls to sysconf and getpagesize [Richard]
>
> v5->v6:
> - Patch 1: added info that the guard page is deducted from stack memory to
> commit msg and headers [Stefan]
> - rebased to master
>
> v4->v5:
> - Patch 1: check if _SC_THREAD_STACK_MIN is defined
> - Patch 1: guard against sysconf(_SC_THREAD_STACK_MIN) returning -1 [Eric]
>
> v3->v4:
> - Patch 1: add a static function to adjust the stack size [Richard]
> - Patch 1: round up the stack size to multiple of the pagesize.
>
> v2->v3:
> - Patch 1,6: adjusted commit message to mention the guard page [Markus]
>
> v1->v2:
> - Patch 1: added an architecture dependend guard page [Richard]
> - Patch 1: avoid stacks smaller than _SC_THREAD_STACK_MIN [Richard]
> - Patch 1: use mmap+mprotect instead of mmap+mmap [Richard]
> - Patch 5: u_int32_t -> uint32_t [Richard]
> - Patch 5: only available if stack grows down
>
> Peter Lieven (6):
> oslib-posix: add helpers for stack alloc and free
> coroutine: add a macro for the coroutine stack size
> coroutine-ucontext: use helper for allocating stack memory
> coroutine-sigaltstack: use helper for allocating stack memory
> oslib-posix: add a configure switch to debug stack usage
> coroutine: reduce stack size to 64kB
>
> configure | 19 ++++++++++
> include/qemu/coroutine_int.h | 2 ++
> include/sysemu/os-posix.h | 27 ++++++++++++++
> util/coroutine-sigaltstack.c | 7 ++--
> util/coroutine-ucontext.c | 9 +++--
> util/coroutine-win32.c | 2 +-
> util/oslib-posix.c | 84
> ++++++++++++++++++++++++++++++++++++++++++++
> 7 files changed, 140 insertions(+), 10 deletions(-)
>
Acked-by: Paolo Bonzini <address@hidden>
I guess Stefan or Kevin will apply it.
Paolo
- [Qemu-devel] [PATCH V7 0/6] coroutine: mmap stack memory and stack size, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 5/6] oslib-posix: add a configure switch to debug stack usage, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 1/6] oslib-posix: add helpers for stack alloc and free, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 2/6] coroutine: add a macro for the coroutine stack size, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 6/6] coroutine: reduce stack size to 64kB, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 3/6] coroutine-ucontext: use helper for allocating stack memory, Peter Lieven, 2016/08/23
- [Qemu-devel] [PATCH V7 4/6] coroutine-sigaltstack: use helper for allocating stack memory, Peter Lieven, 2016/08/23
- Re: [Qemu-devel] [PATCH V7 0/6] coroutine: mmap stack memory and stack size,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH V7 0/6] coroutine: mmap stack memory and stack size, Kevin Wolf, 2016/08/31