[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 09/15] target-arm: Add ARMCPU secure property
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 09/15] target-arm: Add ARMCPU secure property |
Date: |
Mon, 15 Dec 2014 17:01:59 +0000 |
On 11 December 2014 at 23:29, Greg Bellows <address@hidden> wrote:
> Added a "has_el3" state property to the ARMCPU descriptor. This property
> indicates whether the ARMCPU has security extensions enabled (EL3) or not.
> By default it is disabled at this time.
>
> Signed-off-by: Greg Bellows <address@hidden>
>
> ---
>
> v1 -> v2
> - Added set of has_el3 to true when EL3 is enabled
> ---
> target-arm/cpu-qom.h | 2 ++
> target-arm/cpu.c | 24 ++++++++++++++++++++++++
> 2 files changed, 26 insertions(+)
>
> diff --git a/target-arm/cpu-qom.h b/target-arm/cpu-qom.h
> index dcfda7d..ed5a644 100644
> --- a/target-arm/cpu-qom.h
> +++ b/target-arm/cpu-qom.h
> @@ -100,6 +100,8 @@ typedef struct ARMCPU {
> bool start_powered_off;
> /* CPU currently in PSCI powered-off state */
> bool powered_off;
> + /* CPU has security extension */
> + bool has_el3;
>
> /* PSCI conduit used to invoke PSCI methods
> * 0 - disabled, 1 - smc, 2 - hvc
> diff --git a/target-arm/cpu.c b/target-arm/cpu.c
> index 01afed2..758e8f8 100644
> --- a/target-arm/cpu.c
> +++ b/target-arm/cpu.c
> @@ -388,6 +388,9 @@ static Property arm_cpu_reset_hivecs_property =
> static Property arm_cpu_rvbar_property =
> DEFINE_PROP_UINT64("rvbar", ARMCPU, rvbar, 0);
>
> +static Property arm_cpu_has_el3_property =
> + DEFINE_PROP_BOOL("has_el3", ARMCPU, has_el3, false);
I think the default value here should be "true": we want
CPUs to default to having all their features turned on
unless the board specifically disables it.
(This won't affect behaviour til we get to patch 15 because
before then ARM_FEATURE_EL3 isn't set and we never add
the property here.)
> +
> static void arm_cpu_post_init(Object *obj)
> {
> ARMCPU *cpu = ARM_CPU(obj);
> @@ -407,6 +410,15 @@ static void arm_cpu_post_init(Object *obj)
> qdev_property_add_static(DEVICE(obj), &arm_cpu_rvbar_property,
> &error_abort);
> }
> +
> + if (arm_feature(&cpu->env, ARM_FEATURE_EL3)) {
> + /* Add the has_el3 state CPU property only if EL3 is allowed. This
> will
> + * prevent "has_el3" from existing on CPUs which cannot support EL3.
> + */
> + qdev_property_add_static(DEVICE(obj), &arm_cpu_has_el3_property,
> + &error_abort);
> + cpu->has_el3 = true;
> + }
> }
>
> static void arm_cpu_finalizefn(Object *obj)
> @@ -476,6 +488,18 @@ static void arm_cpu_realizefn(DeviceState *dev, Error
> **errp)
> cpu->reset_sctlr |= (1 << 13);
> }
>
> + if (!cpu->has_el3) {
> + /* If the hsa_el3 CPU property is disabled then we need to disable
> the
> + * feature.
Typo: "has_el3".
> + */
> + unset_feature(env, ARM_FEATURE_EL3);
> +
> + /* Disable the security extension feature bits in the processor
> feature
> + * register as well. This is id_pfr1[7:4].
> + */
> + cpu->id_pfr1 &= ~0xf0;
> + }
> +
> register_cp_regs_for_features(cpu);
> arm_cpu_register_gdb_regs_for_features(cpu);
thanks
-- PMM
- Re: [Qemu-devel] [PATCH v2 04/15] target-arm: Add vexpress machine secure property, (continued)
- [Qemu-devel] [PATCH v2 03/15] target-arm: Switch to common vexpress machine init, Greg Bellows, 2014/12/11
- [Qemu-devel] [PATCH v2 06/15] target-arm: Add virt class and machine types, Greg Bellows, 2014/12/11
- [Qemu-devel] [PATCH v2 07/15] target-arm: Add virt machine secure property, Greg Bellows, 2014/12/11
- [Qemu-devel] [PATCH v2 08/15] target-arm: Add feature unset function, Greg Bellows, 2014/12/11
- [Qemu-devel] [PATCH v2 09/15] target-arm: Add ARMCPU secure property, Greg Bellows, 2014/12/11
- Re: [Qemu-devel] [PATCH v2 09/15] target-arm: Add ARMCPU secure property,
Peter Maydell <=
[Qemu-devel] [PATCH v2 11/15] target-arm: Enable CPU has_el3 prop during VE init, Greg Bellows, 2014/12/11
[Qemu-devel] [PATCH v2 10/15] target-arm: Add arm_boot_info secure_boot control, Greg Bellows, 2014/12/11
[Qemu-devel] [PATCH v2 12/15] target-arm: Set CPU has_el3 prop during virt init, Greg Bellows, 2014/12/11