On 01/04/2011 04:18 PM, Rik van Riel wrote:
On 01/03/2011 10:21 PM, Zhai, Edwin wrote:
Riel,
Thanks for your patch. I have changed the ple_gap to 128 on xen side,
but forget the patch for KVM:(

A little bit big is no harm, but more perf data is better.

So should I resend the patch with the ple_gap default
changed to 128, or are you willing to ack the current
patch?


I think 128 is safer given than 41 was too low. We have to take into account newer cpus and slower spin loops. If the spin loop does a cache ping-pong (which would be a bad, bad possible, implementation), even 128 might be too low.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to