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

2016-06-29 Thread Peter Lieven
Am 28.06.2016 um 16:20 schrieb Dr. David Alan Gilbert: * Peter Lieven (p...@kamp.de) wrote: Am 28.06.2016 um 13:35 schrieb Dr. David Alan Gilbert: * Peter Lieven (p...@kamp.de) wrote: Am 28.06.2016 um 12:57 schrieb Dr. David Alan Gilbert: * Paolo Bonzini (pbonz...@redhat.com) wrote: On 28/06

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

2016-06-28 Thread Dr. David Alan Gilbert
* Peter Lieven (p...@kamp.de) wrote: > Am 28.06.2016 um 13:35 schrieb Dr. David Alan Gilbert: > > * Peter Lieven (p...@kamp.de) wrote: > > > Am 28.06.2016 um 12:57 schrieb Dr. David Alan Gilbert: > > > > * Paolo Bonzini (pbonz...@redhat.com) wrote: > > > > > On 28/06/2016 11:01, Peter Lieven wrote:

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

2016-06-28 Thread Peter Lieven
Am 28.06.2016 um 13:35 schrieb Dr. David Alan Gilbert: * Peter Lieven (p...@kamp.de) wrote: Am 28.06.2016 um 12:57 schrieb Dr. David Alan Gilbert: * Paolo Bonzini (pbonz...@redhat.com) wrote: On 28/06/2016 11:01, Peter Lieven wrote: evaluation with the recently introduced maximum stack size m

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

2016-06-28 Thread Dr. David Alan Gilbert
* Peter Lieven (p...@kamp.de) wrote: > Am 28.06.2016 um 12:57 schrieb Dr. David Alan Gilbert: > > * Paolo Bonzini (pbonz...@redhat.com) wrote: > > > > > > On 28/06/2016 11:01, Peter Lieven wrote: > > > > evaluation with the recently introduced maximum stack size monitoring > > > > revealed > > >

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

2016-06-28 Thread Paolo Bonzini
- Original Message - > From: "Peter Lieven" > To: "Paolo Bonzini" , qemu-devel@nongnu.org > Cc: kw...@redhat.com, "peter maydell" , > m...@redhat.com, dgilb...@redhat.com, > mre...@redhat.com, kra...@redhat.com > Sent: Tuesday, June 28, 2016 1:13:26 PM > Subject: Re: [PATCH 03/15] corou

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

2016-06-28 Thread Peter Lieven
Am 28.06.2016 um 12:57 schrieb Dr. David Alan Gilbert: * Paolo Bonzini (pbonz...@redhat.com) wrote: On 28/06/2016 11:01, Peter Lieven wrote: evaluation with the recently introduced maximum stack size monitoring revealed that the actual used stack size was never above 4kB so allocating 1MB stac

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

2016-06-28 Thread Peter Lieven
Am 28.06.2016 um 12:54 schrieb Paolo Bonzini: On 28/06/2016 11:01, Peter Lieven wrote: evaluation with the recently introduced maximum stack size 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 redu

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

2016-06-28 Thread Dr. David Alan Gilbert
* Paolo Bonzini (pbonz...@redhat.com) wrote: > > > On 28/06/2016 11:01, Peter Lieven wrote: > > evaluation with the recently introduced maximum stack size monitoring > > revealed > > that the actual used stack size was never above 4kB so allocating 1MB stack > > for each coroutine is a lot of wa

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

2016-06-28 Thread Paolo Bonzini
On 28/06/2016 11:01, Peter Lieven wrote: > evaluation with the recently introduced maximum stack size 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 s

[Qemu-devel] [PATCH 03/15] coroutine-ucontext: reduce stack size to 64kB

2016-06-28 Thread Peter Lieven
evaluation with the recently introduced maximum stack size 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 L