linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Documentation/kvm : Add documentation on Hypercalls
@ 2012-05-31  8:01 Raghavendra K T
  2012-05-31 17:44 ` Randy Dunlap
  2012-05-31 17:46 ` H. Peter Anvin
  0 siblings, 2 replies; 16+ messages in thread
From: Raghavendra K T @ 2012-05-31  8:01 UTC (permalink / raw)
  To: Rob Landley, linux-kernel, linux-doc
  Cc: Srivatsa Vaddagiri, Alexander Graf, KVM, H. Peter Anvin,
	Raghavendra K T, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> 

Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ 

Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
---
diff --git a/Documentation/virtual/kvm/hypercalls.txt b/Documentation/virtual/kvm/hypercalls.txt
new file mode 100644
index 0000000..c79335a
--- /dev/null
+++ b/Documentation/virtual/kvm/hypercalls.txt
@@ -0,0 +1,46 @@
+KVM Hypercalls Documentation
+===========================
+The template for each hypercall is:
+1. Hypercall name, value.
+2. Architecture(s)
+3. Status (deprecated, obsolete, active)
+4. Purpose
+
+1. KVM_HC_VAPIC_POLL_IRQ
+------------------------
+Value: 1
+Architecture: x86
+Purpose: Trigger guest exit so that the host can check for pending
+interrupts on reentry.
+
+2. KVM_HC_MMU_OP
+------------------------
+Value: 2
+Architecture: x86
+Status: deprecated.
+Purpose: Support MMU operations such as writing to PTE,
+flushing TLB, release PT.
+
+3. KVM_HC_FEATURES
+------------------------
+Value: 3
+Architecture: PPC
+Status: active
+Purpose: Expose hypercall availability to the guest. On x86 platforms, cpuid
+used to enumerate which hypercalls are available. On PPC, either device tree
+based lookup ( which is also what EPAPR dictates) OR KVM specific enumeration
+mechanism (which is this hypercall) can be used.
+
+4. KVM_HC_PPC_MAP_MAGIC_PAGE
+------------------------
+Value: 4
+Architecture: PPC
+Status: active
+Purpose: To enable communication between the hypervisor and guest there is a
+shared page that contains parts of supervisor visible register state.
+The guest can map this shared page to access its supervisor register through
+memory using this hypercall.
+
+TODO:
+1. more information on input and output needed?
+2. Add more detail to purpose of hypercalls.


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31  8:01 [PATCH] Documentation/kvm : Add documentation on Hypercalls Raghavendra K T
@ 2012-05-31 17:44 ` Randy Dunlap
  2012-05-31 18:55   ` Jan Kiszka
  2012-06-04  7:29   ` Raghavendra K T
  2012-05-31 17:46 ` H. Peter Anvin
  1 sibling, 2 replies; 16+ messages in thread
From: Randy Dunlap @ 2012-05-31 17:44 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Rob Landley, linux-kernel, linux-doc, Srivatsa Vaddagiri,
	Alexander Graf, KVM, H. Peter Anvin, Marcelo Tosatti,
	Gleb Natapov, Ingo Molnar, Avi Kivity, Jan Kiszka

On 05/31/2012 01:01 AM, Raghavendra K T wrote:

> From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> 
> 
> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ 
> 
> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> ---
> diff --git a/Documentation/virtual/kvm/hypercalls.txt b/Documentation/virtual/kvm/hypercalls.txt
> new file mode 100644
> index 0000000..c79335a
> --- /dev/null
> +++ b/Documentation/virtual/kvm/hypercalls.txt
> @@ -0,0 +1,46 @@
> +KVM Hypercalls Documentation
> +===========================
> +The template for each hypercall is:
> +1. Hypercall name, value.
> +2. Architecture(s)
> +3. Status (deprecated, obsolete, active)
> +4. Purpose
> +
> +1. KVM_HC_VAPIC_POLL_IRQ
> +------------------------
> +Value: 1
> +Architecture: x86


No "Status", does that imply "Active" ?

> +Purpose: Trigger guest exit so that the host can check for pending
> +interrupts on reentry.
> +
> +2. KVM_HC_MMU_OP
> +------------------------
> +Value: 2
> +Architecture: x86
> +Status: deprecated.
> +Purpose: Support MMU operations such as writing to PTE,
> +flushing TLB, release PT.
> +
> +3. KVM_HC_FEATURES
> +------------------------
> +Value: 3
> +Architecture: PPC
> +Status: active
> +Purpose: Expose hypercall availability to the guest. On x86 platforms, cpuid
> +used to enumerate which hypercalls are available. On PPC, either device tree
> +based lookup ( which is also what EPAPR dictates) OR KVM specific enumeration
> +mechanism (which is this hypercall) can be used.
> +
> +4. KVM_HC_PPC_MAP_MAGIC_PAGE
> +------------------------
> +Value: 4
> +Architecture: PPC
> +Status: active
> +Purpose: To enable communication between the hypervisor and guest there is a
> +shared page that contains parts of supervisor visible register state.
> +The guest can map this shared page to access its supervisor register through
> +memory using this hypercall.
> +
> +TODO:
> +1. more information on input and output needed?
> +2. Add more detail to purpose of hypercalls.
> 
> --


