Re: [PATCH 01/23] Pass PVR in sregs

[Avi Kivity <avi@redhat.com>]

On 07/07/2009 05:17 PM, Alexander Graf wrote:
> Right now sregs is unused on PPC, so we can use it for initialization
> of the CPU.
>
> KVM on BookE always virtualizes the host CPU. On PPC64 we go a step further
> and take the PVR from userspace that tells us what kind of CPU we are supposed
> to virtualize, because we support PPC32 and PPC64 guests.
>
> In order to get that information, we use the sregs ioctl, because we don't
> want to reset the guest CPU on every normal register set.
>
> Signed-off-by: Alexander Graf<agraf@suse.de>
> ---
>   arch/powerpc/include/asm/kvm.h |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/kvm.h b/arch/powerpc/include/asm/kvm.h
> index bb2de6a..96b02cd 100644
> --- a/arch/powerpc/include/asm/kvm.h
> +++ b/arch/powerpc/include/asm/kvm.h
> @@ -46,6 +46,7 @@ struct kvm_regs {
>   };
>
>   struct kvm_sregs {
> +	__u64 pvr;
>   };
>    

You can only do that if existing userspace never calls KVM_SET_SREGS.  
Hollis?

Also, make sure to reserve a bunch of space in there so you if you 
forget something you can add it later.

-- 
error compiling committee.c: too many arguments to function