Re: [PATCH 2/2] kvm tools: arm: add support for ARM Cortex-A7

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

 



On Fri, Sep 27, 2013 at 06:38:52PM +0100, Jonathan Austin wrote:
> The Cortex-A7 is very similar to the Cortex-A15 and as such there is very
> little extra infrastructure required for KVM tool to be able to create
> A7-guests.
> 
> This patch adds the basic support and allows booting of A7 guests on A7
> hosts. It depends on Cortex-A7 support patches posted recently to the kvmarm
> list.
> 
> Signed-off-by: Jonathan Austin <jonathan.austin@xxxxxxx>
> ---
>  tools/kvm/arm/aarch32/arm-cpu.c |    9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/kvm/arm/aarch32/arm-cpu.c b/tools/kvm/arm/aarch32/arm-cpu.c
> index 8817d2a..71b98fe 100644
> --- a/tools/kvm/arm/aarch32/arm-cpu.c
> +++ b/tools/kvm/arm/aarch32/arm-cpu.c
> @@ -28,8 +28,15 @@ static struct kvm_arm_target target_cortex_a15 = {
>  	.init		= arm_cpu__vcpu_init,
>  };
>  
> +static struct kvm_arm_target target_cortex_a7 = {
> +	.id		= KVM_ARM_TARGET_CORTEX_A7,
> +	.compatible	= "arm,cortex-a7",
> +	.init		= arm_cpu__vcpu_init,
> +};
> +
>  static int arm_cpu__core_init(struct kvm *kvm)
>  {
> -	return kvm_cpu__register_kvm_arm_target(&target_cortex_a15);
> +	return (kvm_cpu__register_kvm_arm_target(&target_cortex_a15) ||
> +		kvm_cpu__register_kvm_arm_target(&target_cortex_a7));

  Acked-by: Will Deacon <will.deacon@xxxxxxx>

However, we should hold off applying this to kvmtool until
KVM_ARM_TARGET_CORTEX_A7 is defined in the kernel sources which it is
shipped with. Do you have an ETA on this change going into the mainline
kernel?

(in the meantime, I'm happy to stage stuff in my kvmtool/arm tree).

Will
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm




[Index of Archives]     [Linux KVM]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux