Re: [PATCH RFC V2 5/5] kvm guest : pv-ticketlocks support for linuxguests running on KVM hypervisor

From: Raghavendra K T
Date: Wed Oct 26 2011 - 15:23:25 EST


On 10/26/2011 12:04 AM, Jeremy Fitzhardinge wrote:
On 10/23/2011 12:07 PM, Raghavendra K T wrote:
This patch extends Linux guests running on KVM hypervisor to support
+/*
+ * Setup pv_lock_ops to exploit KVM_FEATURE_WAIT_FOR_KICK if present.
+ * This needs to be setup really early in boot, before the first call to
+ * spinlock is issued!

Actually, it doesn't matter that much. The in-memory format is the same
for regular and PV spinlocks, and the PV paths only come into play if
the "slowpath" flag is set in the lock, which it never will be by the
non-PV code.

In principle, you could defer initializing PV ticketlocks until some
arbitrarily late point if you notice that the system is oversubscribed
enough to require it.

ok.. so this means it will not affect even if it is initialized in
middle somewhere, but better to do it before we start seeing lock contention. our current aim was to have before any printk happens.
So I 'll trim the comment to somethings like :

Setup pv_lock_ops to exploit KVM_FEATURE_WAIT_FOR_KICK if present.
This needs to be setup early in boot. ?


The main constraint at present is that you need to update the
pv_lock_ops structure before pvops patching happens, or you won't see
any effect from making changes.

Hmm. got it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/