-- 
~Randy

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31  8:01 [PATCH] Documentation/kvm : Add documentation on Hypercalls Raghavendra K T
  2012-05-31 17:44 ` Randy Dunlap
@ 2012-05-31 17:46 ` H. Peter Anvin
  2012-06-04  4:00   ` Rob Landley
  1 sibling, 1 reply; 16+ messages in thread
From: H. Peter Anvin @ 2012-05-31 17:46 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Rob Landley, linux-kernel, linux-doc, Srivatsa Vaddagiri,
	Alexander Graf, KVM, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 05/31/2012 01:01 AM, Raghavendra K T wrote:
> +
> +TODO:
> +1. more information on input and output needed?
> +2. Add more detail to purpose of hypercalls.
> 

1. definitely, including the hypercall ABI.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31 17:44 ` Randy Dunlap
@ 2012-05-31 18:55   ` Jan Kiszka
  2012-05-31 18:59     ` H. Peter Anvin
  2012-06-04  7:29   ` Raghavendra K T
  1 sibling, 1 reply; 16+ messages in thread
From: Jan Kiszka @ 2012-05-31 18:55 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: Raghavendra K T, Rob Landley, linux-kernel, linux-doc,
	Srivatsa Vaddagiri, Alexander Graf, KVM, H. Peter Anvin,
	Marcelo Tosatti, Gleb Natapov, Ingo Molnar, Avi Kivity

On 2012-05-31 19:44, Randy Dunlap wrote:
> On 05/31/2012 01:01 AM, Raghavendra K T wrote:
> 
>> From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> 
>>
>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ 
>>
>> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
>> ---
>> diff --git a/Documentation/virtual/kvm/hypercalls.txt b/Documentation/virtual/kvm/hypercalls.txt
>> new file mode 100644
>> index 0000000..c79335a
>> --- /dev/null
>> +++ b/Documentation/virtual/kvm/hypercalls.txt
>> @@ -0,0 +1,46 @@
>> +KVM Hypercalls Documentation
>> +===========================
>> +The template for each hypercall is:
>> +1. Hypercall name, value.
>> +2. Architecture(s)
>> +3. Status (deprecated, obsolete, active)
>> +4. Purpose
>> +
>> +1. KVM_HC_VAPIC_POLL_IRQ
>> +------------------------
>> +Value: 1
>> +Architecture: x86
> 
> 
> No "Status", does that imply "Active" ?

It should actually state that this call is in use.

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31 18:55   ` Jan Kiszka
@ 2012-05-31 18:59     ` H. Peter Anvin
  0 siblings, 0 replies; 16+ messages in thread
From: H. Peter Anvin @ 2012-05-31 18:59 UTC (permalink / raw)
  To: Jan Kiszka
  Cc: Randy Dunlap, Raghavendra K T, Rob Landley, linux-kernel,
	linux-doc, Srivatsa Vaddagiri, Alexander Graf, KVM,
	Marcelo Tosatti, Gleb Natapov, Ingo Molnar, Avi Kivity

On 05/31/2012 11:55 AM, Jan Kiszka wrote:
>>
>> No "Status", does that imply "Active" ?
> 
> It should actually state that this call is in use.
> 

It might be nice to point out when a call was introduced, and for
deprecated calls, when it was deprecated.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31 17:46 ` H. Peter Anvin
@ 2012-06-04  4:00   ` Rob Landley
  2012-06-04  8:21     ` Raghavendra K T
  0 siblings, 1 reply; 16+ messages in thread
From: Rob Landley @ 2012-06-04  4:00 UTC (permalink / raw)
  To: H. Peter Anvin
  Cc: Raghavendra K T, linux-kernel, linux-doc, Srivatsa Vaddagiri,
	Alexander Graf, KVM, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 05/31/2012 12:46 PM, H. Peter Anvin wrote:
> On 05/31/2012 01:01 AM, Raghavendra K T wrote:
>> +
>> +TODO:
>> +1. more information on input and output needed?
>> +2. Add more detail to purpose of hypercalls.
>>
> 
> 1. definitely, including the hypercall ABI.
> 
> 	-hpa
> 

I was wondering about that. It looks like
Documentation/virtual/kvm/api.txt might cover some of that already in
section 5, but it doesn't look complete...

Also, could I get a 00-INDEX file for this directory explaining what
these individual files are? I think "api.txt" is supposed to be
host-side API for controlling a guest VM (from userspace via ioctls,
looks like), and hypercalls.txt is guest-side API for poking the host.
How someone would write host-side code that _responds_ to a hypercall, I
have no idea.  (It goes in the host kernel?)

Rob



-- 
GNU/Linux isn't: Linux=GPLv2, GNU=GPLv3+, they can't share code.
Either it's "mere aggregation", or a license violation.  Pick one.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-05-31 17:44 ` Randy Dunlap
  2012-05-31 18:55   ` Jan Kiszka
@ 2012-06-04  7:29   ` Raghavendra K T
  1 sibling, 0 replies; 16+ messages in thread
From: Raghavendra K T @ 2012-06-04  7:29 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: Rob Landley, linux-kernel, linux-doc, Srivatsa Vaddagiri,
	Alexander Graf, KVM, H. Peter Anvin, Marcelo Tosatti,
	Gleb Natapov, Ingo Molnar, Avi Kivity, Jan Kiszka

