mbox series

[v2,0/2] spapr: Make the nested code TCG only

Message ID 20220325221113.255834-1-farosas@linux.ibm.com (mailing list archive)
Headers show
Series spapr: Make the nested code TCG only | expand

Message

Fabiano Rosas March 25, 2022, 10:11 p.m. UTC
The spapr virtual hypervisor implementation of the nested KVM API
depends on the first level guest to be emulated by TCG. So move the
whole code under CONFIG_TCG.

v2:

- Created hypercall_register_nested instead of reusing
  hypercall_register_softmmu;

- Rearranged the ifdef a bit to keep the hypercall_register_*
  functions closer;

- Dropped the more paranoid patch that checked for KVM at every
  call. I couldn't convince myself anymore that it was necessary.

v1:
https://lists.nongnu.org/archive/html/qemu-ppc/2022-03/msg00412.html

Fabiano Rosas (2):
  spapr: Move hypercall_register_softmmu
  spapr: Move nested KVM hypercalls under a TCG only config.

 hw/ppc/spapr_hcall.c | 74 ++++++++++++++++++++++++++------------------
 1 file changed, 44 insertions(+), 30 deletions(-)

Comments

Daniel Henrique Barboza April 20, 2022, 7:12 p.m. UTC | #1
Queued in gitlab.com/danielhb/qemu/tree/ppc-next. Thanks,


Daniel

On 3/25/22 19:11, Fabiano Rosas wrote:
> The spapr virtual hypervisor implementation of the nested KVM API
> depends on the first level guest to be emulated by TCG. So move the
> whole code under CONFIG_TCG.
> 
> v2:
> 
> - Created hypercall_register_nested instead of reusing
>    hypercall_register_softmmu;
> 
> - Rearranged the ifdef a bit to keep the hypercall_register_*
>    functions closer;
> 
> - Dropped the more paranoid patch that checked for KVM at every
>    call. I couldn't convince myself anymore that it was necessary.
> 
> v1:
> https://lists.nongnu.org/archive/html/qemu-ppc/2022-03/msg00412.html
> 
> Fabiano Rosas (2):
>    spapr: Move hypercall_register_softmmu
>    spapr: Move nested KVM hypercalls under a TCG only config.
> 
>   hw/ppc/spapr_hcall.c | 74 ++++++++++++++++++++++++++------------------
>   1 file changed, 44 insertions(+), 30 deletions(-)
>