* [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi
@ 2013-09-04 8:48 Raghavendra K T
2013-09-12 5:21 ` Raghavendra K T
2013-09-12 5:44 ` Michael S. Tsirkin
0 siblings, 2 replies; 4+ messages in thread
From: Raghavendra K T @ 2013-09-04 8:48 UTC (permalink / raw)
To: Gleb Natapov, Paolo Bonzini
Cc: linux-doc, linux-kernel, kvm, Rob Landley, Michael S. Tsirkin,
mtosatti, Raghavendra K T
Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
---
Changes in V2:
Correction in the description of steal time and added msr info (Michael S Tsirkin)
Documentation/virtual/kvm/cpuid.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/virtual/kvm/cpuid.txt b/Documentation/virtual/kvm/cpuid.txt
index 22ff659..6c4fb20 100644
--- a/Documentation/virtual/kvm/cpuid.txt
+++ b/Documentation/virtual/kvm/cpuid.txt
@@ -43,6 +43,16 @@ KVM_FEATURE_CLOCKSOURCE2 || 3 || kvmclock available at msrs
KVM_FEATURE_ASYNC_PF || 4 || async pf can be enabled by
|| || writing to msr 0x4b564d02
------------------------------------------------------------------------------
+KVM_FEATURE_STEAL_TIME || 5 || Steal time available at msr
+ || || 0x4b564d03. The feature is enabled
+ || || by guest when host has schedstat
+ || || or task delay accounting support.
+------------------------------------------------------------------------------
+KVM_FEATURE_PV_EOI || 6 || overrides the generic EOI
+ || || implementation with a
+ || || paravirtualized version. Available
+ || || at msr 0x4b564d04.
+------------------------------------------------------------------------------
KVM_FEATURE_PV_UNHALT || 7 || guest checks this feature bit
|| || before enabling paravirtualized
|| || spinlock support.
--
1.7.11.7
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi
2013-09-04 8:48 [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi Raghavendra K T
@ 2013-09-12 5:21 ` Raghavendra K T
2013-09-12 5:44 ` Michael S. Tsirkin
1 sibling, 0 replies; 4+ messages in thread
From: Raghavendra K T @ 2013-09-12 5:21 UTC (permalink / raw)
To: Raghavendra K T, Michael S. Tsirkin
Cc: Gleb Natapov, Paolo Bonzini, linux-doc, linux-kernel, kvm,
Rob Landley, mtosatti
On 09/04/2013 02:18 PM, Raghavendra K T wrote:
> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> ---
> Changes in V2:
> Correction in the description of steal time and added msr info (Michael S Tsirkin)
>
> Documentation/virtual/kvm/cpuid.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/virtual/kvm/cpuid.txt b/Documentation/virtual/kvm/cpuid.txt
> index 22ff659..6c4fb20 100644
> --- a/Documentation/virtual/kvm/cpuid.txt
> +++ b/Documentation/virtual/kvm/cpuid.txt
> @@ -43,6 +43,16 @@ KVM_FEATURE_CLOCKSOURCE2 || 3 || kvmclock available at msrs
> KVM_FEATURE_ASYNC_PF || 4 || async pf can be enabled by
> || || writing to msr 0x4b564d02
> ------------------------------------------------------------------------------
> +KVM_FEATURE_STEAL_TIME || 5 || Steal time available at msr
> + || || 0x4b564d03. The feature is enabled
> + || || by guest when host has schedstat
> + || || or task delay accounting support.
> +------------------------------------------------------------------------------
> +KVM_FEATURE_PV_EOI || 6 || overrides the generic EOI
> + || || implementation with a
> + || || paravirtualized version. Available
> + || || at msr 0x4b564d04.
> +------------------------------------------------------------------------------
> KVM_FEATURE_PV_UNHALT || 7 || guest checks this feature bit
> || || before enabling paravirtualized
> || || spinlock support.
>
Michael,
Are you okay with the above patch. Please let me know.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi
2013-09-04 8:48 [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi Raghavendra K T
2013-09-12 5:21 ` Raghavendra K T
@ 2013-09-12 5:44 ` Michael S. Tsirkin
2013-09-12 7:10 ` Raghavendra K T
1 sibling, 1 reply; 4+ messages in thread
From: Michael S. Tsirkin @ 2013-09-12 5:44 UTC (permalink / raw)
To: Raghavendra K T
Cc: Gleb Natapov, Paolo Bonzini, linux-doc, linux-kernel, kvm,
Rob Landley, mtosatti
On Wed, Sep 04, 2013 at 02:18:46PM +0530, Raghavendra K T wrote:
> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> ---
> Changes in V2:
> Correction in the description of steal time and added msr info (Michael S Tsirkin)
Thanks. Some comments below:
> Documentation/virtual/kvm/cpuid.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/virtual/kvm/cpuid.txt b/Documentation/virtual/kvm/cpuid.txt
> index 22ff659..6c4fb20 100644
> --- a/Documentation/virtual/kvm/cpuid.txt
> +++ b/Documentation/virtual/kvm/cpuid.txt
> @@ -43,6 +43,16 @@ KVM_FEATURE_CLOCKSOURCE2 || 3 || kvmclock available at msrs
> KVM_FEATURE_ASYNC_PF || 4 || async pf can be enabled by
> || || writing to msr 0x4b564d02
> ------------------------------------------------------------------------------
> +KVM_FEATURE_STEAL_TIME || 5 || Steal time available at msr
> + || || 0x4b564d03. The feature is enabled
> + || || by guest when host has schedstat
> + || || or task delay accounting support.
Well I think we really should be documenting the host/guest interface
here.
After all it starts out
A guest running on a kvm host, can check some of its features using
cpuid.
And guests really should not try to guess whether host
has schedstat or task delay accounting support based on this bit.
So I would just say:
"steal time can be enabled by writing to msr 0x4b564d02"
just like async pf.
> +------------------------------------------------------------------------------
> +KVM_FEATURE_PV_EOI || 6 || overrides the generic EOI
> + || || implementation with a
> + || || paravirtualized version. Available
> + || || at msr 0x4b564d04.
Actually there's no override: guest can still use the
standard EOI even if PV EOI MSR is enabled.
There's a detailed explanation of PV EOI in
Documentation/virtual/kvm/msr.txt
I think the following would be enough:
"paravirtualized end of interrupt handler can be enabled by writing to msr 0x4b564d04"
> +------------------------------------------------------------------------------
> KVM_FEATURE_PV_UNHALT || 7 || guest checks this feature bit
> || || before enabling paravirtualized
> || || spinlock support.
> --
> 1.7.11.7
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi
2013-09-12 5:44 ` Michael S. Tsirkin
@ 2013-09-12 7:10 ` Raghavendra K T
0 siblings, 0 replies; 4+ messages in thread
From: Raghavendra K T @ 2013-09-12 7:10 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Gleb Natapov, Paolo Bonzini, linux-doc, linux-kernel, kvm,
Rob Landley, mtosatti
On 09/12/2013 11:14 AM, Michael S. Tsirkin wrote:
> On Wed, Sep 04, 2013 at 02:18:46PM +0530, Raghavendra K T wrote:
[...]
>> ------------------------------------------------------------------------------
>> +KVM_FEATURE_STEAL_TIME || 5 || Steal time available at msr
>> + || || 0x4b564d03. The feature is enabled
>> + || || by guest when host has schedstat
>> + || || or task delay accounting support.
>
> Well I think we really should be documenting the host/guest interface
> here.
> After all it starts out
> A guest running on a kvm host, can check some of its features using
> cpuid.
> And guests really should not try to guess whether host
> has schedstat or task delay accounting support based on this bit.
> So I would just say:
> "steal time can be enabled by writing to msr 0x4b564d02"
> just like async pf.
>
It makes sense. will stick to the above line.
>
>> +------------------------------------------------------------------------------
>> +KVM_FEATURE_PV_EOI || 6 || overrides the generic EOI
>> + || || implementation with a
>> + || || paravirtualized version. Available
>> + || || at msr 0x4b564d04.
>
> Actually there's no override: guest can still use the
> standard EOI even if PV EOI MSR is enabled.
>
> There's a detailed explanation of PV EOI in
> Documentation/virtual/kvm/msr.txt
> I think the following would be enough:
>
> "paravirtualized end of interrupt handler can be enabled by writing to msr 0x4b564d04"
>
Okay.
Thanks Michael.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-09-12 7:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-04 8:48 [PATCH V2] Documentation/kvm: Update cpuid documentation for steal time and pv eoi Raghavendra K T
2013-09-12 5:21 ` Raghavendra K T
2013-09-12 5:44 ` Michael S. Tsirkin
2013-09-12 7:10 ` Raghavendra K T
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).