Re: [Qemu-devel] [PATCH 6/6] coroutine: reduce stack size to 64kB

2016-07-04 Thread Paolo Bonzini
On 04/07/2016 08:22, Peter Lieven wrote: >>> >> >> Ought we check that this is not smaller than >> >> sysconf(_SC_THREAD_STACK_MIN) >> >> which (for glibc at least), is 192k for ia64, 128k for aarch64, mips >> and tile (though why it is quite so high in those later cases I don't >> know). >

Re: [Qemu-devel] [PATCH 6/6] coroutine: reduce stack size to 64kB

2016-07-03 Thread Peter Lieven
Am 01.07.2016 um 23:13 schrieb Richard Henderson: On 06/30/2016 12:37 AM, Peter Lieven wrote: evaluation with the recently introduced maximum stack usage monitoring revealed that the actual used stack size was never above 4kB so allocating 1MB stack for each coroutine is a lot of wasted memory.

Re: [Qemu-devel] [PATCH 6/6] coroutine: reduce stack size to 64kB

2016-07-01 Thread Richard Henderson
On 06/30/2016 12:37 AM, Peter Lieven wrote: evaluation with the recently introduced maximum stack usage monitoring revealed that the actual used stack size was never above 4kB so allocating 1MB stack for each coroutine is a lot of wasted memory. So reduce the stack size to 64kB which should still

[Qemu-devel] [PATCH 6/6] coroutine: reduce stack size to 64kB

2016-06-30 Thread Peter Lieven
evaluation with the recently introduced maximum stack usage monitoring revealed that the actual used stack size was never above 4kB so allocating 1MB stack for each coroutine is a lot of wasted memory. So reduce the stack size to 64kB which should still give enough head room. Signed-off-by: Peter