On 05/31/2012 11:14 PM, Randy Dunlap wrote:
> On 05/31/2012 01:01 AM, Raghavendra K T wrote:
>
>> From: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>
>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ
>>
>> Signed-off-by: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>> ---
>> diff --git a/Documentation/virtual/kvm/hypercalls.txt b/Documentation/virtual/kvm/hypercalls.txt
>> new file mode 100644
>> index 0000000..c79335a
>> --- /dev/null
>> +++ b/Documentation/virtual/kvm/hypercalls.txt
>> @@ -0,0 +1,46 @@
>> +KVM Hypercalls Documentation
>> +===========================
>> +The template for each hypercall is:
>> +1. Hypercall name, value.
>> +2. Architecture(s)
>> +3. Status (deprecated, obsolete, active)
>> +4. Purpose
>> +
>> +1. KVM_HC_VAPIC_POLL_IRQ
>> +------------------------
>> +Value: 1
>> +Architecture: x86
>
>
> No "Status", does that imply "Active" ?
>

Oh! It was unintentionally left out. will add that.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-06-04  4:00   ` Rob Landley
@ 2012-06-04  8:21     ` Raghavendra K T
  0 siblings, 0 replies; 16+ messages in thread
From: Raghavendra K T @ 2012-06-04  8:21 UTC (permalink / raw)
  To: Rob Landley
  Cc: H. Peter Anvin, linux-kernel, linux-doc, Srivatsa Vaddagiri,
	Alexander Graf, KVM, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 06/04/2012 09:30 AM, Rob Landley wrote:
> On 05/31/2012 12:46 PM, H. Peter Anvin wrote:
>> On 05/31/2012 01:01 AM, Raghavendra K T wrote:
>>> +
>>> +TODO:
>>> +1. more information on input and output needed?
>>> +2. Add more detail to purpose of hypercalls.
>>>
>>
>> 1. definitely, including the hypercall ABI.
>>
>> 	-hpa
>>
>
> I was wondering about that. It looks like
> Documentation/virtual/kvm/api.txt might cover some of that already in
> section 5, but it doesn't look complete...
>
> Also, could I get a 00-INDEX file for this directory explaining what
> these individual files are? I think "api.txt" is supposed to be
> host-side API for controlling a guest VM (from userspace via ioctls,
> looks like),

api.txt has  plenty of information apart from  features which are 
related to guest side api (generally  controls / knows PV features 
available, that may also involve host query of guest cpuid).
and capability (of) related to host.

  and hypercalls.txt is guest-side API for poking the host.

hypercall can be used to take some actions (as needed by guest / 
exchange information)


> How someone would write host-side code that _responds_ to a hypercall, I
> have no idea.  (It goes in the host kernel?)
>
> Rob
>


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-08-02  7:08       ` Raghavendra K T
@ 2012-08-02 10:13         ` Alexander Graf
  0 siblings, 0 replies; 16+ messages in thread
From: Alexander Graf @ 2012-08-02 10:13 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Marcelo Tosatti, Rob Landley, linux-kernel, linux-doc,
	Randy Dunlap, KVM, H. Peter Anvin, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka



On 02.08.2012, at 09:08, Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> wrote:

> On 08/01/2012 11:55 PM, Marcelo Tosatti wrote:
>> On Wed, Aug 01, 2012 at 04:19:01PM +0530, Raghavendra K T wrote:
>>> On 08/01/2012 08:37 AM, Marcelo Tosatti wrote:
>>>> On Tue, Jul 24, 2012 at 02:23:59PM +0530, Raghavendra K T wrote:
>>>>> From: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>>>> 
>>>>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
>>>>> and Peter (HPA) for suggesting hypercall ABI addition.
>>>>> 
>>>>> Signed-off-by: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>>>> ---
> [...]
>>>>> +
>>>>> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
>>>>> + The hypercall number should be placed in rax and the return value will be
>>>>> + placed in rax.  No other registers will be clobbered unless explicitly stated
>>>>> + by the particular hypercall.
>>>> 
>>>> It depends on the hypercall. It happens that current hypercalls use
>>>> the four registers, but its not an ABI (hyper-v hypercalls uses r8, for example).
>>>> 
>>> 
>>> Okay, agree. I did not know about hyper-v. Above one is for X86 and
>>> KVM  (and not X86 / hyper-v). So I should remove ABI and probably say,
>> 
>> Just say "Linux x86 Hypercall", yeah.
>> 
> 
> Ok, Thanks Marcelo.will respin patches with the changes you suggested.
> 
> Also I would  like add Alex's description on ppc hypercalls
> into Documentation/virtual/kvm/ppc-pv.txt with Alex from/sob or
> suggested-by in a separate patch unless I see any objection to that.
> 

Yeah, don't stall this on me :)

Alex

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-08-01 18:25     ` Marcelo Tosatti
@ 2012-08-02  7:08       ` Raghavendra K T
  2012-08-02 10:13         ` Alexander Graf
  0 siblings, 1 reply; 16+ messages in thread
From: Raghavendra K T @ 2012-08-02  7:08 UTC (permalink / raw)
  To: Marcelo Tosatti, Alexander Graf
  Cc: Rob Landley, linux-kernel, linux-doc, Randy Dunlap, KVM,
	H. Peter Anvin, Gleb Natapov, Ingo Molnar, Avi Kivity,
	Jan Kiszka

On 08/01/2012 11:55 PM, Marcelo Tosatti wrote:
> On Wed, Aug 01, 2012 at 04:19:01PM +0530, Raghavendra K T wrote:
>> On 08/01/2012 08:37 AM, Marcelo Tosatti wrote:
>>> On Tue, Jul 24, 2012 at 02:23:59PM +0530, Raghavendra K T wrote:
>>>> From: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>>>
>>>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
>>>> and Peter (HPA) for suggesting hypercall ABI addition.
>>>>
>>>> Signed-off-by: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>>> ---
[...]
>>>> +
>>>> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
>>>> + The hypercall number should be placed in rax and the return value will be
>>>> + placed in rax.  No other registers will be clobbered unless explicitly stated
>>>> + by the particular hypercall.
>>>
>>> It depends on the hypercall. It happens that current hypercalls use
>>> the four registers, but its not an ABI (hyper-v hypercalls uses r8, for example).
>>>
>>
>> Okay, agree. I did not know about hyper-v. Above one is for X86 and
>> KVM  (and not X86 / hyper-v). So I should remove ABI and probably say,
>
> Just say "Linux x86 Hypercall", yeah.
>

Ok, Thanks Marcelo.will respin patches with the changes you suggested.

Also I would  like add Alex's description on ppc hypercalls
into Documentation/virtual/kvm/ppc-pv.txt with Alex from/sob or
suggested-by in a separate patch unless I see any objection to that.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-08-01 10:49   ` Raghavendra K T
@ 2012-08-01 18:25     ` Marcelo Tosatti
  2012-08-02  7:08       ` Raghavendra K T
  0 siblings, 1 reply; 16+ messages in thread
From: Marcelo Tosatti @ 2012-08-01 18:25 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Rob Landley, linux-kernel, linux-doc, Alexander Graf,
	Randy Dunlap, KVM, H. Peter Anvin, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On Wed, Aug 01, 2012 at 04:19:01PM +0530, Raghavendra K T wrote:
> On 08/01/2012 08:37 AM, Marcelo Tosatti wrote:
> >On Tue, Jul 24, 2012 at 02:23:59PM +0530, Raghavendra K T wrote:
> >>From: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
> >>
> >>Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
> >>and Peter (HPA) for suggesting hypercall ABI addition.
> >>
> >>Signed-off-by: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
> >>---
> >>Please have a closer look at Hypercall ABI newly added
> >>
> >>Changes since last post:
> >>  - Added hypercall ABI (Peter)
> >>  - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)
> >>
> >>TODO: We need to add history details of each hypercall as suggested by HPA,
> >>which I could not trace easily. Hope it is easy for hypercall authors
> >>
> >>  Documentation/virtual/hypercalls.txt |   71 ++++++++++++++++++++++++++++++++++
> >>  1 files changed, 71 insertions(+), 0 deletions(-)
> >>  create mode 100644 Documentation/virtual/hypercalls.txt
> >>
> >>diff --git a/Documentation/virtual/hypercalls.txt b/Documentation/virtual/hypercalls.txt
> >>new file mode 100644
> >>index 0000000..caffc08
> >>--- /dev/null
> >>+++ b/Documentation/virtual/hypercalls.txt
> >>@@ -0,0 +1,71 @@
> >>+Hypercall ABI:
> >>+=============
> >>+A brief look at calling conventions of X86, S390 and PPC
> >>+X86:
> >>+ KVM Hypercalls have a three-byte sequence of either the vmrun or the vmmrun
> >>+ instruction. The hypervisor can replace it with instructions that are
> >>+ guaranteed to be supported.
> >
> >vmcall.
> 
> How is this:
>  KVM Hypercalls have a three-byte sequence of vmcall instruction. But
> The hypervisor can replace it with the instructions that are guaranteed
> to be supported.
> 
> (Related question: Is it not that vmcall instruction is patched by 0f
> 01 09 vmmcall instruction sequence for amd?. 

Yes, it is.

Linux guest binary contains the Intel vmcall instruction, which is
patched on AMD to vmmcall.

> So I think we need to
> change documentation I referred from arch/x86/include/asm/kvm_para.h.
> It should be vmcall / vmmcall instead of vmrun and vmmrun)

Yes.

> >>+
> >>+ Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
> >>+ The hypercall number should be placed in rax and the return value will be
> >>+ placed in rax.  No other registers will be clobbered unless explicitly stated
> >>+ by the particular hypercall.
> >
> >It depends on the hypercall. It happens that current hypercalls use
> >the four registers, but its not an ABI (hyper-v hypercalls uses r8, for example).
> >
> 
> Okay, agree. I did not know about hyper-v. Above one is for X86 and
> KVM  (and not X86 / hyper-v). So I should remove ABI and probably say,

Just say "Linux x86 Hypercall", yeah.

