[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/12] spapr-rtas: add CPU argument to RTAS call
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 07/12] spapr-rtas: add CPU argument to RTAS calls |
Date: |
Thu, 20 Jun 2013 11:10:36 -0500 |
User-agent: |
Notmuch/0.15.2+77~g661dcf8 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Andreas Färber <address@hidden> writes:
> Am 19.06.2013 22:40, schrieb Anthony Liguori:
>> RTAS is a hypervisor provided binary blob that a guest loads and
>> calls into to execute certain functions. It's similar to the
>> vsyscall page in Linux or the short lived VMCI paravirt interface
>> from VMware.
>>
>> The QEMU implementation of the RTAS blob is simply a passthrough
>> that proxies all RTAS calls to the hypervisor via an hypercall.
>>
>> While we pass a CPU argument for hypercall handling in QEMU, we
>> don't pass it for RTAS calls. Since some RTAs calls require
>
> "RTAS"
>
>> making hypercalls (normally RTAS is implemented as guest code) we
>> have nasty hacks to allow that.
>
> Where are such nasty hacks being removed? I just see the cpu argument
> propagated mostly unused throughout code.
[PATCH 08/12] spapr-rtas: use hypercall interface and remove special vty
interfaces
Regards,
Anthony Liguori
>
>>
>> Add a CPU argument to RTAS call handling so we can more easily
>> invoke hypercalls just as guest code would.
>>
>> Signed-off-by: Anthony Liguori <address@hidden>
>> ---
>> hw/nvram/spapr_nvram.c | 4 ++--
>> hw/ppc/spapr_events.c | 2 +-
>> hw/ppc/spapr_hcall.c | 2 +-
>> hw/ppc/spapr_pci.c | 13 +++++++------
>> hw/ppc/spapr_rtas.c | 21 +++++++++++----------
>> hw/ppc/spapr_vio.c | 6 ++++--
>> hw/ppc/xics.c | 12 ++++++++----
>> include/hw/ppc/spapr.h | 5 +++--
>> 8 files changed, 37 insertions(+), 28 deletions(-)
>
> Otherwise,
>
> Reviewed-by: Andreas Färber <address@hidden>
>
> Andreas
>
> --
> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- Re: [Qemu-devel] [PATCH 08/12] spapr-rtas: use hypercall interface and remove special vty interfaces, (continued)
[Qemu-devel] [PATCH 12/12] spapr-vty: remove unfixable FIXME, Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 10/12] spapr-vty: refactor the code to improve consistency, Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 04/12] qtest: add interface to save/restore, Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 07/12] spapr-rtas: add CPU argument to RTAS calls, Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 11/12] spapr-vio: pass type to spapr_vio_find_by_reg(), Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 05/12] spapr-vty: add qtest test case, Anthony Liguori, 2013/06/19
[Qemu-devel] [PATCH 02/12] qtest: add spapr hypercall support, Anthony Liguori, 2013/06/19