From patchwork Thu Apr 11 16:36:23 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Will Deacon X-Patchwork-Id: 2429571 Return-Path: X-Original-To: patchwork-kvm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 198233FD40 for ; Thu, 11 Apr 2013 16:36:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762507Ab3DKQgr (ORCPT ); Thu, 11 Apr 2013 12:36:47 -0400 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:64793 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754699Ab3DKQgq (ORCPT ); Thu, 11 Apr 2013 12:36:46 -0400 Received: from mudshark.cambridge.arm.com (mudshark.cambridge.arm.com [10.1.79.58]) by cam-admin0.cambridge.arm.com (8.12.6/8.12.6) with ESMTP id r3BGadki007863; Thu, 11 Apr 2013 17:36:39 +0100 (BST) Received: by mudshark.cambridge.arm.com (Postfix, from userid 1000) id 66A5EC2A9D; Thu, 11 Apr 2013 17:36:28 +0100 (BST) From: Will Deacon To: kvm@vger.kernel.org Cc: penberg@kernel.org, sasha.levin@oracle.com, marc.zyngier@arm.com, Marc Zyngier , Will Deacon Subject: [PATCH 2/5] kvm tools: arm: add CPU compatible string to target structure Date: Thu, 11 Apr 2013 17:36:23 +0100 Message-Id: <1365698186-27355-3-git-send-email-will.deacon@arm.com> X-Mailer: git-send-email 1.8.0 In-Reply-To: <1365698186-27355-1-git-send-email-will.deacon@arm.com> References: <1365698186-27355-1-git-send-email-will.deacon@arm.com> Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org From: Marc Zyngier Instead of hardcoding the CPU compatible string, store it in the target descriptor. This allows similar CPUs to be managed by the same backend, and yet have different compatible properties. Also remove a check for a condition that can never occur in both A15 and A57 backends. Signed-off-by: Marc Zyngier Signed-off-by: Will Deacon --- tools/kvm/arm/aarch32/cortex-a15.c | 12 ++++-------- tools/kvm/arm/aarch64/cortex-a57.c | 12 ++++-------- tools/kvm/arm/include/arm-common/kvm-cpu-arch.h | 6 ++++-- tools/kvm/arm/kvm-cpu.c | 9 ++++++--- 4 files changed, 18 insertions(+), 21 deletions(-) diff --git a/tools/kvm/arm/aarch32/cortex-a15.c b/tools/kvm/arm/aarch32/cortex-a15.c index 8031747..4030e53 100644 --- a/tools/kvm/arm/aarch32/cortex-a15.c +++ b/tools/kvm/arm/aarch32/cortex-a15.c @@ -20,16 +20,11 @@ static void generate_cpu_nodes(void *fdt, struct kvm *kvm) for (cpu = 0; cpu < kvm->nrcpus; ++cpu) { char cpu_name[CPU_NAME_MAX_LEN]; - if (kvm->cpus[cpu]->cpu_type != KVM_ARM_TARGET_CORTEX_A15) { - pr_warning("Ignoring unknown type for CPU %d\n", cpu); - continue; - } - snprintf(cpu_name, CPU_NAME_MAX_LEN, "cpu@%d", cpu); _FDT(fdt_begin_node(fdt, cpu_name)); _FDT(fdt_property_string(fdt, "device_type", "cpu")); - _FDT(fdt_property_string(fdt, "compatible", "arm,cortex-a15")); + _FDT(fdt_property_string(fdt, "compatible", kvm->cpus[cpu]->cpu_compatible)); if (kvm->nrcpus > 1) _FDT(fdt_property_string(fdt, "enable-method", "psci")); @@ -83,8 +78,9 @@ static int cortex_a15__vcpu_init(struct kvm_cpu *vcpu) } static struct kvm_arm_target target_cortex_a15 = { - .id = KVM_ARM_TARGET_CORTEX_A15, - .init = cortex_a15__vcpu_init, + .id = KVM_ARM_TARGET_CORTEX_A15, + .compatible = "arm,cortex-a15", + .init = cortex_a15__vcpu_init, }; static int cortex_a15__core_init(struct kvm *kvm) diff --git a/tools/kvm/arm/aarch64/cortex-a57.c b/tools/kvm/arm/aarch64/cortex-a57.c index 4fd11ba..f636ef7 100644 --- a/tools/kvm/arm/aarch64/cortex-a57.c +++ b/tools/kvm/arm/aarch64/cortex-a57.c @@ -20,16 +20,11 @@ static void generate_cpu_nodes(void *fdt, struct kvm *kvm) for (cpu = 0; cpu < kvm->nrcpus; ++cpu) { char cpu_name[CPU_NAME_MAX_LEN]; - if (kvm->cpus[cpu]->cpu_type != KVM_ARM_TARGET_CORTEX_A57) { - pr_warning("Ignoring unknown type for CPU %d\n", cpu); - continue; - } - snprintf(cpu_name, CPU_NAME_MAX_LEN, "cpu@%d", cpu); _FDT(fdt_begin_node(fdt, cpu_name)); _FDT(fdt_property_string(fdt, "device_type", "cpu")); - _FDT(fdt_property_string(fdt, "compatible", "arm,cortex-a57")); + _FDT(fdt_property_string(fdt, "compatible", kvm->cpus[cpu]->cpu_compatible)); if (kvm->nrcpus > 1) _FDT(fdt_property_string(fdt, "enable-method", "psci")); @@ -84,8 +79,9 @@ static int cortex_a57__vcpu_init(struct kvm_cpu *vcpu) } static struct kvm_arm_target target_cortex_a57 = { - .id = KVM_ARM_TARGET_CORTEX_A57, - .init = cortex_a57__vcpu_init, + .id = KVM_ARM_TARGET_CORTEX_A57, + .compatible = "arm,cortex-a57", + .init = cortex_a57__vcpu_init, }; static int cortex_a57__core_init(struct kvm *kvm) diff --git a/tools/kvm/arm/include/arm-common/kvm-cpu-arch.h b/tools/kvm/arm/include/arm-common/kvm-cpu-arch.h index 351fbe6..b514dd5 100644 --- a/tools/kvm/arm/include/arm-common/kvm-cpu-arch.h +++ b/tools/kvm/arm/include/arm-common/kvm-cpu-arch.h @@ -12,6 +12,7 @@ struct kvm_cpu { unsigned long cpu_id; unsigned long cpu_type; + const char *cpu_compatible; struct kvm *kvm; int vcpu_fd; @@ -28,8 +29,9 @@ struct kvm_cpu { }; struct kvm_arm_target { - u32 id; - int (*init)(struct kvm_cpu *vcpu); + u32 id; + const char *compatible; + int (*init)(struct kvm_cpu *vcpu); }; int kvm_cpu__register_kvm_arm_target(struct kvm_arm_target *target); diff --git a/tools/kvm/arm/kvm-cpu.c b/tools/kvm/arm/kvm-cpu.c index 2716690..6d4f306 100644 --- a/tools/kvm/arm/kvm-cpu.c +++ b/tools/kvm/arm/kvm-cpu.c @@ -30,6 +30,7 @@ int kvm_cpu__register_kvm_arm_target(struct kvm_arm_target *target) struct kvm_cpu *kvm_cpu__arch_init(struct kvm *kvm, unsigned long cpu_id) { + struct kvm_arm_target *target; struct kvm_cpu *vcpu; int coalesced_offset, mmap_size, err = -1; unsigned int i; @@ -58,13 +59,14 @@ struct kvm_cpu *kvm_cpu__arch_init(struct kvm *kvm, unsigned long cpu_id) for (i = 0; i < ARRAY_SIZE(kvm_arm_targets); ++i) { if (!kvm_arm_targets[i]) continue; - vcpu_init.target = kvm_arm_targets[i]->id; + target = kvm_arm_targets[i]; + vcpu_init.target = target->id; err = ioctl(vcpu->vcpu_fd, KVM_ARM_VCPU_INIT, &vcpu_init); if (!err) break; } - if (err || kvm_arm_targets[i]->init(vcpu)) + if (err || target->init(vcpu)) die("Unable to initialise ARM vcpu"); coalesced_offset = ioctl(kvm->sys_fd, KVM_CHECK_EXTENSION, @@ -76,7 +78,8 @@ struct kvm_cpu *kvm_cpu__arch_init(struct kvm *kvm, unsigned long cpu_id) /* Populate the vcpu structure. */ vcpu->kvm = kvm; vcpu->cpu_id = cpu_id; - vcpu->cpu_type = vcpu_init.target; + vcpu->cpu_type = target->id; + vcpu->cpu_compatible = target->compatible; vcpu->is_running = true; return vcpu; }