> KVM x86 Hypercall
> ==================
> Up to four arguments may be passed in rbx, rcx, rdx, and rsi
> respectively. The hypercall number should be placed in rax and the
> return value will be placed in rax.  No other registers will be
> clobbered unless explicitly stated by the particular hypercall.
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-08-01  3:07 ` Marcelo Tosatti
@ 2012-08-01 10:49   ` Raghavendra K T
  2012-08-01 18:25     ` Marcelo Tosatti
  0 siblings, 1 reply; 16+ messages in thread
From: Raghavendra K T @ 2012-08-01 10:49 UTC (permalink / raw)
  To: Marcelo Tosatti
  Cc: Rob Landley, linux-kernel, linux-doc, Alexander Graf,
	Randy Dunlap, KVM, H. Peter Anvin, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 08/01/2012 08:37 AM, Marcelo Tosatti wrote:
> On Tue, Jul 24, 2012 at 02:23:59PM +0530, Raghavendra K T wrote:
>> From: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>>
>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
>> and Peter (HPA) for suggesting hypercall ABI addition.
>>
>> Signed-off-by: Raghavendra K T<raghavendra.kt@linux.vnet.ibm.com>
>> ---
>> Please have a closer look at Hypercall ABI newly added
>>
>> Changes since last post:
>>   - Added hypercall ABI (Peter)
>>   - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)
>>
>> TODO: We need to add history details of each hypercall as suggested by HPA,
>> which I could not trace easily. Hope it is easy for hypercall authors
>>
>>   Documentation/virtual/hypercalls.txt |   71 ++++++++++++++++++++++++++++++++++
>>   1 files changed, 71 insertions(+), 0 deletions(-)
>>   create mode 100644 Documentation/virtual/hypercalls.txt
>>
>> diff --git a/Documentation/virtual/hypercalls.txt b/Documentation/virtual/hypercalls.txt
>> new file mode 100644
>> index 0000000..caffc08
>> --- /dev/null
>> +++ b/Documentation/virtual/hypercalls.txt
>> @@ -0,0 +1,71 @@
>> +Hypercall ABI:
>> +=============
>> +A brief look at calling conventions of X86, S390 and PPC
>> +X86:
>> + KVM Hypercalls have a three-byte sequence of either the vmrun or the vmmrun
>> + instruction. The hypervisor can replace it with instructions that are
>> + guaranteed to be supported.
>
> vmcall.

How is this:
  KVM Hypercalls have a three-byte sequence of vmcall instruction. But
The hypervisor can replace it with the instructions that are guaranteed
to be supported.

(Related question: Is it not that vmcall instruction is patched by 0f
01 09 vmmcall instruction sequence for amd?. So I think we need to
change documentation I referred from arch/x86/include/asm/kvm_para.h.
It should be vmcall / vmmcall instead of vmrun and vmmrun)

>
>> +
>> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
>> + The hypercall number should be placed in rax and the return value will be
>> + placed in rax.  No other registers will be clobbered unless explicitly stated
>> + by the particular hypercall.
>
> It depends on the hypercall. It happens that current hypercalls use
> the four registers, but its not an ABI (hyper-v hypercalls uses r8, for example).
>

Okay, agree. I did not know about hyper-v. Above one is for X86 and
KVM  (and not X86 / hyper-v). So I should remove ABI and probably say,

KVM x86 Hypercall
==================
Up to four arguments may be passed in rbx, rcx, rdx, and rsi
respectively. The hypercall number should be placed in rax and the
return value will be placed in rax.  No other registers will be
clobbered unless explicitly stated by the particular hypercall.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-07-24  8:53 Raghavendra K T
  2012-07-24 12:13 ` Alexander Graf
@ 2012-08-01  3:07 ` Marcelo Tosatti
  2012-08-01 10:49   ` Raghavendra K T
  1 sibling, 1 reply; 16+ messages in thread
From: Marcelo Tosatti @ 2012-08-01  3:07 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Rob Landley, linux-kernel, linux-doc, Alexander Graf,
	Randy Dunlap, KVM, H. Peter Anvin, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On Tue, Jul 24, 2012 at 02:23:59PM +0530, Raghavendra K T wrote:
> From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> 
> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
> and Peter (HPA) for suggesting hypercall ABI addition.
> 
> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> ---
> Please have a closer look at Hypercall ABI newly added
> 
> Changes since last post:
>  - Added hypercall ABI (Peter)
>  - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)
> 
> TODO: We need to add history details of each hypercall as suggested by HPA,
> which I could not trace easily. Hope it is easy for hypercall authors
> 
>  Documentation/virtual/hypercalls.txt |   71 ++++++++++++++++++++++++++++++++++
>  1 files changed, 71 insertions(+), 0 deletions(-)
>  create mode 100644 Documentation/virtual/hypercalls.txt
> 
> diff --git a/Documentation/virtual/hypercalls.txt b/Documentation/virtual/hypercalls.txt
> new file mode 100644
> index 0000000..caffc08
> --- /dev/null
> +++ b/Documentation/virtual/hypercalls.txt
> @@ -0,0 +1,71 @@
> +Hypercall ABI:
> +=============
> +A brief look at calling conventions of X86, S390 and PPC
> +X86:
> + KVM Hypercalls have a three-byte sequence of either the vmrun or the vmmrun
> + instruction. The hypervisor can replace it with instructions that are
> + guaranteed to be supported.

vmcall.

> +
> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
> + The hypercall number should be placed in rax and the return value will be
> + placed in rax.  No other registers will be clobbered unless explicitly stated
> + by the particular hypercall.

It depends on the hypercall. It happens that current hypercalls use
the four registers, but its not an ABI (hyper-v hypercalls uses r8, for example).


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-07-24 12:13 ` Alexander Graf
@ 2012-07-24 13:14   ` Raghavendra K T
  0 siblings, 0 replies; 16+ messages in thread
From: Raghavendra K T @ 2012-07-24 13:14 UTC (permalink / raw)
  To: Alexander Graf
  Cc: Rob Landley, linux-kernel, linux-doc, Randy Dunlap, KVM,
	H. Peter Anvin, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 07/24/2012 05:43 PM, Alexander Graf wrote:
> On 07/24/2012 10:53 AM, Raghavendra K T wrote:
>> From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
>>
>> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
>> and Peter (HPA) for suggesting hypercall ABI addition.
>>
>> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
>> ---
>> Please have a closer look at Hypercall ABI newly added
>>
>> Changes since last post:
>> - Added hypercall ABI (Peter)
>> - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)
>>
>> TODO: We need to add history details of each hypercall as suggested by
>> HPA,
>> which I could not trace easily. Hope it is easy for hypercall authors
>>
>> Documentation/virtual/hypercalls.txt | 71
>> ++++++++++++++++++++++++++++++++++
>> 1 files changed, 71 insertions(+), 0 deletions(-)
>> create mode 100644 Documentation/virtual/hypercalls.txt
>>
>> diff --git a/Documentation/virtual/hypercalls.txt
>> b/Documentation/virtual/hypercalls.txt
>> new file mode 100644
>> index 0000000..caffc08
>> --- /dev/null
>> +++ b/Documentation/virtual/hypercalls.txt
>> @@ -0,0 +1,71 @@
>> +Hypercall ABI:
>> +=============
>> +A brief look at calling conventions of X86, S390 and PPC
>> +X86:
>> + KVM Hypercalls have a three-byte sequence of either the vmrun or the
>> vmmrun
>> + instruction. The hypervisor can replace it with instructions that are
>> + guaranteed to be supported.
>> +
>> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi
>> respectively.
>> + The hypercall number should be placed in rax and the return value
>> will be
>> + placed in rax. No other registers will be clobbered unless
>> explicitly stated
>> + by the particular hypercall.
>> +
>> +S390:
>> + R2-R7 are used for parameters 1-6. In addition, R1 is used for
>> hypercall
>> + number. The return value is written to R2.
>> +
>> + S390 uses diagnose instruction as hypercall (0x500) along with
>> hypercall
>> + number in R1.
>> +
>> + PoewerPC:
>
> PowerPC

Sorry for the typo.

>
>> + It uses R3-R10 and hypercall number in R11. R4-R11 are used as
>> output registers.
>> + Return value is placed in R3.
>> +
>> + KVM hypercalls uses 4 byte opcode, that are patched with
>> 'hypercall-instructions'
>> + property inside the device tree's /hypervisor node.
>> + For more information refer to Documentation/virtual/kvm/ppc-pv.txt
>
> What exactly is this document supposed to cover? We have 3 different
> hypercall ABIs in KVM on PowerPC:

Thanks for the detailed information on PPC hypercalls.
My intention was to cover KVM specific hypercalls here.

Patch is wrongly generated against Documentation/virtual/ instead of
Documentation/virtual/kvm will correct that.

Information you have provided should be very useful in understanding
PPC hypercalls. Do you wish to have (2), (3) below goto some doc?


> 1) KVM hypercalls (ePAPR)
>
> The ePAPR compliant hypercall implementation. This one is used for KVM
> specific hypercalls. All hypercalls get a KVM vendor prefix (42) and
> then the hypercall id. We also implement generic hypercalls here, like
> the ePAPR idle hcall. The instruction sequence for these is listed in
> the device tree, as you noted above. It's available on all targets.
>
> 2) PAPR hypercalls
>
> To run server PowerPC PAPR guests (-M pseries in QEMU), we need to
> handle PAPR hypercalls. These are the same hypercalls that pHyp, the
> POWER hypervisor implements. Some of them are handled in the kernel,
> some are handled in user space. This is only available on book3s_64.
>
> 3) OSI hypercalls
>
> In parallel to QEMU, there is another user of KVM on PowerPC:
> Mac-on-Linux. That project had its own hypercalls long before we came
> along with KVM, so to maintain compatibility we also support their
> hypercalls. All of these get forwarded to user space. This is only
> useful on book3s_32, but can be used with book3s_64 as well.
>
>> +
>> +KVM Hypercalls Documentation
>> +===========================
>> +The template for each hypercall is:
>> +1. Hypercall name, value.
>> +2. Architecture(s)
>> +3. Status (deprecated, obsolete, active)
>> +4. Purpose
>> +
>> +1. KVM_HC_VAPIC_POLL_IRQ
>> +------------------------
>> +Value: 1
>
> What are the value fields supposed to mean?

Value field is just #define value of the hypercall in header.
If that does not make much sense probably it can be dropped.


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH] Documentation/kvm : Add documentation on Hypercalls
  2012-07-24  8:53 Raghavendra K T
