Re: [PATCH 12/12] KVM: PPC: booke: Get/set guest EPCR register using ONE_REG interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11.10.2012, at 18:13, Mihai Caraman wrote:

> Implement ONE_REG interface for EPCR register adding KVM_REG_PPC_EPCR to
> the list of ONE_REG PPC supported registers.
> 
> Signed-off-by: Mihai Caraman <mihai.caraman@xxxxxxxxxxxxx>
> ---
> Documentation/virtual/kvm/api.txt |    1 +
> arch/powerpc/include/asm/kvm.h    |    2 ++
> arch/powerpc/kvm/booke.c          |   16 ++++++++++++++++
> 3 files changed, 19 insertions(+), 0 deletions(-)
> 
> diff --git a/Documentation/virtual/kvm/api.txt b/Documentation/virtual/kvm/api.txt
> index e726d76..c78dff4 100644
> --- a/Documentation/virtual/kvm/api.txt
> +++ b/Documentation/virtual/kvm/api.txt
> @@ -1773,6 +1773,7 @@ registers, find a list below:
>   PPC   | KVM_REG_PPC_VPA_ADDR  | 64
>   PPC   | KVM_REG_PPC_VPA_SLB   | 128
>   PPC   | KVM_REG_PPC_VPA_DTL   | 128
> +  PPC   | KVM_REG_PPC_EPCR	| 32
> 
> 4.69 KVM_GET_ONE_REG
> 
> diff --git a/arch/powerpc/include/asm/kvm.h b/arch/powerpc/include/asm/kvm.h
> index b89ae4d..beb6b20 100644
> --- a/arch/powerpc/include/asm/kvm.h
> +++ b/arch/powerpc/include/asm/kvm.h
> @@ -386,4 +386,6 @@ struct kvm_book3e_206_tlb_params {
> #define KVM_REG_PPC_VPA_SLB	(KVM_REG_PPC | KVM_REG_SIZE_U128 | 0x83)
> #define KVM_REG_PPC_VPA_DTL	(KVM_REG_PPC | KVM_REG_SIZE_U128 | 0x84)
> 
> +#define KVM_REG_PPC_EPCR	(KVM_REG_PPC | KVM_REG_SIZE_U32 | 0x85)
> +
> #endif /* __LINUX_KVM_POWERPC_H */
> diff --git a/arch/powerpc/kvm/booke.c b/arch/powerpc/kvm/booke.c
> index d396374..60b8237 100644
> --- a/arch/powerpc/kvm/booke.c
> +++ b/arch/powerpc/kvm/booke.c
> @@ -1395,6 +1395,12 @@ int kvm_vcpu_ioctl_get_one_reg(struct kvm_vcpu *vcpu, struct kvm_one_reg *reg)
> 				 &vcpu->arch.dbg_reg.dac[dac], sizeof(u64));
> 		break;
> 	}
> +#if defined(CONFIG_64BIT) && defined(CONFIG_KVM_BOOKE_HV)

No need for the HV dependency here.

> +	case KVM_REG_PPC_EPCR:
> +		r = copy_to_user((u32 __user *)(long)reg->addr,
> +				 &vcpu->arch.epcr, sizeof(u32));

This can be put_user. The reason we don't do it for the other ONE_REG variables is that they are u64s which can't be put_user'd on 32bit systems.

> +		break;
> +#endif
> 	default:
> 		break;
> 	}
> @@ -1422,6 +1428,16 @@ int kvm_vcpu_ioctl_set_one_reg(struct kvm_vcpu *vcpu, struct kvm_one_reg *reg)
> 			     (u64 __user *)(long)reg->addr, sizeof(u64));
> 		break;
> 	}
> +#if defined(CONFIG_64BIT) && defined(CONFIG_KVM_BOOKE_HV)

Same as above

> +	case KVM_REG_PPC_EPCR: {
> +		u32 new_epcr;
> +		r = copy_from_user(&new_epcr,
> +			     (u32 __user *)(long)reg->addr, sizeof(u32));

Same as above.

I'll fix those up for you when applying the patch.


Alex

> +		if (r == 0)
> +			kvmppc_set_epcr(vcpu, new_epcr);
> +		break;
> +	}
> +#endif
> 	default:
> 		break;
> 	}
> -- 
> 1.7.4.1
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux