[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 10/12] target/arm: Add the Cortex-A72
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH v1 10/12] target/arm: Add the Cortex-A72 |
Date: |
Mon, 8 Oct 2018 23:34:07 +0200 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Mon, Oct 08, 2018 at 02:10:29PM +0100, Peter Maydell wrote:
> On 3 October 2018 at 16:07, Edgar E. Iglesias <address@hidden> wrote:
> > From: "Edgar E. Iglesias" <address@hidden>
> >
> > Add the ARM Cortex-A72.
> >
> > Signed-off-by: Edgar E. Iglesias <address@hidden>
> > ---
> > target/arm/cpu64.c | 59
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 59 insertions(+)
> >
> > diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c
> > index 800bff7..02e500b 100644
> > --- a/target/arm/cpu64.c
> > +++ b/target/arm/cpu64.c
> > @@ -218,6 +218,64 @@ static void aarch64_a53_initfn(Object *obj)
> > define_arm_cp_regs(cpu, cortex_a57_a53_cp_reginfo);
> > }
> >
> > +static void aarch64_a72_initfn(Object *obj)
> > +{
> > + ARMCPU *cpu = ARM_CPU(obj);
> > +
> > + cpu->dtb_compatible = "arm,cortex-a72";
> > + set_feature(&cpu->env, ARM_FEATURE_V8);
> > + set_feature(&cpu->env, ARM_FEATURE_VFP4);
> > + set_feature(&cpu->env, ARM_FEATURE_NEON);
> > + set_feature(&cpu->env, ARM_FEATURE_GENERIC_TIMER);
> > + set_feature(&cpu->env, ARM_FEATURE_AARCH64);
> > + set_feature(&cpu->env, ARM_FEATURE_CBAR_RO);
> > + set_feature(&cpu->env, ARM_FEATURE_V8_AES);
> > + set_feature(&cpu->env, ARM_FEATURE_V8_SHA1);
> > + set_feature(&cpu->env, ARM_FEATURE_V8_SHA256);
> > + set_feature(&cpu->env, ARM_FEATURE_V8_PMULL);
> > + set_feature(&cpu->env, ARM_FEATURE_CRC);
> > + set_feature(&cpu->env, ARM_FEATURE_EL2);
> > + set_feature(&cpu->env, ARM_FEATURE_EL3);
>
> No ARM_FEATURE_PMU ?
Will add in v2.
>
> > + cpu->midr = 0x410fd083;
> > + cpu->revidr = 0x00000000;
> > + cpu->reset_fpsid = 0x41034080;
> > + cpu->mvfr0 = 0x10110222;
> > + cpu->mvfr1 = 0x12111111;
> > + cpu->mvfr2 = 0x00000043;
> > + cpu->ctr = 0x8444c004;
> > + cpu->reset_sctlr = 0x00c50838;
>
> Do you happen to have the hardware to hand to check what the
> top 4 bits of the reset value of SCTLR_ELx are? I think they
> should be 0x3 -- the Arm ARM says that [29:28] are RES1 (as
> does the A72 TRM, though its top level summary table lists
> 0x00c50838 as the reset value for some of the SCTLR_ELx.)
>
> QEMU may have the wrong value for A53/A57 here too, I suspect.
I don't have access to the A72s at the moment but looking at logs
it seems to be 0x00c50838 for both the A53 and A72.
Looking at "Table 4-118 SCTLR bit assignments" in the A72 TRM,
bits [30:28] seem to have been allocated. Bit 30 depends on
configuration inputs to the core and [29:28] seem to be hard-coded
to zero.
On ZynqMP and Versal, bit 30 (CFGTE) seems to be runtime configurable
prior to releasing the A53s/A72s from reset. I haven't tried it though.
I ran a test on the ZynqMP a53s and got the folloing right after reset,
for EL1, EL2 and EL3:
sctlr c52838 30c50838 c52838
The 0x2000 mask is due to V-hivecs being set.
SCLTR_EL2 seems to get the RES1 bits set to 1...
Perhaps we should just leave it as is for now?
>
> > + cpu->id_pfr0 = 0x00000131;
> > + cpu->id_pfr1 = 0x00011011;
> > + cpu->id_dfr0 = 0x03010066;
> > + cpu->id_afr0 = 0x00000000;
> > + cpu->id_mmfr0 = 0x10201105;
> > + cpu->id_mmfr1 = 0x40000000;
> > + cpu->id_mmfr2 = 0x01260000;
> > + cpu->id_mmfr3 = 0x02102211;
> > + cpu->id_isar0 = 0x02101110;
> > + cpu->id_isar1 = 0x13112111;
> > + cpu->id_isar2 = 0x21232042;
> > + cpu->id_isar3 = 0x01112131;
> > + cpu->id_isar4 = 0x00011142;
> > + cpu->id_isar5 = 0x00011121;
> > + cpu->id_aa64pfr0 = 0x00002222;
> > + cpu->id_aa64dfr0 = 0x10305106;
> > + cpu->pmceid0 = 0x00000000;
> > + cpu->pmceid1 = 0x00000000;
> > + cpu->id_aa64isar0 = 0x00011120;
> > + cpu->id_aa64mmfr0 = 0x00001124;
> > + cpu->dbgdidr = 0x3516d000;
> > + cpu->clidr = 0x0a200023;
> > + cpu->ccsidr[0] = 0x701fe00a; /* 32KB L1 dcache */
> > + cpu->ccsidr[1] = 0x201fe012; /* 48KB L1 icache */
> > + cpu->ccsidr[2] = 0x707fe07a; /* 1MB L2 cache */
> > + cpu->dcz_blocksize = 4; /* 64 bytes */
> > + cpu->gic_num_lrs = 4;
> > + cpu->gic_vpribits = 5;
> > + cpu->gic_vprebits = 5;
> > + define_arm_cp_regs(cpu, cortex_a57_a53_cp_reginfo);
>
> The impdef registers do seem to be basically the same as
> the A53/A57. The function name is now a bit inaccurate,
> though, but I don't have a good idea for a better name.
>
> Otherwise
> Reviewed-by: Peter Maydell <address@hidden>
>
> thanks
> -- PMM
- Re: [Qemu-devel] [PATCH v1 06/12] net: cadence_gem: Add support for selecting the DMA MemoryRegion, (continued)
[Qemu-devel] [PATCH v1 11/12] hw/arm: versal: Add a model of Xilinx Versal SoC, Edgar E. Iglesias, 2018/10/03
[Qemu-devel] [PATCH v1 12/12] hw/arm: versal: Add a virtual Xilinx Versal board, Edgar E. Iglesias, 2018/10/03
Re: [Qemu-devel] [PATCH v1 00/12] arm: Add first models of Xilinx Versal SoC, Peter Maydell, 2018/10/08