From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49534) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bMXCT-0004Zt-3f for qemu-devel@nongnu.org; Mon, 11 Jul 2016 05:08:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bMXCO-0003Dy-QB for qemu-devel@nongnu.org; Mon, 11 Jul 2016 05:08:23 -0400 Received: from mx-v6.kamp.de ([2a02:248:0:51::16]:49201 helo=mx01.kamp.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bMXCO-0003Di-GY for qemu-devel@nongnu.org; Mon, 11 Jul 2016 05:08:20 -0400 From: Peter Lieven Date: Mon, 11 Jul 2016 11:07:56 +0200 Message-Id: <1468228082-7492-1-git-send-email-pl@kamp.de> Subject: [Qemu-devel] [PATCH V4 0/6] coroutine: mmap stack memory and stack size List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, mreitz@redhat.com, pbonzini@redhat.com, mst@redhat.com, dgilbert@redhat.com, peter.maydell@linaro.org, eblake@redhat.com, rth@twiddle.net, armbru@redhat.com, Peter Lieven 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. 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 | 23 +++++++++++++ util/coroutine-sigaltstack.c | 7 ++-- util/coroutine-ucontext.c | 9 +++-- util/coroutine-win32.c | 2 +- util/oslib-posix.c | 81 ++++++++++++++++++++++++++++++++++++++++++++ 7 files changed, 133 insertions(+), 10 deletions(-) -- 1.9.1