From: Alexander Graf <agraf-l3A5Bk7waGM@public.gmane.org>
To: Benjamin Herrenschmidt
<benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
Cc: Segher Boessenkool
<segher-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>,
linuxppc-dev
<linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
KVM list <kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
kvm-ppc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 27/27] KVM: PPC: Add Documentation about PV interface
Date: Sun, 4 Jul 2010 11:04:11 +0200 [thread overview]
Message-ID: <79514591-DCC1-4D9E-AFB7-AA985ADF3C0F@suse.de> (raw)
In-Reply-To: <1278196909.4200.389.camel@pasglop>
On 04.07.2010, at 00:41, Benjamin Herrenschmidt wrote:
> On Fri, 2010-07-02 at 18:27 +0200, Segher Boessenkool wrote:
>>> +To find out if we're running on KVM or not, we overlay the PVR
>>> register. Usually
>>> +the PVR register contains an id that identifies your CPU type. If,
>>> however, you
>>> +pass KVM_PVR_PARA in the register that you want the PVR result in,
>>> the register
>>> +still contains KVM_PVR_PARA after the mfpvr call.
>>> +
>>> + LOAD_REG_IMM(r5, KVM_PVR_PARA)
>>> + mfpvr r5
>>> + [r5 still contains KVM_PVR_PARA]
>>
>> I love this part :-)
>
> Me not :-)
>
> It should be in the device-tree instead, or something like that. Enough
> games with PVR...
My biggest concern about putting things in the device-tree is that I was trying to keep things as separate as possible. Why does the firmware have to know that it's running in KVM? Why do I have to patch 3 projects (Linux, OpenBIOS, Qemu) when I could go with patching a single one (Linux)?
Alex
WARNING: multiple messages have this Message-ID (diff)
From: Alexander Graf <agraf@suse.de>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: kvm-ppc@vger.kernel.org,
linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
KVM list <kvm@vger.kernel.org>
Subject: Re: [PATCH 27/27] KVM: PPC: Add Documentation about PV interface
Date: Sun, 4 Jul 2010 11:04:11 +0200 [thread overview]
Message-ID: <79514591-DCC1-4D9E-AFB7-AA985ADF3C0F@suse.de> (raw)
In-Reply-To: <1278196909.4200.389.camel@pasglop>
On 04.07.2010, at 00:41, Benjamin Herrenschmidt wrote:
> On Fri, 2010-07-02 at 18:27 +0200, Segher Boessenkool wrote:
>>> +To find out if we're running on KVM or not, we overlay the PVR =20
>>> register. Usually
>>> +the PVR register contains an id that identifies your CPU type. If, =20=
>>> however, you
>>> +pass KVM_PVR_PARA in the register that you want the PVR result in, =20=
>>> the register
>>> +still contains KVM_PVR_PARA after the mfpvr call.
>>> +
>>> + LOAD_REG_IMM(r5, KVM_PVR_PARA)
>>> + mfpvr r5
>>> + [r5 still contains KVM_PVR_PARA]
>>=20
>> I love this part :-)
>=20
> Me not :-)
>=20
> It should be in the device-tree instead, or something like that. =
Enough
> games with PVR...
My biggest concern about putting things in the device-tree is that I was =
trying to keep things as separate as possible. Why does the firmware =
have to know that it's running in KVM? Why do I have to patch 3 projects =
(Linux, OpenBIOS, Qemu) when I could go with patching a single one =
(Linux)?
Alex
WARNING: multiple messages have this Message-ID (diff)
From: Alexander Graf <agraf@suse.de>
To: Benjamin Herrenschmidt
<benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
Cc: Segher Boessenkool
<segher-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>,
linuxppc-dev
<linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
KVM list <kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
kvm-ppc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 27/27] KVM: PPC: Add Documentation about PV interface
Date: Sun, 04 Jul 2010 09:04:11 +0000 [thread overview]
Message-ID: <79514591-DCC1-4D9E-AFB7-AA985ADF3C0F@suse.de> (raw)
In-Reply-To: <1278196909.4200.389.camel@pasglop>
On 04.07.2010, at 00:41, Benjamin Herrenschmidt wrote:
> On Fri, 2010-07-02 at 18:27 +0200, Segher Boessenkool wrote:
>>> +To find out if we're running on KVM or not, we overlay the PVR
>>> register. Usually
>>> +the PVR register contains an id that identifies your CPU type. If,
>>> however, you
>>> +pass KVM_PVR_PARA in the register that you want the PVR result in,
>>> the register
>>> +still contains KVM_PVR_PARA after the mfpvr call.
>>> +
>>> + LOAD_REG_IMM(r5, KVM_PVR_PARA)
>>> + mfpvr r5
>>> + [r5 still contains KVM_PVR_PARA]
>>
>> I love this part :-)
>
> Me not :-)
>
> It should be in the device-tree instead, or something like that. Enough
> games with PVR...
My biggest concern about putting things in the device-tree is that I was trying to keep things as separate as possible. Why does the firmware have to know that it's running in KVM? Why do I have to patch 3 projects (Linux, OpenBIOS, Qemu) when I could go with patching a single one (Linux)?
Alex
next prev parent reply other threads:[~2010-07-04 9:04 UTC|newest]
Thread overview: 171+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-01 10:42 [PATCH 00/27] KVM PPC PV framework Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 02/27] KVM: PPC: Convert MSR to shared page Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 03/27] KVM: PPC: Convert DSISR " Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 04/27] KVM: PPC: Convert DAR " Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 05/27] KVM: PPC: Convert SRR0 and SRR1 " Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 08/27] KVM: PPC: Add PV guest critical sections Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 09/27] KVM: PPC: Add PV guest scratch registers Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 11/27] KVM: PPC: Make RMO a define Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
[not found] ` <1277980982-12433-12-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-02 16:23 ` Segher Boessenkool
2010-07-02 16:23 ` Segher Boessenkool
2010-07-02 16:23 ` Segher Boessenkool
2010-07-01 10:42 ` [PATCH 14/27] KVM: PPC: Magic Page BookE support Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
[not found] ` <1277980982-12433-15-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-01 11:18 ` Josh Boyer
2010-07-01 11:18 ` Josh Boyer
2010-07-01 11:18 ` Josh Boyer
[not found] ` <20100701111823.GE7756-jnIq6iUNB1CO8iYQgJmm/0EOCMrvLtNR@public.gmane.org>
2010-07-01 12:25 ` Alexander Graf
2010-07-01 12:25 ` Alexander Graf
2010-07-01 12:25 ` Alexander Graf
2010-07-12 11:24 ` Liu Yu-B13201
2010-07-12 11:24 ` Liu Yu-B13201
2010-07-12 11:24 ` Liu Yu-B13201
2010-07-01 10:42 ` [PATCH 15/27] KVM: PPC: Expose magic page support to guest Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 17/27] KVM: PPC: Generic KVM PV guest support Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 18/27] KVM: PPC: KVM PV guest stubs Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 19/27] KVM: PPC: PV instructions to loads and stores Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 21/27] KVM: PPC: Introduce kvm_tmp framework Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
[not found] ` <1277980982-12433-1-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-01 10:42 ` [PATCH 01/27] KVM: PPC: Introduce shared page Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 06/27] KVM: PPC: Convert SPRG[0-4] to " Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 07/27] KVM: PPC: Implement hypervisor interface Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 10/27] KVM: PPC: Tell guest about pending interrupts Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 12/27] KVM: PPC: First magic page steps Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 13/27] KVM: PPC: Magic Page Book3s support Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
[not found] ` <1277980982-12433-14-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-02 15:37 ` Alexander Graf
2010-07-02 15:37 ` Alexander Graf
2010-07-02 15:37 ` Alexander Graf
[not found] ` <4C2E07AC.4010801-l3A5Bk7waGM@public.gmane.org>
2010-07-04 9:42 ` Avi Kivity
2010-07-04 9:42 ` Avi Kivity
2010-07-04 9:42 ` Avi Kivity
2010-07-01 10:42 ` [PATCH 16/27] KVM: Move kvm_guest_init out of generic code Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
[not found] ` <1277980982-12433-17-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-02 7:41 ` Geert Uytterhoeven
2010-07-02 7:41 ` Geert Uytterhoeven
2010-07-02 7:41 ` Geert Uytterhoeven
2010-07-02 7:44 ` Alexander Graf
2010-07-02 7:44 ` Alexander Graf
2010-07-02 7:44 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 20/27] KVM: PPC: PV tlbsync to nop Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 22/27] KVM: PPC: Introduce branch patching helper Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:43 ` [PATCH 25/27] KVM: PPC: PV mtmsrd L=0 and mtmsr Alexander Graf
2010-07-01 10:43 ` Alexander Graf
2010-07-01 10:43 ` Alexander Graf
2010-07-01 10:43 ` [PATCH 26/27] KVM: PPC: PV wrteei Alexander Graf
2010-07-01 10:43 ` Alexander Graf
2010-07-01 10:43 ` Alexander Graf
2010-07-09 4:57 ` [PATCH 00/27] KVM PPC PV framework MJ embd
2010-07-09 4:57 ` MJ embd
2010-07-09 4:57 ` MJ embd
2010-07-09 6:33 ` Alexander Graf
2010-07-09 6:33 ` Alexander Graf
2010-07-09 6:33 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 23/27] KVM: PPC: PV assembler helpers Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` [PATCH 24/27] KVM: PPC: PV mtmsrd L=1 Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:42 ` Alexander Graf
2010-07-01 10:43 ` [PATCH 27/27] KVM: PPC: Add Documentation about PV interface Alexander Graf
2010-07-01 10:43 ` Alexander Graf
2010-07-01 10:43 ` Alexander Graf
[not found] ` <1277980982-12433-28-git-send-email-agraf-l3A5Bk7waGM@public.gmane.org>
2010-07-02 16:27 ` Segher Boessenkool
2010-07-02 16:27 ` Segher Boessenkool
2010-07-02 16:27 ` Segher Boessenkool
2010-07-02 18:41 ` Alexander Graf
2010-07-02 18:41 ` Alexander Graf
2010-07-02 18:41 ` Alexander Graf
[not found] ` <A98DA9FC-DC42-452E-BD44-5F86D1707BF0-l3A5Bk7waGM@public.gmane.org>
2010-07-03 22:42 ` Benjamin Herrenschmidt
2010-07-03 22:42 ` Benjamin Herrenschmidt
2010-07-03 22:42 ` Benjamin Herrenschmidt
2010-07-04 9:04 ` Alexander Graf
2010-07-04 9:04 ` Alexander Graf
2010-07-04 9:04 ` Alexander Graf
[not found] ` <EB2E1C5B-118B-481B-83D6-44CFAA2E55D3-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2010-07-03 22:41 ` Benjamin Herrenschmidt
2010-07-03 22:41 ` Benjamin Herrenschmidt
2010-07-03 22:41 ` Benjamin Herrenschmidt
2010-07-04 9:04 ` Alexander Graf [this message]
2010-07-04 9:04 ` Alexander Graf
2010-07-04 9:04 ` Alexander Graf
2010-07-04 9:10 ` Avi Kivity
2010-07-04 9:10 ` Avi Kivity
2010-07-04 9:10 ` Avi Kivity
[not found] ` <4C305001.7060301-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-07-04 9:17 ` Alexander Graf
2010-07-04 9:17 ` Alexander Graf
2010-07-04 9:17 ` Alexander Graf
[not found] ` <40488555-C73E-4C04-BFAD-31ED99CDBBDA-l3A5Bk7waGM@public.gmane.org>
2010-07-04 9:30 ` Alexander Graf
2010-07-04 9:30 ` Alexander Graf
2010-07-04 9:30 ` Alexander Graf
[not found] ` <486564E7-7942-4021-8EB2-67DC4E56580D-l3A5Bk7waGM@public.gmane.org>
2010-07-04 9:41 ` Avi Kivity
2010-07-04 9:41 ` Avi Kivity
2010-07-04 9:41 ` Avi Kivity
2010-07-04 9:37 ` Avi Kivity
2010-07-04 9:37 ` Avi Kivity
2010-07-04 9:37 ` Avi Kivity
2010-07-02 17:59 ` Hollis Blanchard
2010-07-02 17:59 ` Hollis Blanchard
2010-07-02 17:59 ` Hollis Blanchard
2010-07-02 18:47 ` Alexander Graf
2010-07-02 18:47 ` Alexander Graf
2010-07-02 18:47 ` Alexander Graf
[not found] ` <3085B58A-01A1-4B5C-A0E7-024DCFDFD4B2-l3A5Bk7waGM@public.gmane.org>
2010-07-02 19:10 ` Scott Wood
2010-07-02 19:10 ` Scott Wood
2010-07-02 19:10 ` Scott Wood
[not found] ` <20100702141003.75302a9a-1MYqz8GpK7RekFaExTCHk1jVikpgYyvb5NbjCUgZEJk@public.gmane.org>
2010-07-04 9:02 ` Alexander Graf
2010-07-04 9:02 ` Alexander Graf
2010-07-04 9:02 ` Alexander Graf
2010-07-09 9:11 ` MJ embd
2010-07-09 9:23 ` MJ embd
2010-07-09 9:11 ` MJ embd
[not found] ` <AANLkTil6RekYBNnpM2MjSH7rcrHQZdHRwK5cBrvKQKhM-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-07-09 9:15 ` Alexander Graf
2010-07-09 9:15 ` Alexander Graf
2010-07-09 9:15 ` Alexander Graf
2010-07-02 16:22 ` [PATCH 00/27] KVM PPC PV framework Segher Boessenkool
2010-07-02 16:22 ` Segher Boessenkool
2010-07-02 16:22 ` Segher Boessenkool
[not found] ` <82C19122-91B6-4F91-9EF0-BEA2759A349D-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2010-07-02 16:59 ` Alexander Graf
2010-07-02 16:59 ` Alexander Graf
2010-07-02 16:59 ` Alexander Graf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=79514591-DCC1-4D9E-AFB7-AA985ADF3C0F@suse.de \
--to=agraf-l3a5bk7wagm@public.gmane.org \
--cc=benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
--cc=kvm-ppc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=segher-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.