target-arm: Enable vPMU support under TCG mode
This patch contains several fixes to enable vPMU under TCG mode. It first removes the checking of kvm_enabled() while unsetting ARM_FEATURE_PMU. With it, the .pmu option can be used to turn on/off vPMU under TCG mode. Secondly the PMU node of DT table is now created under TCG. The last fix is to disable the masking of PMUver field of ID_AA64DFR0_EL1. Signed-off-by: Wei Huang <wei@redhat.com> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Message-id: 1486504171-26807-5-git-send-email-wei@redhat.com Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
e6ec54571e
commit
d6f02ce3b8
@ -471,7 +471,7 @@ static void fdt_add_pmu_nodes(const VirtMachineState *vms)
|
|||||||
CPU_FOREACH(cpu) {
|
CPU_FOREACH(cpu) {
|
||||||
armcpu = ARM_CPU(cpu);
|
armcpu = ARM_CPU(cpu);
|
||||||
if (!arm_feature(&armcpu->env, ARM_FEATURE_PMU) ||
|
if (!arm_feature(&armcpu->env, ARM_FEATURE_PMU) ||
|
||||||
!kvm_arm_pmu_create(cpu, PPI(VIRTUAL_PMU_IRQ))) {
|
(kvm_enabled() && !kvm_arm_pmu_create(cpu, PPI(VIRTUAL_PMU_IRQ)))) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -781,7 +781,7 @@ static void arm_cpu_realizefn(DeviceState *dev, Error **errp)
|
|||||||
unset_feature(env, ARM_FEATURE_EL2);
|
unset_feature(env, ARM_FEATURE_EL2);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!cpu->has_pmu || !kvm_enabled()) {
|
if (!cpu->has_pmu) {
|
||||||
cpu->has_pmu = false;
|
cpu->has_pmu = false;
|
||||||
unset_feature(env, ARM_FEATURE_PMU);
|
unset_feature(env, ARM_FEATURE_PMU);
|
||||||
}
|
}
|
||||||
|
@ -4633,12 +4633,7 @@ void register_cp_regs_for_features(ARMCPU *cpu)
|
|||||||
{ .name = "ID_AA64DFR0_EL1", .state = ARM_CP_STATE_AA64,
|
{ .name = "ID_AA64DFR0_EL1", .state = ARM_CP_STATE_AA64,
|
||||||
.opc0 = 3, .opc1 = 0, .crn = 0, .crm = 5, .opc2 = 0,
|
.opc0 = 3, .opc1 = 0, .crn = 0, .crm = 5, .opc2 = 0,
|
||||||
.access = PL1_R, .type = ARM_CP_CONST,
|
.access = PL1_R, .type = ARM_CP_CONST,
|
||||||
/* We mask out the PMUVer field, because we don't currently
|
.resetvalue = cpu->id_aa64dfr0 },
|
||||||
* implement the PMU. Not advertising it prevents the guest
|
|
||||||
* from trying to use it and getting UNDEFs on registers we
|
|
||||||
* don't implement.
|
|
||||||
*/
|
|
||||||
.resetvalue = cpu->id_aa64dfr0 & ~0xf00 },
|
|
||||||
{ .name = "ID_AA64DFR1_EL1", .state = ARM_CP_STATE_AA64,
|
{ .name = "ID_AA64DFR1_EL1", .state = ARM_CP_STATE_AA64,
|
||||||
.opc0 = 3, .opc1 = 0, .crn = 0, .crm = 5, .opc2 = 1,
|
.opc0 = 3, .opc1 = 0, .crn = 0, .crm = 5, .opc2 = 1,
|
||||||
.access = PL1_R, .type = ARM_CP_CONST,
|
.access = PL1_R, .type = ARM_CP_CONST,
|
||||||
|
Loading…
Reference in New Issue
Block a user