Re: [PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-25 Thread Paolo Bonzini
On 24/09/2015 13:06, Christian Borntraeger wrote: > Am 18.09.2015 um 13:29 schrieb Paolo Bonzini: >> >> >> On 18/09/2015 12:54, Christian Borntraeger wrote: -/* halt polling only reduces halt latency by 5-7 us, 500us is enough */ -static unsigned int halt_poll_ns = 50; +/*

Re: [PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-24 Thread Christian Borntraeger
Am 18.09.2015 um 13:29 schrieb Paolo Bonzini: > > > On 18/09/2015 12:54, Christian Borntraeger wrote: >>> -/* halt polling only reduces halt latency by 5-7 us, 500us is enough */ >>> -static unsigned int halt_poll_ns = 50; >>> +/* Architectures should define their poll value according to the

Re: [PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-18 Thread Christian Borntraeger
Am 18.09.2015 um 12:34 schrieb David Hildenbrand: > We observed some performance degradation on s390x with dynamic > halt polling. Until we can provide a proper fix, let's enable > halt_poll_ns as default only for supported architectures. > > Architectures are now free to set their own

Re: [PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-18 Thread Christian Borntraeger
Am 18.09.2015 um 13:29 schrieb Paolo Bonzini: > > > On 18/09/2015 12:54, Christian Borntraeger wrote: >>> -/* halt polling only reduces halt latency by 5-7 us, 500us is enough */ >>> -static unsigned int halt_poll_ns = 50; >>> +/* Architectures should define their poll value according to the

Re: [PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-18 Thread Paolo Bonzini
On 18/09/2015 12:54, Christian Borntraeger wrote: > > -/* halt polling only reduces halt latency by 5-7 us, 500us is enough */ > > -static unsigned int halt_poll_ns = 50; > > +/* Architectures should define their poll value according to the halt > > latency */ > > +static unsigned int

[PATCH] KVM: disable halt_poll_ns as default for s390x

2015-09-18 Thread David Hildenbrand
We observed some performance degradation on s390x with dynamic halt polling. Until we can provide a proper fix, let's enable halt_poll_ns as default only for supported architectures. Architectures are now free to set their own halt_poll_ns default value. Signed-off-by: David Hildenbrand