@ 2012-07-24 12:13 ` Alexander Graf
  2012-07-24 13:14   ` Raghavendra K T
  2012-08-01  3:07 ` Marcelo Tosatti
  1 sibling, 1 reply; 16+ messages in thread
From: Alexander Graf @ 2012-07-24 12:13 UTC (permalink / raw)
  To: Raghavendra K T
  Cc: Rob Landley, linux-kernel, linux-doc, Randy Dunlap, KVM,
	H. Peter Anvin, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

On 07/24/2012 10:53 AM, Raghavendra K T wrote:
> From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
>
> Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
> and Peter (HPA) for suggesting hypercall ABI addition.
>
> Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
> ---
> Please have a closer look at Hypercall ABI newly added
>
> Changes since last post:
>   - Added hypercall ABI (Peter)
>   - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)
>
> TODO: We need to add history details of each hypercall as suggested by HPA,
> which I could not trace easily. Hope it is easy for hypercall authors
>
>   Documentation/virtual/hypercalls.txt |   71 ++++++++++++++++++++++++++++++++++
>   1 files changed, 71 insertions(+), 0 deletions(-)
>   create mode 100644 Documentation/virtual/hypercalls.txt
>
> diff --git a/Documentation/virtual/hypercalls.txt b/Documentation/virtual/hypercalls.txt
> new file mode 100644
> index 0000000..caffc08
> --- /dev/null
> +++ b/Documentation/virtual/hypercalls.txt
> @@ -0,0 +1,71 @@
> +Hypercall ABI:
> +=============
> +A brief look at calling conventions of X86, S390 and PPC
> +X86:
> + KVM Hypercalls have a three-byte sequence of either the vmrun or the vmmrun
> + instruction. The hypervisor can replace it with instructions that are
> + guaranteed to be supported.
> +
> + Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
> + The hypercall number should be placed in rax and the return value will be
> + placed in rax.  No other registers will be clobbered unless explicitly stated
> + by the particular hypercall.
> +
> +S390:
> +  R2-R7 are used for parameters 1-6. In addition, R1 is used for hypercall
> +  number. The return value is written to R2.
> +
> +  S390 uses diagnose instruction as hypercall (0x500) along with hypercall
> +  number in R1.
> +
> + PoewerPC:

PowerPC

> +  It uses R3-R10 and hypercall number in R11. R4-R11 are used as output registers.
> +  Return value is placed in R3.
> +
> +  KVM hypercalls uses 4 byte opcode, that are patched with 'hypercall-instructions'
> +  property inside the device tree's /hypervisor node.
> +  For more information refer to Documentation/virtual/kvm/ppc-pv.txt

What exactly is this document supposed to cover? We have 3 different 
hypercall ABIs in KVM on PowerPC:

   1) KVM hypercalls (ePAPR)

The ePAPR compliant hypercall implementation. This one is used for KVM 
specific hypercalls. All hypercalls get a KVM vendor prefix (42) and 
then the hypercall id. We also implement generic hypercalls here, like 
the ePAPR idle hcall. The instruction sequence for these is listed in 
the device tree, as you noted above. It's available on all targets.

   2) PAPR hypercalls

To run server PowerPC PAPR guests (-M pseries in QEMU), we need to 
handle PAPR hypercalls. These are the same hypercalls that pHyp, the 
POWER hypervisor implements. Some of them are handled in the kernel, 
some are handled in user space. This is only available on book3s_64.

   3) OSI hypercalls

In parallel to QEMU, there is another user of KVM on PowerPC: 
Mac-on-Linux. That project had its own hypercalls long before we came 
along with KVM, so to maintain compatibility we also support their 
hypercalls. All of these get forwarded to user space. This is only 
useful on book3s_32, but can be used with book3s_64 as well.

> +
> +KVM Hypercalls Documentation
> +===========================
> +The template for each hypercall is:
> +1. Hypercall name, value.
> +2. Architecture(s)
> +3. Status (deprecated, obsolete, active)
> +4. Purpose
> +
> +1. KVM_HC_VAPIC_POLL_IRQ
> +------------------------
> +Value: 1

What are the value fields supposed to mean?


Alex

> +Architecture: x86
> +Status: active
> +Purpose: Trigger guest exit so that the host can check for pending
> +interrupts on reentry.
> +
> +2. KVM_HC_MMU_OP
> +------------------------
> +Value: 2
> +Architecture: x86
> +Status: deprecated.
> +Purpose: Support MMU operations such as writing to PTE,
> +flushing TLB, release PT.
> +
> +3. KVM_HC_FEATURES
> +------------------------
> +Value: 3
> +Architecture: PPC
> +Status: active
> +Purpose: Expose hypercall availability to the guest. On x86 platforms, cpuid
> +used to enumerate which hypercalls are available. On PPC, either device tree
> +based lookup ( which is also what EPAPR dictates) OR KVM specific enumeration
> +mechanism (which is this hypercall) can be used.
> +
> +4. KVM_HC_PPC_MAP_MAGIC_PAGE
> +------------------------
> +Value: 4
> +Architecture: PPC
> +Status: active
> +Purpose: To enable communication between the hypervisor and guest there is a
> +shared page that contains parts of supervisor visible register state.
> +The guest can map this shared page to access its supervisor register through
> +memory using this hypercall.
>


^ permalink raw reply	[flat|nested] 16+ messages in thread

* [PATCH] Documentation/kvm : Add documentation on Hypercalls
@ 2012-07-24  8:53 Raghavendra K T
  2012-07-24 12:13 ` Alexander Graf
  2012-08-01  3:07 ` Marcelo Tosatti
  0 siblings, 2 replies; 16+ messages in thread
From: Raghavendra K T @ 2012-07-24  8:53 UTC (permalink / raw)
  To: Rob Landley, linux-kernel, linux-doc
  Cc: Alexander Graf, Randy Dunlap, KVM, H. Peter Anvin,
	Raghavendra K T, Marcelo Tosatti, Gleb Natapov, Ingo Molnar,
	Avi Kivity, Jan Kiszka

From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>

Thanks Alex for KVM_HC_FEATURES inputs and Jan for VAPIC_POLL_IRQ,
and Peter (HPA) for suggesting hypercall ABI addition.

Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
---
Please have a closer look at Hypercall ABI newly added

Changes since last post:
 - Added hypercall ABI (Peter)
 - made KVM_HC_VAPIC_POLL_IRQ active explicitly (Randy)

TODO: We need to add history details of each hypercall as suggested by HPA,
which I could not trace easily. Hope it is easy for hypercall authors

 Documentation/virtual/hypercalls.txt |   71 ++++++++++++++++++++++++++++++++++
 1 files changed, 71 insertions(+), 0 deletions(-)
 create mode 100644 Documentation/virtual/hypercalls.txt

diff --git a/Documentation/virtual/hypercalls.txt b/Documentation/virtual/hypercalls.txt
new file mode 100644
index 0000000..caffc08
--- /dev/null
+++ b/Documentation/virtual/hypercalls.txt
@@ -0,0 +1,71 @@
+Hypercall ABI:
+=============
+A brief look at calling conventions of X86, S390 and PPC
+X86:
+ KVM Hypercalls have a three-byte sequence of either the vmrun or the vmmrun
+ instruction. The hypervisor can replace it with instructions that are
+ guaranteed to be supported.
+
+ Up to four arguments may be passed in rbx, rcx, rdx, and rsi respectively.
+ The hypercall number should be placed in rax and the return value will be
+ placed in rax.  No other registers will be clobbered unless explicitly stated
+ by the particular hypercall.
+
+S390:
+  R2-R7 are used for parameters 1-6. In addition, R1 is used for hypercall
+  number. The return value is written to R2.
+
+  S390 uses diagnose instruction as hypercall (0x500) along with hypercall
+  number in R1.
+
+ PoewerPC:
+  It uses R3-R10 and hypercall number in R11. R4-R11 are used as output registers.
+  Return value is placed in R3.
+
+  KVM hypercalls uses 4 byte opcode, that are patched with 'hypercall-instructions'
+  property inside the device tree's /hypervisor node.
+  For more information refer to Documentation/virtual/kvm/ppc-pv.txt
+
+KVM Hypercalls Documentation
+===========================
+The template for each hypercall is:
+1. Hypercall name, value.
+2. Architecture(s)
+3. Status (deprecated, obsolete, active)
+4. Purpose
+
+1. KVM_HC_VAPIC_POLL_IRQ
+------------------------
+Value: 1
+Architecture: x86
+Status: active
+Purpose: Trigger guest exit so that the host can check for pending
+interrupts on reentry.
+
+2. KVM_HC_MMU_OP
+------------------------
+Value: 2
+Architecture: x86
+Status: deprecated.
+Purpose: Support MMU operations such as writing to PTE,
+flushing TLB, release PT.
+
+3. KVM_HC_FEATURES
+------------------------
+Value: 3
+Architecture: PPC
+Status: active
+Purpose: Expose hypercall availability to the guest. On x86 platforms, cpuid
+used to enumerate which hypercalls are available. On PPC, either device tree
+based lookup ( which is also what EPAPR dictates) OR KVM specific enumeration
+mechanism (which is this hypercall) can be used.
+
+4. KVM_HC_PPC_MAP_MAGIC_PAGE
+------------------------
+Value: 4
+Architecture: PPC
+Status: active
+Purpose: To enable communication between the hypervisor and guest there is a
+shared page that contains parts of supervisor visible register state.
+The guest can map this shared page to access its supervisor register through
+memory using this hypercall.


^ permalink raw reply related	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2012-08-02 10:12 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-31  8:01 [PATCH] Documentation/kvm : Add documentation on Hypercalls Raghavendra K T
2012-05-31 17:44 ` Randy Dunlap
2012-05-31 18:55   ` Jan Kiszka
2012-05-31 18:59     ` H. Peter Anvin
2012-06-04  7:29   ` Raghavendra K T
2012-05-31 17:46 ` H. Peter Anvin
2012-06-04  4:00   ` Rob Landley
2012-06-04  8:21     ` Raghavendra K T
2012-07-24  8:53 Raghavendra K T
2012-07-24 12:13 ` Alexander Graf
2012-07-24 13:14   ` Raghavendra K T
2012-08-01  3:07 ` Marcelo Tosatti
2012-08-01 10:49   ` Raghavendra K T
2012-08-01 18:25     ` Marcelo Tosatti
2012-08-02  7:08       ` Raghavendra K T
2012-08-02 10:13         ` Alexander Graf

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).