1
Nothing very exciting this time around...
1
A last small test of bug fixes before rc1.
2
2
3
thanks
3
-- PMM
4
-- PMM
4
5
5
The following changes since commit 37a712a0f969ca2df7f01182409a6c4825cebfb5:
6
The following changes since commit ed8ad9728a9c0eec34db9dff61dfa2f1dd625637:
6
7
7
Merge remote-tracking branch 'remotes/bonzini-gitlab/tags/for-upstream' into staging (2020-10-01 12:23:19 +0100)
8
Merge tag 'pull-tpm-2023-07-14-1' of https://github.com/stefanberger/qemu-tpm into staging (2023-07-15 14:54:04 +0100)
8
9
9
are available in the Git repository at:
10
are available in the Git repository at:
10
11
11
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20201001
12
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20230717
12
13
13
for you to fetch changes up to cdfaa57dcb53ba012439765a1462247dfda8595d:
14
for you to fetch changes up to c2c1c4a35c7c2b1a4140b0942b9797c857e476a4:
14
15
15
hw/arm/raspi: Remove use of the 'version' value in the board code (2020-10-01 15:31:01 +0100)
16
hw/nvram: Avoid unnecessary Xilinx eFuse backstore write (2023-07-17 11:05:52 +0100)
16
17
17
----------------------------------------------------------------
18
----------------------------------------------------------------
18
target-arm queue:
19
target-arm queue:
19
* Make isar_feature_aa32_fp16_arith() handle M-profile
20
* hw/arm/sbsa-ref: set 'slots' property of xhci
20
* Fix SVE splice
21
* linux-user: Remove pointless NULL check in clock_adjtime handling
21
* Fix SVE LDR/STR
22
* ptw: Fix S1_ptw_translate() debug path
22
* Remove ignore_memory_transaction_failures on the raspi2
23
* ptw: Account for FEAT_RME when applying {N}SW, SA bits
23
* raspi: Various cleanup/refactoring
24
* accel/tcg: Zero-pad PC in TCG CPU exec trace lines
25
* hw/nvram: Avoid unnecessary Xilinx eFuse backstore write
24
26
25
----------------------------------------------------------------
27
----------------------------------------------------------------
26
Peter Maydell (5):
28
Peter Maydell (5):
27
target/arm: Replace ARM_FEATURE_PXN with ID_MMFR0.VMSA check
29
linux-user: Remove pointless NULL check in clock_adjtime handling
28
target/arm: Move id_pfr0, id_pfr1 into ARMISARegisters
30
target/arm/ptw.c: Add comments to S1Translate struct fields
29
hw/intc/armv7m_nvic: Only show ID register values for Main Extension CPUs
31
target/arm: Fix S1_ptw_translate() debug path
30
target/arm: Add ID register values for Cortex-M0
32
target/arm/ptw.c: Account for FEAT_RME when applying {N}SW, SA bits
31
target/arm: Make isar_feature_aa32_fp16_arith() handle M-profile
33
accel/tcg: Zero-pad PC in TCG CPU exec trace lines
32
34
33
Philippe Mathieu-Daudé (11):
35
Tong Ho (1):
34
hw/arm/raspi: Define various blocks base addresses
36
hw/nvram: Avoid unnecessary Xilinx eFuse backstore write
35
hw/arm/bcm2835: Add more unimplemented peripherals
36
hw/arm/raspi: Remove ignore_memory_transaction_failures on the raspi2
37
hw/arm/raspi: Display the board revision in the machine description
38
hw/arm/raspi: Load the firmware on the first core
39
hw/arm/raspi: Move arm_boot_info structure to RaspiMachineState
40
hw/arm/raspi: Avoid using TypeInfo::class_data pointer
41
hw/arm/raspi: Use more specific machine names
42
hw/arm/raspi: Introduce RaspiProcessorId enum
43
hw/arm/raspi: Use RaspiProcessorId to set the firmware load address
44
hw/arm/raspi: Remove use of the 'version' value in the board code
45
37
46
Richard Henderson (2):
38
Yuquan Wang (1):
47
target/arm: Fix sve ldr/str
39
hw/arm/sbsa-ref: set 'slots' property of xhci
48
target/arm: Fix SVE splice
49
40
50
include/hw/arm/bcm2835_peripherals.h | 2 +
41
accel/tcg/cpu-exec.c | 4 +--
51
include/hw/arm/raspi_platform.h | 51 ++++++++++--
42
accel/tcg/translate-all.c | 2 +-
52
target/arm/cpu.h | 50 +++++++++--
43
hw/arm/sbsa-ref.c | 1 +
53
hw/arm/bcm2835_peripherals.c | 2 +
44
hw/nvram/xlnx-efuse.c | 11 ++++--
54
hw/arm/raspi.c | 155 +++++++++++++++++++----------------
45
linux-user/syscall.c | 12 +++----
55
hw/intc/armv7m_nvic.c | 46 ++++++++++-
46
target/arm/ptw.c | 90 +++++++++++++++++++++++++++++++++++++++++------
56
target/arm/cpu.c | 21 +++--
47
6 files changed, 98 insertions(+), 22 deletions(-)
57
target/arm/cpu64.c | 12 +--
58
target/arm/cpu_tcg.c | 60 ++++++++++----
59
target/arm/helper.c | 9 +-
60
target/arm/kvm64.c | 4 +
61
target/arm/translate-sve.c | 6 +-
62
12 files changed, 286 insertions(+), 132 deletions(-)
63
diff view generated by jsdifflib
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
1
From: Yuquan Wang <wangyuquan1236@phytium.com.cn>
2
2
3
We expected the 'version' ID to match the board processor ID,
3
This extends the slots of xhci to 64, since the default xhci_sysbus
4
but this is not always true (for example boards with revision
4
just supports one slot.
5
id 0xa02042/0xa22042 are Raspberry Pi 2 with a BCM2837 SoC).
6
This was not important because we were not modelling them, but
7
since the recent refactor now allow to model these boards, it
8
is safer to check the processor id directly. Remove the version
9
check.
10
5
11
Suggested-by: Peter Maydell <peter.maydell@linaro.org>
6
Signed-off-by: Wang Yuquan <wangyuquan1236@phytium.com.cn>
12
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
7
Signed-off-by: Chen Baozi <chenbaozi@phytium.com.cn>
13
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
8
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
14
Message-id: 20200924111808.77168-9-f4bug@amsat.org
9
Reviewed-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
10
Tested-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
11
Message-id: 20230710063750.473510-2-wangyuquan1236@phytium.com.cn
15
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
16
---
13
---
17
hw/arm/raspi.c | 29 +++++++++++++----------------
14
hw/arm/sbsa-ref.c | 1 +
18
1 file changed, 13 insertions(+), 16 deletions(-)
15
1 file changed, 1 insertion(+)
19
16
20
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
17
diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c
21
index XXXXXXX..XXXXXXX 100644
18
index XXXXXXX..XXXXXXX 100644
22
--- a/hw/arm/raspi.c
19
--- a/hw/arm/sbsa-ref.c
23
+++ b/hw/arm/raspi.c
20
+++ b/hw/arm/sbsa-ref.c
24
@@ -XXX,XX +XXX,XX @@ static RaspiProcessorId board_processor_id(uint32_t board_rev)
21
@@ -XXX,XX +XXX,XX @@ static void create_xhci(const SBSAMachineState *sms)
25
return proc_id;
22
hwaddr base = sbsa_ref_memmap[SBSA_XHCI].base;
26
}
23
int irq = sbsa_ref_irqmap[SBSA_XHCI];
27
24
DeviceState *dev = qdev_new(TYPE_XHCI_SYSBUS);
28
-static int board_version(uint32_t board_rev)
25
+ qdev_prop_set_uint32(dev, "slots", XHCI_MAXSLOTS);
29
-{
26
30
- return board_processor_id(board_rev) + 1;
27
sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
31
-}
28
sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, base);
32
-
33
static const char *board_soc_type(uint32_t board_rev)
34
{
35
return soc_property[board_processor_id(board_rev)].type;
36
@@ -XXX,XX +XXX,XX @@ static void reset_secondary(ARMCPU *cpu, const struct arm_boot_info *info)
37
cpu_set_pc(cs, info->smp_loader_start);
38
}
39
40
-static void setup_boot(MachineState *machine, int version, size_t ram_size)
41
+static void setup_boot(MachineState *machine, RaspiProcessorId processor_id,
42
+ size_t ram_size)
43
{
44
RaspiMachineState *s = RASPI_MACHINE(machine);
45
int r;
46
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
47
s->binfo.ram_size = ram_size;
48
s->binfo.nb_cpus = machine->smp.cpus;
49
50
- if (version <= 2) {
51
- /* The rpi1 and 2 require some custom setup code to run in Secure
52
- * mode before booting a kernel (to set up the SMC vectors so
53
- * that we get a no-op SMC; this is used by Linux to call the
54
+ if (processor_id <= PROCESSOR_ID_BCM2836) {
55
+ /*
56
+ * The BCM2835 and BCM2836 require some custom setup code to run
57
+ * in Secure mode before booting a kernel (to set up the SMC vectors
58
+ * so that we get a no-op SMC; this is used by Linux to call the
59
* firmware for some cache maintenance operations.
60
- * The rpi3 doesn't need this.
61
+ * The BCM2837 doesn't need this.
62
*/
63
s->binfo.board_setup_addr = BOARDSETUP_ADDR;
64
s->binfo.write_board_setup = write_board_setup;
65
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
66
s->binfo.secure_boot = true;
67
}
68
69
- /* Pi2 and Pi3 requires SMP setup */
70
- if (version >= 2) {
71
+ /* BCM2836 and BCM2837 requires SMP setup */
72
+ if (processor_id >= PROCESSOR_ID_BCM2836) {
73
s->binfo.smp_loader_start = SMPBOOT_ADDR;
74
- if (version == 2) {
75
+ if (processor_id == PROCESSOR_ID_BCM2836) {
76
s->binfo.write_secondary_boot = write_smpboot;
77
} else {
78
s->binfo.write_secondary_boot = write_smpboot64;
79
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_init(MachineState *machine)
80
RaspiMachineClass *mc = RASPI_MACHINE_GET_CLASS(machine);
81
RaspiMachineState *s = RASPI_MACHINE(machine);
82
uint32_t board_rev = mc->board_rev;
83
- int version = board_version(board_rev);
84
uint64_t ram_size = board_ram_size(board_rev);
85
uint32_t vcram_size;
86
DriveInfo *di;
87
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_init(MachineState *machine)
88
89
vcram_size = object_property_get_uint(OBJECT(&s->soc), "vcram-size",
90
&error_abort);
91
- setup_boot(machine, version, machine->ram_size - vcram_size);
92
+ setup_boot(machine, board_processor_id(mc->board_rev),
93
+ machine->ram_size - vcram_size);
94
}
95
96
static void raspi_machine_class_common_init(MachineClass *mc,
97
--
29
--
98
2.20.1
30
2.34.1
99
100
diff view generated by jsdifflib
1
M-profile CPUs only implement the ID registers as guest-visible if
1
In the code for TARGET_NR_clock_adjtime, we set the pointer phtx to
2
the CPU implements the Main Extension (all our current CPUs except
2
the address of the local variable htx. This means it can never be
3
the Cortex-M0 do).
3
NULL, but later in the code we check it for NULL anyway. Coverity
4
complains about this (CID 1507683) because the NULL check comes after
5
a call to clock_adjtime() that assumes it is non-NULL.
4
6
5
Currently we handle this by having the Cortex-M0 leave the ID
7
Since phtx is always &htx, and is used only in three places, it's not
6
register values in the ARMCPU struct as zero, but this conflicts with
8
really necessary. Remove it, bringing the code structure in to line
7
our design decision to make QEMU behaviour be keyed off ID register
9
with that for TARGET_NR_clock_adjtime64, which already uses a simple
8
fields wherever possible.
10
'&htx' when it wants a pointer to 'htx'.
9
10
Explicitly code the ID registers in the NVIC to return 0 if the Main
11
Extension is not implemented, so we can make the M0 model set the
12
ARMCPU struct fields to obtain the correct behaviour without those
13
values becoming guest-visible.
14
11
15
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
13
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
16
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
14
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
17
Message-id: 20200910173855.4068-4-peter.maydell@linaro.org
15
Message-id: 20230623144410.1837261-1-peter.maydell@linaro.org
18
---
16
---
19
hw/intc/armv7m_nvic.c | 42 ++++++++++++++++++++++++++++++++++++++++++
17
linux-user/syscall.c | 12 +++++-------
20
1 file changed, 42 insertions(+)
18
1 file changed, 5 insertions(+), 7 deletions(-)
21
19
22
diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c
20
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
23
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
24
--- a/hw/intc/armv7m_nvic.c
22
--- a/linux-user/syscall.c
25
+++ b/hw/intc/armv7m_nvic.c
23
+++ b/linux-user/syscall.c
26
@@ -XXX,XX +XXX,XX @@ static uint32_t nvic_readl(NVICState *s, uint32_t offset, MemTxAttrs attrs)
24
@@ -XXX,XX +XXX,XX @@ static abi_long do_syscall1(CPUArchState *cpu_env, int num, abi_long arg1,
27
"Aux Fault status registers unimplemented\n");
25
#if defined(TARGET_NR_clock_adjtime) && defined(CONFIG_CLOCK_ADJTIME)
28
return 0;
26
case TARGET_NR_clock_adjtime:
29
case 0xd40: /* PFR0. */
27
{
30
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
28
- struct timex htx, *phtx = &htx;
31
+ goto bad_offset;
29
+ struct timex htx;
32
+ }
30
33
return cpu->isar.id_pfr0;
31
- if (target_to_host_timex(phtx, arg2) != 0) {
34
case 0xd44: /* PFR1. */
32
+ if (target_to_host_timex(&htx, arg2) != 0) {
35
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
33
return -TARGET_EFAULT;
36
+ goto bad_offset;
34
}
37
+ }
35
- ret = get_errno(clock_adjtime(arg1, phtx));
38
return cpu->isar.id_pfr1;
36
- if (!is_error(ret) && phtx) {
39
case 0xd48: /* DFR0. */
37
- if (host_to_target_timex(arg2, phtx) != 0) {
40
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
38
- return -TARGET_EFAULT;
41
+ goto bad_offset;
39
- }
42
+ }
40
+ ret = get_errno(clock_adjtime(arg1, &htx));
43
return cpu->isar.id_dfr0;
41
+ if (!is_error(ret) && host_to_target_timex(arg2, &htx)) {
44
case 0xd4c: /* AFR0. */
42
+ return -TARGET_EFAULT;
45
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
43
}
46
+ goto bad_offset;
44
}
47
+ }
45
return ret;
48
return cpu->id_afr0;
49
case 0xd50: /* MMFR0. */
50
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
51
+ goto bad_offset;
52
+ }
53
return cpu->isar.id_mmfr0;
54
case 0xd54: /* MMFR1. */
55
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
56
+ goto bad_offset;
57
+ }
58
return cpu->isar.id_mmfr1;
59
case 0xd58: /* MMFR2. */
60
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
61
+ goto bad_offset;
62
+ }
63
return cpu->isar.id_mmfr2;
64
case 0xd5c: /* MMFR3. */
65
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
66
+ goto bad_offset;
67
+ }
68
return cpu->isar.id_mmfr3;
69
case 0xd60: /* ISAR0. */
70
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
71
+ goto bad_offset;
72
+ }
73
return cpu->isar.id_isar0;
74
case 0xd64: /* ISAR1. */
75
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
76
+ goto bad_offset;
77
+ }
78
return cpu->isar.id_isar1;
79
case 0xd68: /* ISAR2. */
80
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
81
+ goto bad_offset;
82
+ }
83
return cpu->isar.id_isar2;
84
case 0xd6c: /* ISAR3. */
85
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
86
+ goto bad_offset;
87
+ }
88
return cpu->isar.id_isar3;
89
case 0xd70: /* ISAR4. */
90
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
91
+ goto bad_offset;
92
+ }
93
return cpu->isar.id_isar4;
94
case 0xd74: /* ISAR5. */
95
+ if (!arm_feature(&cpu->env, ARM_FEATURE_M_MAIN)) {
96
+ goto bad_offset;
97
+ }
98
return cpu->isar.id_isar5;
99
case 0xd78: /* CLIDR */
100
return cpu->clidr;
101
--
46
--
102
2.20.1
47
2.34.1
103
48
104
49
diff view generated by jsdifflib
1
Move the id_pfr0 and id_pfr1 fields into the ARMISARegisters
1
Add comments to the in_* fields in the S1Translate struct
2
sub-struct. We're going to want id_pfr1 for an isar_features
2
that explain what they're doing.
3
check, and moving both at the same time avoids an odd
4
inconsistency.
5
6
Changes other than the ones to cpu.h and kvm64.c made
7
automatically with:
8
perl -p -i -e 's/cpu->id_pfr/cpu->isar.id_pfr/' target/arm/*.c hw/intc/armv7m_nvic.c
9
3
10
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
4
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
11
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
5
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
12
Message-id: 20200910173855.4068-3-peter.maydell@linaro.org
6
Message-id: 20230710152130.3928330-2-peter.maydell@linaro.org
13
---
7
---
14
target/arm/cpu.h | 4 ++--
8
target/arm/ptw.c | 40 ++++++++++++++++++++++++++++++++++++++++
15
hw/intc/armv7m_nvic.c | 4 ++--
9
1 file changed, 40 insertions(+)
16
target/arm/cpu.c | 20 ++++++++++----------
17
target/arm/cpu64.c | 12 ++++++------
18
target/arm/cpu_tcg.c | 36 ++++++++++++++++++------------------
19
target/arm/helper.c | 4 ++--
20
target/arm/kvm64.c | 4 ++++
21
7 files changed, 44 insertions(+), 40 deletions(-)
22
10
23
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
11
diff --git a/target/arm/ptw.c b/target/arm/ptw.c
24
index XXXXXXX..XXXXXXX 100644
12
index XXXXXXX..XXXXXXX 100644
25
--- a/target/arm/cpu.h
13
--- a/target/arm/ptw.c
26
+++ b/target/arm/cpu.h
14
+++ b/target/arm/ptw.c
27
@@ -XXX,XX +XXX,XX @@ struct ARMCPU {
15
@@ -XXX,XX +XXX,XX @@
28
uint32_t id_mmfr2;
16
#endif
29
uint32_t id_mmfr3;
17
30
uint32_t id_mmfr4;
18
typedef struct S1Translate {
31
+ uint32_t id_pfr0;
19
+ /*
32
+ uint32_t id_pfr1;
20
+ * in_mmu_idx : specifies which TTBR, TCR, etc to use for the walk.
33
uint32_t mvfr0;
21
+ * Together with in_space, specifies the architectural translation regime.
34
uint32_t mvfr1;
22
+ */
35
uint32_t mvfr2;
23
ARMMMUIdx in_mmu_idx;
36
@@ -XXX,XX +XXX,XX @@ struct ARMCPU {
24
+ /*
37
uint32_t reset_fpsid;
25
+ * in_ptw_idx: specifies which mmuidx to use for the actual
38
uint32_t ctr;
26
+ * page table descriptor load operations. This will be one of the
39
uint32_t reset_sctlr;
27
+ * ARMMMUIdx_Stage2* or one of the ARMMMUIdx_Phys_* indexes.
40
- uint32_t id_pfr0;
28
+ * If a Secure ptw is "downgraded" to NonSecure by an NSTable bit,
41
- uint32_t id_pfr1;
29
+ * this field is updated accordingly.
42
uint64_t pmceid0;
30
+ */
43
uint64_t pmceid1;
31
ARMMMUIdx in_ptw_idx;
44
uint32_t id_afr0;
32
+ /*
45
diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c
33
+ * in_space: the security space for this walk. This plus
46
index XXXXXXX..XXXXXXX 100644
34
+ * the in_mmu_idx specify the architectural translation regime.
47
--- a/hw/intc/armv7m_nvic.c
35
+ * If a Secure ptw is "downgraded" to NonSecure by an NSTable bit,
48
+++ b/hw/intc/armv7m_nvic.c
36
+ * this field is updated accordingly.
49
@@ -XXX,XX +XXX,XX @@ static uint32_t nvic_readl(NVICState *s, uint32_t offset, MemTxAttrs attrs)
37
+ *
50
"Aux Fault status registers unimplemented\n");
38
+ * Note that the security space for the in_ptw_idx may be different
51
return 0;
39
+ * from that for the in_mmu_idx. We do not need to explicitly track
52
case 0xd40: /* PFR0. */
40
+ * the in_ptw_idx security space because:
53
- return cpu->id_pfr0;
41
+ * - if the in_ptw_idx is an ARMMMUIdx_Phys_* then the mmuidx
54
+ return cpu->isar.id_pfr0;
42
+ * itself specifies the security space
55
case 0xd44: /* PFR1. */
43
+ * - if the in_ptw_idx is an ARMMMUIdx_Stage2* then the security
56
- return cpu->id_pfr1;
44
+ * space used for ptw reads is the same as that of the security
57
+ return cpu->isar.id_pfr1;
45
+ * space of the stage 1 translation for all cases except where
58
case 0xd48: /* DFR0. */
46
+ * stage 1 is Secure; in that case the only possibilities for
59
return cpu->isar.id_dfr0;
47
+ * the ptw read are Secure and NonSecure, and the in_ptw_idx
60
case 0xd4c: /* AFR0. */
48
+ * value being Stage2 vs Stage2_S distinguishes those.
61
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
49
+ */
62
index XXXXXXX..XXXXXXX 100644
50
ARMSecuritySpace in_space;
63
--- a/target/arm/cpu.c
51
+ /*
64
+++ b/target/arm/cpu.c
52
+ * in_secure: whether the translation regime is a Secure one.
65
@@ -XXX,XX +XXX,XX @@ static void arm_cpu_realizefn(DeviceState *dev, Error **errp)
53
+ * This is always equal to arm_space_is_secure(in_space).
66
/* Disable the security extension feature bits in the processor feature
54
+ * If a Secure ptw is "downgraded" to NonSecure by an NSTable bit,
67
* registers as well. These are id_pfr1[7:4] and id_aa64pfr0[15:12].
55
+ * this field is updated accordingly.
68
*/
56
+ */
69
- cpu->id_pfr1 &= ~0xf0;
57
bool in_secure;
70
+ cpu->isar.id_pfr1 &= ~0xf0;
58
+ /*
71
cpu->isar.id_aa64pfr0 &= ~0xf000;
59
+ * in_debug: is this a QEMU debug access (gdbstub, etc)? Debug
72
}
60
+ * accesses will not update the guest page table access flags
73
61
+ * and will not change the state of the softmmu TLBs.
74
@@ -XXX,XX +XXX,XX @@ static void arm_cpu_realizefn(DeviceState *dev, Error **errp)
62
+ */
75
* id_aa64pfr0_el1[11:8].
63
bool in_debug;
76
*/
64
/*
77
cpu->isar.id_aa64pfr0 &= ~0xf00;
65
* If this is stage 2 of a stage 1+2 page table walk, then this must
78
- cpu->id_pfr1 &= ~0xf000;
79
+ cpu->isar.id_pfr1 &= ~0xf000;
80
}
81
82
#ifndef CONFIG_USER_ONLY
83
@@ -XXX,XX +XXX,XX @@ static void cortex_a8_initfn(Object *obj)
84
cpu->isar.mvfr1 = 0x00011111;
85
cpu->ctr = 0x82048004;
86
cpu->reset_sctlr = 0x00c50078;
87
- cpu->id_pfr0 = 0x1031;
88
- cpu->id_pfr1 = 0x11;
89
+ cpu->isar.id_pfr0 = 0x1031;
90
+ cpu->isar.id_pfr1 = 0x11;
91
cpu->isar.id_dfr0 = 0x400;
92
cpu->id_afr0 = 0;
93
cpu->isar.id_mmfr0 = 0x31100003;
94
@@ -XXX,XX +XXX,XX @@ static void cortex_a9_initfn(Object *obj)
95
cpu->isar.mvfr1 = 0x01111111;
96
cpu->ctr = 0x80038003;
97
cpu->reset_sctlr = 0x00c50078;
98
- cpu->id_pfr0 = 0x1031;
99
- cpu->id_pfr1 = 0x11;
100
+ cpu->isar.id_pfr0 = 0x1031;
101
+ cpu->isar.id_pfr1 = 0x11;
102
cpu->isar.id_dfr0 = 0x000;
103
cpu->id_afr0 = 0;
104
cpu->isar.id_mmfr0 = 0x00100103;
105
@@ -XXX,XX +XXX,XX @@ static void cortex_a7_initfn(Object *obj)
106
cpu->isar.mvfr1 = 0x11111111;
107
cpu->ctr = 0x84448003;
108
cpu->reset_sctlr = 0x00c50078;
109
- cpu->id_pfr0 = 0x00001131;
110
- cpu->id_pfr1 = 0x00011011;
111
+ cpu->isar.id_pfr0 = 0x00001131;
112
+ cpu->isar.id_pfr1 = 0x00011011;
113
cpu->isar.id_dfr0 = 0x02010555;
114
cpu->id_afr0 = 0x00000000;
115
cpu->isar.id_mmfr0 = 0x10101105;
116
@@ -XXX,XX +XXX,XX @@ static void cortex_a15_initfn(Object *obj)
117
cpu->isar.mvfr1 = 0x11111111;
118
cpu->ctr = 0x8444c004;
119
cpu->reset_sctlr = 0x00c50078;
120
- cpu->id_pfr0 = 0x00001131;
121
- cpu->id_pfr1 = 0x00011011;
122
+ cpu->isar.id_pfr0 = 0x00001131;
123
+ cpu->isar.id_pfr1 = 0x00011011;
124
cpu->isar.id_dfr0 = 0x02010555;
125
cpu->id_afr0 = 0x00000000;
126
cpu->isar.id_mmfr0 = 0x10201105;
127
diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c
128
index XXXXXXX..XXXXXXX 100644
129
--- a/target/arm/cpu64.c
130
+++ b/target/arm/cpu64.c
131
@@ -XXX,XX +XXX,XX @@ static void aarch64_a57_initfn(Object *obj)
132
cpu->isar.mvfr2 = 0x00000043;
133
cpu->ctr = 0x8444c004;
134
cpu->reset_sctlr = 0x00c50838;
135
- cpu->id_pfr0 = 0x00000131;
136
- cpu->id_pfr1 = 0x00011011;
137
+ cpu->isar.id_pfr0 = 0x00000131;
138
+ cpu->isar.id_pfr1 = 0x00011011;
139
cpu->isar.id_dfr0 = 0x03010066;
140
cpu->id_afr0 = 0x00000000;
141
cpu->isar.id_mmfr0 = 0x10101105;
142
@@ -XXX,XX +XXX,XX @@ static void aarch64_a53_initfn(Object *obj)
143
cpu->isar.mvfr2 = 0x00000043;
144
cpu->ctr = 0x84448004; /* L1Ip = VIPT */
145
cpu->reset_sctlr = 0x00c50838;
146
- cpu->id_pfr0 = 0x00000131;
147
- cpu->id_pfr1 = 0x00011011;
148
+ cpu->isar.id_pfr0 = 0x00000131;
149
+ cpu->isar.id_pfr1 = 0x00011011;
150
cpu->isar.id_dfr0 = 0x03010066;
151
cpu->id_afr0 = 0x00000000;
152
cpu->isar.id_mmfr0 = 0x10101105;
153
@@ -XXX,XX +XXX,XX @@ static void aarch64_a72_initfn(Object *obj)
154
cpu->isar.mvfr2 = 0x00000043;
155
cpu->ctr = 0x8444c004;
156
cpu->reset_sctlr = 0x00c50838;
157
- cpu->id_pfr0 = 0x00000131;
158
- cpu->id_pfr1 = 0x00011011;
159
+ cpu->isar.id_pfr0 = 0x00000131;
160
+ cpu->isar.id_pfr1 = 0x00011011;
161
cpu->isar.id_dfr0 = 0x03010066;
162
cpu->id_afr0 = 0x00000000;
163
cpu->isar.id_mmfr0 = 0x10201105;
164
diff --git a/target/arm/cpu_tcg.c b/target/arm/cpu_tcg.c
165
index XXXXXXX..XXXXXXX 100644
166
--- a/target/arm/cpu_tcg.c
167
+++ b/target/arm/cpu_tcg.c
168
@@ -XXX,XX +XXX,XX @@ static void arm1136_r2_initfn(Object *obj)
169
cpu->isar.mvfr1 = 0x00000000;
170
cpu->ctr = 0x1dd20d2;
171
cpu->reset_sctlr = 0x00050078;
172
- cpu->id_pfr0 = 0x111;
173
- cpu->id_pfr1 = 0x1;
174
+ cpu->isar.id_pfr0 = 0x111;
175
+ cpu->isar.id_pfr1 = 0x1;
176
cpu->isar.id_dfr0 = 0x2;
177
cpu->id_afr0 = 0x3;
178
cpu->isar.id_mmfr0 = 0x01130003;
179
@@ -XXX,XX +XXX,XX @@ static void arm1136_initfn(Object *obj)
180
cpu->isar.mvfr1 = 0x00000000;
181
cpu->ctr = 0x1dd20d2;
182
cpu->reset_sctlr = 0x00050078;
183
- cpu->id_pfr0 = 0x111;
184
- cpu->id_pfr1 = 0x1;
185
+ cpu->isar.id_pfr0 = 0x111;
186
+ cpu->isar.id_pfr1 = 0x1;
187
cpu->isar.id_dfr0 = 0x2;
188
cpu->id_afr0 = 0x3;
189
cpu->isar.id_mmfr0 = 0x01130003;
190
@@ -XXX,XX +XXX,XX @@ static void arm1176_initfn(Object *obj)
191
cpu->isar.mvfr1 = 0x00000000;
192
cpu->ctr = 0x1dd20d2;
193
cpu->reset_sctlr = 0x00050078;
194
- cpu->id_pfr0 = 0x111;
195
- cpu->id_pfr1 = 0x11;
196
+ cpu->isar.id_pfr0 = 0x111;
197
+ cpu->isar.id_pfr1 = 0x11;
198
cpu->isar.id_dfr0 = 0x33;
199
cpu->id_afr0 = 0;
200
cpu->isar.id_mmfr0 = 0x01130003;
201
@@ -XXX,XX +XXX,XX @@ static void arm11mpcore_initfn(Object *obj)
202
cpu->isar.mvfr0 = 0x11111111;
203
cpu->isar.mvfr1 = 0x00000000;
204
cpu->ctr = 0x1d192992; /* 32K icache 32K dcache */
205
- cpu->id_pfr0 = 0x111;
206
- cpu->id_pfr1 = 0x1;
207
+ cpu->isar.id_pfr0 = 0x111;
208
+ cpu->isar.id_pfr1 = 0x1;
209
cpu->isar.id_dfr0 = 0;
210
cpu->id_afr0 = 0x2;
211
cpu->isar.id_mmfr0 = 0x01100103;
212
@@ -XXX,XX +XXX,XX @@ static void cortex_m3_initfn(Object *obj)
213
set_feature(&cpu->env, ARM_FEATURE_M_MAIN);
214
cpu->midr = 0x410fc231;
215
cpu->pmsav7_dregion = 8;
216
- cpu->id_pfr0 = 0x00000030;
217
- cpu->id_pfr1 = 0x00000200;
218
+ cpu->isar.id_pfr0 = 0x00000030;
219
+ cpu->isar.id_pfr1 = 0x00000200;
220
cpu->isar.id_dfr0 = 0x00100000;
221
cpu->id_afr0 = 0x00000000;
222
cpu->isar.id_mmfr0 = 0x00000030;
223
@@ -XXX,XX +XXX,XX @@ static void cortex_m4_initfn(Object *obj)
224
cpu->isar.mvfr0 = 0x10110021;
225
cpu->isar.mvfr1 = 0x11000011;
226
cpu->isar.mvfr2 = 0x00000000;
227
- cpu->id_pfr0 = 0x00000030;
228
- cpu->id_pfr1 = 0x00000200;
229
+ cpu->isar.id_pfr0 = 0x00000030;
230
+ cpu->isar.id_pfr1 = 0x00000200;
231
cpu->isar.id_dfr0 = 0x00100000;
232
cpu->id_afr0 = 0x00000000;
233
cpu->isar.id_mmfr0 = 0x00000030;
234
@@ -XXX,XX +XXX,XX @@ static void cortex_m7_initfn(Object *obj)
235
cpu->isar.mvfr0 = 0x10110221;
236
cpu->isar.mvfr1 = 0x12000011;
237
cpu->isar.mvfr2 = 0x00000040;
238
- cpu->id_pfr0 = 0x00000030;
239
- cpu->id_pfr1 = 0x00000200;
240
+ cpu->isar.id_pfr0 = 0x00000030;
241
+ cpu->isar.id_pfr1 = 0x00000200;
242
cpu->isar.id_dfr0 = 0x00100000;
243
cpu->id_afr0 = 0x00000000;
244
cpu->isar.id_mmfr0 = 0x00100030;
245
@@ -XXX,XX +XXX,XX @@ static void cortex_m33_initfn(Object *obj)
246
cpu->isar.mvfr0 = 0x10110021;
247
cpu->isar.mvfr1 = 0x11000011;
248
cpu->isar.mvfr2 = 0x00000040;
249
- cpu->id_pfr0 = 0x00000030;
250
- cpu->id_pfr1 = 0x00000210;
251
+ cpu->isar.id_pfr0 = 0x00000030;
252
+ cpu->isar.id_pfr1 = 0x00000210;
253
cpu->isar.id_dfr0 = 0x00200000;
254
cpu->id_afr0 = 0x00000000;
255
cpu->isar.id_mmfr0 = 0x00101F40;
256
@@ -XXX,XX +XXX,XX @@ static void cortex_r5_initfn(Object *obj)
257
set_feature(&cpu->env, ARM_FEATURE_PMSA);
258
set_feature(&cpu->env, ARM_FEATURE_PMU);
259
cpu->midr = 0x411fc153; /* r1p3 */
260
- cpu->id_pfr0 = 0x0131;
261
- cpu->id_pfr1 = 0x001;
262
+ cpu->isar.id_pfr0 = 0x0131;
263
+ cpu->isar.id_pfr1 = 0x001;
264
cpu->isar.id_dfr0 = 0x010400;
265
cpu->id_afr0 = 0x0;
266
cpu->isar.id_mmfr0 = 0x0210030;
267
diff --git a/target/arm/helper.c b/target/arm/helper.c
268
index XXXXXXX..XXXXXXX 100644
269
--- a/target/arm/helper.c
270
+++ b/target/arm/helper.c
271
@@ -XXX,XX +XXX,XX @@ static void define_pmu_regs(ARMCPU *cpu)
272
static uint64_t id_pfr1_read(CPUARMState *env, const ARMCPRegInfo *ri)
273
{
274
ARMCPU *cpu = env_archcpu(env);
275
- uint64_t pfr1 = cpu->id_pfr1;
276
+ uint64_t pfr1 = cpu->isar.id_pfr1;
277
278
if (env->gicv3state) {
279
pfr1 |= 1 << 28;
280
@@ -XXX,XX +XXX,XX @@ void register_cp_regs_for_features(ARMCPU *cpu)
281
.opc0 = 3, .opc1 = 0, .crn = 0, .crm = 1, .opc2 = 0,
282
.access = PL1_R, .type = ARM_CP_CONST,
283
.accessfn = access_aa32_tid3,
284
- .resetvalue = cpu->id_pfr0 },
285
+ .resetvalue = cpu->isar.id_pfr0 },
286
/* ID_PFR1 is not a plain ARM_CP_CONST because we don't know
287
* the value of the GIC field until after we define these regs.
288
*/
289
diff --git a/target/arm/kvm64.c b/target/arm/kvm64.c
290
index XXXXXXX..XXXXXXX 100644
291
--- a/target/arm/kvm64.c
292
+++ b/target/arm/kvm64.c
293
@@ -XXX,XX +XXX,XX @@ bool kvm_arm_get_host_cpu_features(ARMHostCPUFeatures *ahcf)
294
* than skipping the reads and leaving 0, as we must avoid
295
* considering the values in every case.
296
*/
297
+ err |= read_sys_reg32(fdarray[2], &ahcf->isar.id_pfr0,
298
+ ARM64_SYS_REG(3, 0, 0, 1, 0));
299
+ err |= read_sys_reg32(fdarray[2], &ahcf->isar.id_pfr1,
300
+ ARM64_SYS_REG(3, 0, 0, 1, 1));
301
err |= read_sys_reg32(fdarray[2], &ahcf->isar.id_dfr0,
302
ARM64_SYS_REG(3, 0, 0, 1, 2));
303
err |= read_sys_reg32(fdarray[2], &ahcf->isar.id_mmfr0,
304
--
66
--
305
2.20.1
67
2.34.1
306
307
diff view generated by jsdifflib
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
1
In commit fe4a5472ccd6 we rearranged the logic in S1_ptw_translate()
2
so that the debug-access "call get_phys_addr_*" codepath is used both
3
when S1 is doing ptw reads from stage 2 and when it is doing ptw
4
reads from physical memory. However, we didn't update the
5
calculation of s2ptw->in_space and s2ptw->in_secure to account for
6
the "ptw reads from physical memory" case. This meant that debug
7
accesses when in Secure state broke.
2
8
3
Using class_data pointer to create a MachineClass is not
9
Create a new function S2_security_space() which returns the
4
the recommended way anymore. The correct way is to open-code
10
correct security space to use for the ptw load, and use it to
5
the MachineClass::fields in the class_init() method.
11
determine the correct .in_secure and .in_space fields for the
12
stage 2 lookup for the ptw load.
6
13
7
We can not use TYPE_RASPI_MACHINE::class_base_init() because
14
Reported-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
8
it is called *before* each machine class_init(), therefore the
15
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
9
board_rev field is not populated. We have to manually call
16
Tested-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
10
raspi_machine_class_common_init() for each machine.
11
12
This partly reverts commit a03bde3674e.
13
14
Suggested-by: Igor Mammedov <imammedo@redhat.com>
15
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
17
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
16
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
18
Message-id: 20230710152130.3928330-3-peter.maydell@linaro.org
17
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
19
Fixes: fe4a5472ccd6 ("target/arm: Use get_phys_addr_with_struct in S1_ptw_translate")
18
Message-id: 20200924111808.77168-5-f4bug@amsat.org
19
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
20
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
20
---
21
---
21
hw/arm/raspi.c | 34 ++++++++++++++++++++++++----------
22
target/arm/ptw.c | 37 ++++++++++++++++++++++++++++++++-----
22
1 file changed, 24 insertions(+), 10 deletions(-)
23
1 file changed, 32 insertions(+), 5 deletions(-)
23
24
24
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
25
diff --git a/target/arm/ptw.c b/target/arm/ptw.c
25
index XXXXXXX..XXXXXXX 100644
26
index XXXXXXX..XXXXXXX 100644
26
--- a/hw/arm/raspi.c
27
--- a/target/arm/ptw.c
27
+++ b/hw/arm/raspi.c
28
+++ b/target/arm/ptw.c
28
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_init(MachineState *machine)
29
@@ -XXX,XX +XXX,XX @@ static bool S2_attrs_are_device(uint64_t hcr, uint8_t attrs)
29
setup_boot(machine, version, machine->ram_size - vcram_size);
30
}
30
}
31
}
31
32
32
-static void raspi_machine_class_init(ObjectClass *oc, void *data)
33
+static ARMSecuritySpace S2_security_space(ARMSecuritySpace s1_space,
33
+static void raspi_machine_class_common_init(MachineClass *mc,
34
+ ARMMMUIdx s2_mmu_idx)
34
+ uint32_t board_rev)
35
+{
36
+ /*
37
+ * Return the security space to use for stage 2 when doing
38
+ * the S1 page table descriptor load.
39
+ */
40
+ if (regime_is_stage2(s2_mmu_idx)) {
41
+ /*
42
+ * The security space for ptw reads is almost always the same
43
+ * as that of the security space of the stage 1 translation.
44
+ * The only exception is when stage 1 is Secure; in that case
45
+ * the ptw read might be to the Secure or the NonSecure space
46
+ * (but never Realm or Root), and the s2_mmu_idx tells us which.
47
+ * Root translations are always single-stage.
48
+ */
49
+ if (s1_space == ARMSS_Secure) {
50
+ return arm_secure_to_space(s2_mmu_idx == ARMMMUIdx_Stage2_S);
51
+ } else {
52
+ assert(s2_mmu_idx != ARMMMUIdx_Stage2_S);
53
+ assert(s1_space != ARMSS_Root);
54
+ return s1_space;
55
+ }
56
+ } else {
57
+ /* ptw loads are from phys: the mmu idx itself says which space */
58
+ return arm_phys_to_space(s2_mmu_idx);
59
+ }
60
+}
61
+
62
/* Translate a S1 pagetable walk through S2 if needed. */
63
static bool S1_ptw_translate(CPUARMState *env, S1Translate *ptw,
64
hwaddr addr, ARMMMUFaultInfo *fi)
35
{
65
{
36
- MachineClass *mc = MACHINE_CLASS(oc);
66
- ARMSecuritySpace space = ptw->in_space;
37
- RaspiMachineClass *rmc = RASPI_MACHINE_CLASS(oc);
67
bool is_secure = ptw->in_secure;
38
- uint32_t board_rev = (uint32_t)(uintptr_t)data;
68
ARMMMUIdx mmu_idx = ptw->in_mmu_idx;
39
-
69
ARMMMUIdx s2_mmu_idx = ptw->in_ptw_idx;
40
- rmc->board_rev = board_rev;
70
@@ -XXX,XX +XXX,XX @@ static bool S1_ptw_translate(CPUARMState *env, S1Translate *ptw,
41
mc->desc = g_strdup_printf("Raspberry Pi %s (revision 1.%u)",
71
* From gdbstub, do not use softmmu so that we don't modify the
42
board_type(board_rev),
72
* state of the cpu at all, including softmmu tlb contents.
43
FIELD_EX32(board_rev, REV_CODE, REVISION));
73
*/
44
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_class_init(ObjectClass *oc, void *data)
74
+ ARMSecuritySpace s2_space = S2_security_space(ptw->in_space, s2_mmu_idx);
45
mc->default_ram_id = "ram";
75
S1Translate s2ptw = {
46
};
76
.in_mmu_idx = s2_mmu_idx,
47
77
.in_ptw_idx = ptw_idx_for_stage_2(env, s2_mmu_idx),
48
+static void raspi2b_machine_class_init(ObjectClass *oc, void *data)
78
- .in_secure = s2_mmu_idx == ARMMMUIdx_Stage2_S,
49
+{
79
- .in_space = (s2_mmu_idx == ARMMMUIdx_Stage2_S ? ARMSS_Secure
50
+ MachineClass *mc = MACHINE_CLASS(oc);
80
- : space == ARMSS_Realm ? ARMSS_Realm
51
+ RaspiMachineClass *rmc = RASPI_MACHINE_CLASS(oc);
81
- : ARMSS_NonSecure),
52
+
82
+ .in_secure = arm_space_is_secure(s2_space),
53
+ rmc->board_rev = 0xa21041;
83
+ .in_space = s2_space,
54
+ raspi_machine_class_common_init(mc, rmc->board_rev);
84
.in_debug = true,
55
+};
85
};
56
+
86
GetPhysAddrResult s2 = { };
57
+#ifdef TARGET_AARCH64
58
+static void raspi3b_machine_class_init(ObjectClass *oc, void *data)
59
+{
60
+ MachineClass *mc = MACHINE_CLASS(oc);
61
+ RaspiMachineClass *rmc = RASPI_MACHINE_CLASS(oc);
62
+
63
+ rmc->board_rev = 0xa02082;
64
+ raspi_machine_class_common_init(mc, rmc->board_rev);
65
+};
66
+#endif /* TARGET_AARCH64 */
67
+
68
static const TypeInfo raspi_machine_types[] = {
69
{
70
.name = MACHINE_TYPE_NAME("raspi2"),
71
.parent = TYPE_RASPI_MACHINE,
72
- .class_init = raspi_machine_class_init,
73
- .class_data = (void *)0xa21041,
74
+ .class_init = raspi2b_machine_class_init,
75
#ifdef TARGET_AARCH64
76
}, {
77
.name = MACHINE_TYPE_NAME("raspi3"),
78
.parent = TYPE_RASPI_MACHINE,
79
- .class_init = raspi_machine_class_init,
80
- .class_data = (void *)0xa02082,
81
+ .class_init = raspi3b_machine_class_init,
82
#endif
83
}, {
84
.name = TYPE_RASPI_MACHINE,
85
--
87
--
86
2.20.1
88
2.34.1
87
88
diff view generated by jsdifflib
1
The ARM_FEATURE_PXN bit indicates whether the CPU supports the PXN
1
In get_phys_addr_twostage() the code that applies the effects of
2
bit in short-descriptor translation table format descriptors. This
2
VSTCR.{SA,SW} and VTCR.{NSA,NSW} only updates result->f.attrs.secure.
3
is indicated by ID_MMFR0.VMSA being at least 0b0100. Replace the
3
Now we also have f.attrs.space for FEAT_RME, we need to keep the two
4
feature bit with an ID register check, in line with our preference
4
in sync.
5
for ID register checks over feature bits.
5
6
These bits only have an effect for Secure space translations, not
7
for Root, so use the input in_space field to determine whether to
8
apply them rather than the input is_secure. This doesn't actually
9
make a difference because Root translations are never two-stage,
10
but it's a little clearer.
6
11
7
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
8
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
13
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
9
Message-id: 20200910173855.4068-2-peter.maydell@linaro.org
14
Message-id: 20230710152130.3928330-4-peter.maydell@linaro.org
10
---
15
---
11
target/arm/cpu.h | 15 ++++++++++++++-
16
target/arm/ptw.c | 13 ++++++++-----
12
target/arm/cpu.c | 1 -
17
1 file changed, 8 insertions(+), 5 deletions(-)
13
target/arm/helper.c | 5 +++--
14
3 files changed, 17 insertions(+), 4 deletions(-)
15
18
16
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
19
diff --git a/target/arm/ptw.c b/target/arm/ptw.c
17
index XXXXXXX..XXXXXXX 100644
20
index XXXXXXX..XXXXXXX 100644
18
--- a/target/arm/cpu.h
21
--- a/target/arm/ptw.c
19
+++ b/target/arm/cpu.h
22
+++ b/target/arm/ptw.c
20
@@ -XXX,XX +XXX,XX @@ FIELD(ID_ISAR6, FHM, 8, 4)
23
@@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_twostage(CPUARMState *env, S1Translate *ptw,
21
FIELD(ID_ISAR6, SB, 12, 4)
24
hwaddr ipa;
22
FIELD(ID_ISAR6, SPECRES, 16, 4)
25
int s1_prot, s1_lgpgsz;
23
26
bool is_secure = ptw->in_secure;
24
+FIELD(ID_MMFR0, VMSA, 0, 4)
27
+ ARMSecuritySpace in_space = ptw->in_space;
25
+FIELD(ID_MMFR0, PMSA, 4, 4)
28
bool ret, ipa_secure;
26
+FIELD(ID_MMFR0, OUTERSHR, 8, 4)
29
ARMCacheAttrs cacheattrs1;
27
+FIELD(ID_MMFR0, SHARELVL, 12, 4)
30
ARMSecuritySpace ipa_space;
28
+FIELD(ID_MMFR0, TCM, 16, 4)
31
@@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_twostage(CPUARMState *env, S1Translate *ptw,
29
+FIELD(ID_MMFR0, AUXREG, 20, 4)
32
* Check if IPA translates to secure or non-secure PA space.
30
+FIELD(ID_MMFR0, FCSE, 24, 4)
33
* Note that VSTCR overrides VTCR and {N}SW overrides {N}SA.
31
+FIELD(ID_MMFR0, INNERSHR, 28, 4)
34
*/
32
+
35
- result->f.attrs.secure =
33
FIELD(ID_MMFR3, CMAINTVA, 0, 4)
36
- (is_secure
34
FIELD(ID_MMFR3, CMAINTSW, 4, 4)
37
- && !(env->cp15.vstcr_el2 & (VSTCR_SA | VSTCR_SW))
35
FIELD(ID_MMFR3, BPMAINT, 8, 4)
38
- && (ipa_secure
36
@@ -XXX,XX +XXX,XX @@ enum arm_features {
39
- || !(env->cp15.vtcr_el2 & (VTCR_NSA | VTCR_NSW))));
37
ARM_FEATURE_CACHE_DIRTY_REG, /* 1136/1176 cache dirty status register */
40
+ if (in_space == ARMSS_Secure) {
38
ARM_FEATURE_CACHE_BLOCK_OPS, /* v6 optional cache block operations */
41
+ result->f.attrs.secure =
39
ARM_FEATURE_MPIDR, /* has cp15 MPIDR */
42
+ !(env->cp15.vstcr_el2 & (VSTCR_SA | VSTCR_SW))
40
- ARM_FEATURE_PXN, /* has Privileged Execute Never bit */
43
+ && (ipa_secure
41
ARM_FEATURE_LPAE, /* has Large Physical Address Extension */
44
+ || !(env->cp15.vtcr_el2 & (VTCR_NSA | VTCR_NSW)));
42
ARM_FEATURE_V8,
45
+ result->f.attrs.space = arm_secure_to_space(result->f.attrs.secure);
43
ARM_FEATURE_AARCH64, /* supports 64 bit mode */
46
+ }
44
@@ -XXX,XX +XXX,XX @@ static inline bool isar_feature_aa32_vminmaxnm(const ARMISARegisters *id)
47
45
return FIELD_EX32(id->mvfr2, MVFR2, FPMISC) >= 4;
48
return false;
46
}
49
}
47
48
+static inline bool isar_feature_aa32_pxn(const ARMISARegisters *id)
49
+{
50
+ return FIELD_EX32(id->id_mmfr0, ID_MMFR0, VMSA) >= 4;
51
+}
52
+
53
static inline bool isar_feature_aa32_pan(const ARMISARegisters *id)
54
{
55
return FIELD_EX32(id->id_mmfr3, ID_MMFR3, PAN) != 0;
56
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
57
index XXXXXXX..XXXXXXX 100644
58
--- a/target/arm/cpu.c
59
+++ b/target/arm/cpu.c
60
@@ -XXX,XX +XXX,XX @@ static void arm_cpu_realizefn(DeviceState *dev, Error **errp)
61
}
62
if (arm_feature(env, ARM_FEATURE_LPAE)) {
63
set_feature(env, ARM_FEATURE_V7MP);
64
- set_feature(env, ARM_FEATURE_PXN);
65
}
66
if (arm_feature(env, ARM_FEATURE_CBAR_RO)) {
67
set_feature(env, ARM_FEATURE_CBAR);
68
diff --git a/target/arm/helper.c b/target/arm/helper.c
69
index XXXXXXX..XXXXXXX 100644
70
--- a/target/arm/helper.c
71
+++ b/target/arm/helper.c
72
@@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_v6(CPUARMState *env, uint32_t address,
73
target_ulong *page_size, ARMMMUFaultInfo *fi)
74
{
75
CPUState *cs = env_cpu(env);
76
+ ARMCPU *cpu = env_archcpu(env);
77
int level = 1;
78
uint32_t table;
79
uint32_t desc;
80
@@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_v6(CPUARMState *env, uint32_t address,
81
goto do_fault;
82
}
83
type = (desc & 3);
84
- if (type == 0 || (type == 3 && !arm_feature(env, ARM_FEATURE_PXN))) {
85
+ if (type == 0 || (type == 3 && !cpu_isar_feature(aa32_pxn, cpu))) {
86
/* Section translation fault, or attempt to use the encoding
87
* which is Reserved on implementations without PXN.
88
*/
89
@@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_v6(CPUARMState *env, uint32_t address,
90
pxn = desc & 1;
91
ns = extract32(desc, 19, 1);
92
} else {
93
- if (arm_feature(env, ARM_FEATURE_PXN)) {
94
+ if (cpu_isar_feature(aa32_pxn, cpu)) {
95
pxn = (desc >> 2) & 1;
96
}
97
ns = extract32(desc, 3, 1);
98
--
50
--
99
2.20.1
51
2.34.1
100
101
diff view generated by jsdifflib
Deleted patch
1
Give the Cortex-M0 ID register values corresponding to its
2
implemented behaviour. These will not be guest-visible but will be
3
used to govern the behaviour of QEMU's emulation. We use the same
4
values that the Cortex-M3 does.
5
1
6
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
7
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
8
Message-id: 20200910173855.4068-5-peter.maydell@linaro.org
9
---
10
target/arm/cpu_tcg.c | 24 ++++++++++++++++++++++++
11
1 file changed, 24 insertions(+)
12
13
diff --git a/target/arm/cpu_tcg.c b/target/arm/cpu_tcg.c
14
index XXXXXXX..XXXXXXX 100644
15
--- a/target/arm/cpu_tcg.c
16
+++ b/target/arm/cpu_tcg.c
17
@@ -XXX,XX +XXX,XX @@ static void cortex_m0_initfn(Object *obj)
18
set_feature(&cpu->env, ARM_FEATURE_M);
19
20
cpu->midr = 0x410cc200;
21
+
22
+ /*
23
+ * These ID register values are not guest visible, because
24
+ * we do not implement the Main Extension. They must be set
25
+ * to values corresponding to the Cortex-M0's implemented
26
+ * features, because QEMU generally controls its emulation
27
+ * by looking at ID register fields. We use the same values as
28
+ * for the M3.
29
+ */
30
+ cpu->isar.id_pfr0 = 0x00000030;
31
+ cpu->isar.id_pfr1 = 0x00000200;
32
+ cpu->isar.id_dfr0 = 0x00100000;
33
+ cpu->id_afr0 = 0x00000000;
34
+ cpu->isar.id_mmfr0 = 0x00000030;
35
+ cpu->isar.id_mmfr1 = 0x00000000;
36
+ cpu->isar.id_mmfr2 = 0x00000000;
37
+ cpu->isar.id_mmfr3 = 0x00000000;
38
+ cpu->isar.id_isar0 = 0x01141110;
39
+ cpu->isar.id_isar1 = 0x02111000;
40
+ cpu->isar.id_isar2 = 0x21112231;
41
+ cpu->isar.id_isar3 = 0x01111110;
42
+ cpu->isar.id_isar4 = 0x01310102;
43
+ cpu->isar.id_isar5 = 0x00000000;
44
+ cpu->isar.id_isar6 = 0x00000000;
45
}
46
47
static void cortex_m3_initfn(Object *obj)
48
--
49
2.20.1
50
51
diff view generated by jsdifflib
Deleted patch
1
The M-profile definition of the MVFR1 ID register differs slightly
2
from the A-profile one, and in particular the check for "does the CPU
3
support fp16 arithmetic" is not the same.
4
1
5
We don't currently implement any M-profile CPUs with fp16 arithmetic,
6
so this is not yet a visible bug, but correcting the logic now
7
disarms this beartrap for when we eventually do.
8
9
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
10
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
11
Message-id: 20200910173855.4068-6-peter.maydell@linaro.org
12
---
13
target/arm/cpu.h | 31 ++++++++++++++++++++++++++-----
14
1 file changed, 26 insertions(+), 5 deletions(-)
15
16
diff --git a/target/arm/cpu.h b/target/arm/cpu.h
17
index XXXXXXX..XXXXXXX 100644
18
--- a/target/arm/cpu.h
19
+++ b/target/arm/cpu.h
20
@@ -XXX,XX +XXX,XX @@ FIELD(ID_MMFR4, LSM, 20, 4)
21
FIELD(ID_MMFR4, CCIDX, 24, 4)
22
FIELD(ID_MMFR4, EVT, 28, 4)
23
24
+FIELD(ID_PFR1, PROGMOD, 0, 4)
25
+FIELD(ID_PFR1, SECURITY, 4, 4)
26
+FIELD(ID_PFR1, MPROGMOD, 8, 4)
27
+FIELD(ID_PFR1, VIRTUALIZATION, 12, 4)
28
+FIELD(ID_PFR1, GENTIMER, 16, 4)
29
+FIELD(ID_PFR1, SEC_FRAC, 20, 4)
30
+FIELD(ID_PFR1, VIRT_FRAC, 24, 4)
31
+FIELD(ID_PFR1, GIC, 28, 4)
32
+
33
FIELD(ID_AA64ISAR0, AES, 4, 4)
34
FIELD(ID_AA64ISAR0, SHA1, 8, 4)
35
FIELD(ID_AA64ISAR0, SHA2, 12, 4)
36
@@ -XXX,XX +XXX,XX @@ FIELD(MVFR0, FPROUND, 28, 4)
37
38
FIELD(MVFR1, FPFTZ, 0, 4)
39
FIELD(MVFR1, FPDNAN, 4, 4)
40
-FIELD(MVFR1, SIMDLS, 8, 4)
41
-FIELD(MVFR1, SIMDINT, 12, 4)
42
-FIELD(MVFR1, SIMDSP, 16, 4)
43
-FIELD(MVFR1, SIMDHP, 20, 4)
44
+FIELD(MVFR1, SIMDLS, 8, 4) /* A-profile only */
45
+FIELD(MVFR1, SIMDINT, 12, 4) /* A-profile only */
46
+FIELD(MVFR1, SIMDSP, 16, 4) /* A-profile only */
47
+FIELD(MVFR1, SIMDHP, 20, 4) /* A-profile only */
48
+FIELD(MVFR1, MVE, 8, 4) /* M-profile only */
49
+FIELD(MVFR1, FP16, 20, 4) /* M-profile only */
50
FIELD(MVFR1, FPHP, 24, 4)
51
FIELD(MVFR1, SIMDFMAC, 28, 4)
52
53
@@ -XXX,XX +XXX,XX @@ static inline bool isar_feature_aa32_predinv(const ARMISARegisters *id)
54
return FIELD_EX32(id->id_isar6, ID_ISAR6, SPECRES) != 0;
55
}
56
57
+static inline bool isar_feature_aa32_mprofile(const ARMISARegisters *id)
58
+{
59
+ return FIELD_EX32(id->id_pfr1, ID_PFR1, MPROGMOD) != 0;
60
+}
61
+
62
static inline bool isar_feature_aa32_fp16_arith(const ARMISARegisters *id)
63
{
64
- return FIELD_EX32(id->mvfr1, MVFR1, FPHP) >= 3;
65
+ /* Sadly this is encoded differently for A-profile and M-profile */
66
+ if (isar_feature_aa32_mprofile(id)) {
67
+ return FIELD_EX32(id->mvfr1, MVFR1, FP16) > 0;
68
+ } else {
69
+ return FIELD_EX32(id->mvfr1, MVFR1, FPHP) >= 3;
70
+ }
71
}
72
73
static inline bool isar_feature_aa32_vfp_simd(const ARMISARegisters *id)
74
--
75
2.20.1
76
77
diff view generated by jsdifflib
Deleted patch
1
From: Richard Henderson <richard.henderson@linaro.org>
2
1
3
The mte update missed a bit when producing clean addresses.
4
5
Fixes: b2aa8879b88
6
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
7
Message-id: 20200916014102.2446323-1-richard.henderson@linaro.org
8
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
9
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
10
---
11
target/arm/translate-sve.c | 4 ++--
12
1 file changed, 2 insertions(+), 2 deletions(-)
13
14
diff --git a/target/arm/translate-sve.c b/target/arm/translate-sve.c
15
index XXXXXXX..XXXXXXX 100644
16
--- a/target/arm/translate-sve.c
17
+++ b/target/arm/translate-sve.c
18
@@ -XXX,XX +XXX,XX @@ static void do_ldr(DisasContext *s, uint32_t vofs, int len, int rn, int imm)
19
for (i = 0; i < len_align; i += 8) {
20
tcg_gen_qemu_ld_i64(t0, clean_addr, midx, MO_LEQ);
21
tcg_gen_st_i64(t0, cpu_env, vofs + i);
22
- tcg_gen_addi_i64(clean_addr, cpu_reg_sp(s, rn), 8);
23
+ tcg_gen_addi_i64(clean_addr, clean_addr, 8);
24
}
25
tcg_temp_free_i64(t0);
26
} else {
27
@@ -XXX,XX +XXX,XX @@ static void do_str(DisasContext *s, uint32_t vofs, int len, int rn, int imm)
28
for (i = 0; i < len_align; i += 8) {
29
tcg_gen_ld_i64(t0, cpu_env, vofs + i);
30
tcg_gen_qemu_st_i64(t0, clean_addr, midx, MO_LEQ);
31
- tcg_gen_addi_i64(clean_addr, cpu_reg_sp(s, rn), 8);
32
+ tcg_gen_addi_i64(clean_addr, clean_addr, 8);
33
}
34
tcg_temp_free_i64(t0);
35
} else {
36
--
37
2.20.1
38
39
diff view generated by jsdifflib
Deleted patch
1
From: Richard Henderson <richard.henderson@linaro.org>
2
1
3
While converting to gen_gvec_ool_zzzp, we lost passing
4
a->esz as the data argument to the function.
5
6
Fixes: 36cbb7a8e71
7
Cc: qemu-stable@nongnu.org
8
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
9
Message-id: 20200918000500.2690937-1-richard.henderson@linaro.org
10
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
target/arm/translate-sve.c | 2 +-
14
1 file changed, 1 insertion(+), 1 deletion(-)
15
16
diff --git a/target/arm/translate-sve.c b/target/arm/translate-sve.c
17
index XXXXXXX..XXXXXXX 100644
18
--- a/target/arm/translate-sve.c
19
+++ b/target/arm/translate-sve.c
20
@@ -XXX,XX +XXX,XX @@ static bool trans_SPLICE(DisasContext *s, arg_rprr_esz *a)
21
{
22
if (sve_access_check(s)) {
23
gen_gvec_ool_zzzp(s, gen_helper_sve_splice,
24
- a->rd, a->rn, a->rm, a->pg, 0);
25
+ a->rd, a->rn, a->rm, a->pg, a->esz);
26
}
27
return true;
28
}
29
--
30
2.20.1
31
32
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
The Raspberry firmware is closed-source. While running it, it
4
accesses various I/O registers. Logging these accesses as UNIMP
5
(unimplemented) help to understand what the firmware is doing
6
(ideally we want it able to boot a Linux kernel).
7
8
Document various blocks we might use later.
9
10
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
11
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
12
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
13
Message-id: 20200921034729.432931-2-f4bug@amsat.org
14
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
15
---
16
include/hw/arm/raspi_platform.h | 51 +++++++++++++++++++++++++++------
17
1 file changed, 43 insertions(+), 8 deletions(-)
18
19
diff --git a/include/hw/arm/raspi_platform.h b/include/hw/arm/raspi_platform.h
20
index XXXXXXX..XXXXXXX 100644
21
--- a/include/hw/arm/raspi_platform.h
22
+++ b/include/hw/arm/raspi_platform.h
23
@@ -XXX,XX +XXX,XX @@
24
* You should have received a copy of the GNU General Public License
25
* along with this program; if not, write to the Free Software
26
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
27
+ *
28
+ * Various undocumented addresses and names come from Herman Hermitage's VC4
29
+ * documentation:
30
+ * https://github.com/hermanhermitage/videocoreiv/wiki/MMIO-Register-map
31
*/
32
33
#ifndef HW_ARM_RASPI_PLATFORM_H
34
#define HW_ARM_RASPI_PLATFORM_H
35
36
#define MSYNC_OFFSET 0x0000 /* Multicore Sync Block */
37
-#define IC0_OFFSET 0x2000
38
+#define CCPT_OFFSET 0x1000 /* Compact Camera Port 2 TX */
39
+#define INTE_OFFSET 0x2000 /* VC Interrupt controller */
40
#define ST_OFFSET 0x3000 /* System Timer */
41
+#define TXP_OFFSET 0x4000 /* Transposer */
42
+#define JPEG_OFFSET 0x5000
43
#define MPHI_OFFSET 0x6000 /* Message-based Parallel Host Intf. */
44
#define DMA_OFFSET 0x7000 /* DMA controller, channels 0-14 */
45
-#define ARM_OFFSET 0xB000 /* BCM2708 ARM control block */
46
+#define ARBA_OFFSET 0x9000
47
+#define BRDG_OFFSET 0xa000
48
+#define ARM_OFFSET 0xB000 /* ARM control block */
49
#define ARMCTRL_OFFSET (ARM_OFFSET + 0x000)
50
#define ARMCTRL_IC_OFFSET (ARM_OFFSET + 0x200) /* Interrupt controller */
51
-#define ARMCTRL_TIMER0_1_OFFSET (ARM_OFFSET + 0x400) /* Timer 0 and 1 */
52
+#define ARMCTRL_TIMER0_1_OFFSET (ARM_OFFSET + 0x400) /* Timer 0 and 1 (SP804) */
53
#define ARMCTRL_0_SBM_OFFSET (ARM_OFFSET + 0x800) /* User 0 (ARM) Semaphores
54
* Doorbells & Mailboxes */
55
#define CPRMAN_OFFSET 0x100000 /* Power Management, Watchdog */
56
@@ -XXX,XX +XXX,XX @@
57
#define AVS_OFFSET 0x103000 /* Audio Video Standard */
58
#define RNG_OFFSET 0x104000
59
#define GPIO_OFFSET 0x200000
60
-#define UART0_OFFSET 0x201000
61
-#define MMCI0_OFFSET 0x202000
62
-#define I2S_OFFSET 0x203000
63
-#define SPI0_OFFSET 0x204000
64
+#define UART0_OFFSET 0x201000 /* PL011 */
65
+#define MMCI0_OFFSET 0x202000 /* Legacy MMC */
66
+#define I2S_OFFSET 0x203000 /* PCM */
67
+#define SPI0_OFFSET 0x204000 /* SPI master */
68
#define BSC0_OFFSET 0x205000 /* BSC0 I2C/TWI */
69
+#define PIXV0_OFFSET 0x206000
70
+#define PIXV1_OFFSET 0x207000
71
+#define DPI_OFFSET 0x208000
72
+#define DSI0_OFFSET 0x209000 /* Display Serial Interface */
73
+#define PWM_OFFSET 0x20c000
74
+#define PERM_OFFSET 0x20d000
75
+#define TEC_OFFSET 0x20e000
76
#define OTP_OFFSET 0x20f000
77
+#define SLIM_OFFSET 0x210000 /* SLIMbus */
78
+#define CPG_OFFSET 0x211000
79
#define THERMAL_OFFSET 0x212000
80
-#define BSC_SL_OFFSET 0x214000 /* SPI slave */
81
+#define AVSP_OFFSET 0x213000
82
+#define BSC_SL_OFFSET 0x214000 /* SPI slave (bootrom) */
83
#define AUX_OFFSET 0x215000 /* AUX: UART1/SPI1/SPI2 */
84
#define EMMC1_OFFSET 0x300000
85
+#define EMMC2_OFFSET 0x340000
86
+#define HVS_OFFSET 0x400000
87
#define SMI_OFFSET 0x600000
88
+#define DSI1_OFFSET 0x700000
89
+#define UCAM_OFFSET 0x800000
90
+#define CMI_OFFSET 0x802000
91
#define BSC1_OFFSET 0x804000 /* BSC1 I2C/TWI */
92
#define BSC2_OFFSET 0x805000 /* BSC2 I2C/TWI */
93
+#define VECA_OFFSET 0x806000
94
+#define PIXV2_OFFSET 0x807000
95
+#define HDMI_OFFSET 0x808000
96
+#define HDCP_OFFSET 0x809000
97
+#define ARBR0_OFFSET 0x80a000
98
#define DBUS_OFFSET 0x900000
99
#define AVE0_OFFSET 0x910000
100
#define USB_OTG_OFFSET 0x980000 /* DTC_OTG USB controller */
101
+#define V3D_OFFSET 0xc00000
102
#define SDRAMC_OFFSET 0xe00000
103
+#define L2CC_OFFSET 0xe01000 /* Level 2 Cache controller */
104
+#define L1CC_OFFSET 0xe02000 /* Level 1 Cache controller */
105
+#define ARBR1_OFFSET 0xe04000
106
#define DMA15_OFFSET 0xE05000 /* DMA controller, channel 15 */
107
+#define DCRC_OFFSET 0xe07000
108
+#define AXIP_OFFSET 0xe08000
109
110
/* GPU interrupts */
111
#define INTERRUPT_TIMER0 0
112
--
113
2.20.1
114
115
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
The bcm2835-v3d is used since Linux 4.7, see commit
4
49ac67e0c39c ("ARM: bcm2835: Add VC4 to the device tree"),
5
and the bcm2835-txp since Linux 4.19, see commit
6
b7dd29b401f5 ("ARM: dts: bcm283x: Add Transposer block").
7
8
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
9
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
10
Message-id: 20200921034729.432931-3-f4bug@amsat.org
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
include/hw/arm/bcm2835_peripherals.h | 2 ++
14
hw/arm/bcm2835_peripherals.c | 2 ++
15
2 files changed, 4 insertions(+)
16
17
diff --git a/include/hw/arm/bcm2835_peripherals.h b/include/hw/arm/bcm2835_peripherals.h
18
index XXXXXXX..XXXXXXX 100644
19
--- a/include/hw/arm/bcm2835_peripherals.h
20
+++ b/include/hw/arm/bcm2835_peripherals.h
21
@@ -XXX,XX +XXX,XX @@ struct BCM2835PeripheralState {
22
23
BCM2835SystemTimerState systmr;
24
BCM2835MphiState mphi;
25
+ UnimplementedDeviceState txp;
26
UnimplementedDeviceState armtmr;
27
UnimplementedDeviceState cprman;
28
UnimplementedDeviceState a2w;
29
@@ -XXX,XX +XXX,XX @@ struct BCM2835PeripheralState {
30
UnimplementedDeviceState otp;
31
UnimplementedDeviceState dbus;
32
UnimplementedDeviceState ave0;
33
+ UnimplementedDeviceState v3d;
34
UnimplementedDeviceState bscsl;
35
UnimplementedDeviceState smi;
36
DWC2State dwc2;
37
diff --git a/hw/arm/bcm2835_peripherals.c b/hw/arm/bcm2835_peripherals.c
38
index XXXXXXX..XXXXXXX 100644
39
--- a/hw/arm/bcm2835_peripherals.c
40
+++ b/hw/arm/bcm2835_peripherals.c
41
@@ -XXX,XX +XXX,XX @@ static void bcm2835_peripherals_realize(DeviceState *dev, Error **errp)
42
qdev_get_gpio_in_named(DEVICE(&s->ic), BCM2835_IC_GPU_IRQ,
43
INTERRUPT_USB));
44
45
+ create_unimp(s, &s->txp, "bcm2835-txp", TXP_OFFSET, 0x1000);
46
create_unimp(s, &s->armtmr, "bcm2835-sp804", ARMCTRL_TIMER0_1_OFFSET, 0x40);
47
create_unimp(s, &s->cprman, "bcm2835-cprman", CPRMAN_OFFSET, 0x1000);
48
create_unimp(s, &s->a2w, "bcm2835-a2w", A2W_OFFSET, 0x1000);
49
@@ -XXX,XX +XXX,XX @@ static void bcm2835_peripherals_realize(DeviceState *dev, Error **errp)
50
create_unimp(s, &s->otp, "bcm2835-otp", OTP_OFFSET, 0x80);
51
create_unimp(s, &s->dbus, "bcm2835-dbus", DBUS_OFFSET, 0x8000);
52
create_unimp(s, &s->ave0, "bcm2835-ave0", AVE0_OFFSET, 0x8000);
53
+ create_unimp(s, &s->v3d, "bcm2835-v3d", V3D_OFFSET, 0x1000);
54
create_unimp(s, &s->sdramc, "bcm2835-sdramc", SDRAMC_OFFSET, 0x100);
55
}
56
57
--
58
2.20.1
59
60
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
Commit 1c3db49d39 added the raspi3, which uses the same peripherals
4
than the raspi2 (but with different ARM cores). The raspi3 was
5
introduced without the ignore_memory_transaction_failures flag.
6
Almost 2 years later, the machine is usable running U-Boot and
7
Linux.
8
In commit 00cbd5bd74 we mapped a lot of unimplemented devices,
9
commit d442d95f added thermal block and commit 0e5bbd7406 the
10
system timer.
11
As we are happy with the raspi3, let's remove this flag on the
12
raspi2.
13
14
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
15
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
16
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
17
Message-id: 20200921034729.432931-4-f4bug@amsat.org
18
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
19
---
20
hw/arm/raspi.c | 3 ---
21
1 file changed, 3 deletions(-)
22
23
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
24
index XXXXXXX..XXXXXXX 100644
25
--- a/hw/arm/raspi.c
26
+++ b/hw/arm/raspi.c
27
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_class_init(ObjectClass *oc, void *data)
28
mc->default_cpus = mc->min_cpus = mc->max_cpus = cores_count(board_rev);
29
mc->default_ram_size = board_ram_size(board_rev);
30
mc->default_ram_id = "ram";
31
- if (board_version(board_rev) == 2) {
32
- mc->ignore_memory_transaction_failures = true;
33
- }
34
};
35
36
static const TypeInfo raspi_machine_types[] = {
37
--
38
2.20.1
39
40
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
Display the board revision in the machine description.
4
5
Before:
6
7
$ qemu-system-aarch64 -M help | fgrep raspi
8
raspi2 Raspberry Pi 2B
9
raspi3 Raspberry Pi 3B
10
11
After:
12
13
raspi2 Raspberry Pi 2B (revision 1.1)
14
raspi3 Raspberry Pi 3B (revision 1.2)
15
16
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
17
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
18
Message-id: 20200924111808.77168-2-f4bug@amsat.org
19
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
20
---
21
hw/arm/raspi.c | 4 +++-
22
1 file changed, 3 insertions(+), 1 deletion(-)
23
24
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
25
index XXXXXXX..XXXXXXX 100644
26
--- a/hw/arm/raspi.c
27
+++ b/hw/arm/raspi.c
28
@@ -XXX,XX +XXX,XX @@ static void raspi_machine_class_init(ObjectClass *oc, void *data)
29
uint32_t board_rev = (uint32_t)(uintptr_t)data;
30
31
rmc->board_rev = board_rev;
32
- mc->desc = g_strdup_printf("Raspberry Pi %s", board_type(board_rev));
33
+ mc->desc = g_strdup_printf("Raspberry Pi %s (revision 1.%u)",
34
+ board_type(board_rev),
35
+ FIELD_EX32(board_rev, REV_CODE, REVISION));
36
mc->init = raspi_machine_init;
37
mc->block_default_type = IF_SD;
38
mc->no_parallel = 1;
39
--
40
2.20.1
41
42
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
The 'first_cpu' is more a QEMU accelerator-related concept
4
than a variable the machine requires to use.
5
Since the machine is aware of its CPUs, directly use the
6
first one to load the firmware.
7
8
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
9
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
10
Message-id: 20200924111808.77168-3-f4bug@amsat.org
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
hw/arm/raspi.c | 3 ++-
14
1 file changed, 2 insertions(+), 1 deletion(-)
15
16
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
17
index XXXXXXX..XXXXXXX 100644
18
--- a/hw/arm/raspi.c
19
+++ b/hw/arm/raspi.c
20
@@ -XXX,XX +XXX,XX @@ static void reset_secondary(ARMCPU *cpu, const struct arm_boot_info *info)
21
22
static void setup_boot(MachineState *machine, int version, size_t ram_size)
23
{
24
+ RaspiMachineState *s = RASPI_MACHINE(machine);
25
static struct arm_boot_info binfo;
26
int r;
27
28
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
29
binfo.firmware_loaded = true;
30
}
31
32
- arm_load_kernel(ARM_CPU(first_cpu), machine, &binfo);
33
+ arm_load_kernel(&s->soc.cpu[0].core, machine, &binfo);
34
}
35
36
static void raspi_machine_init(MachineState *machine)
37
--
38
2.20.1
39
40
diff view generated by jsdifflib
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
1
In commit f0a08b0913befbd we changed the type of the PC from
2
target_ulong to vaddr. In doing so we inadvertently dropped the
3
zero-padding on the PC in trace lines (the second item inside the []
4
in these lines). They used to look like this on AArch64, for
5
instance:
2
6
3
The firmware load address depends on the SoC ("processor id") used,
7
Trace 0: 0x7f2260000100 [00000000/0000000040000000/00000061/ff200000]
4
not on the version of the board.
5
8
6
Suggested-by: Luc Michel <luc.michel@greensocs.com>
9
and now they look like this:
7
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
10
Trace 0: 0x7f4f50000100 [00000000/40000000/00000061/ff200000]
8
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
11
9
Message-id: 20200924111808.77168-8-f4bug@amsat.org
12
and if the PC happens to be somewhere low like 0x5000
13
then the field is shown as /5000/.
14
15
This is because TARGET_FMT_lx is a "%08x" or "%016x" specifier,
16
depending on TARGET_LONG_SIZE, whereas VADDR_PRIx is just PRIx64
17
with no width specifier.
18
19
Restore the zero-padding by adding an 016 width specifier to
20
this tracing and a couple of others that were similarly recently
21
changed to use VADDR_PRIx without a width specifier.
22
23
We can't unfortunately restore the "32-bit guests are padded to
24
8 hex digits and 64-bit guests to 16 hex digits" behaviour so
25
easily.
26
27
Fixes: f0a08b0913befbd ("accel/tcg/cpu-exec.c: Widen pc to vaddr")
10
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
28
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
29
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
30
Reviewed-by: Anton Johansson <anjo@rev.ng>
31
Message-id: 20230711165434.4123674-1-peter.maydell@linaro.org
11
---
32
---
12
hw/arm/raspi.c | 3 ++-
33
accel/tcg/cpu-exec.c | 4 ++--
13
1 file changed, 2 insertions(+), 1 deletion(-)
34
accel/tcg/translate-all.c | 2 +-
35
2 files changed, 3 insertions(+), 3 deletions(-)
14
36
15
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
37
diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c
16
index XXXXXXX..XXXXXXX 100644
38
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/arm/raspi.c
39
--- a/accel/tcg/cpu-exec.c
18
+++ b/hw/arm/raspi.c
40
+++ b/accel/tcg/cpu-exec.c
19
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
41
@@ -XXX,XX +XXX,XX @@ static void log_cpu_exec(vaddr pc, CPUState *cpu,
20
* the normal Linux boot process
42
if (qemu_log_in_addr_range(pc)) {
21
*/
43
qemu_log_mask(CPU_LOG_EXEC,
22
if (machine->firmware) {
44
"Trace %d: %p [%08" PRIx64
23
- hwaddr firmware_addr = version == 3 ? FIRMWARE_ADDR_3 : FIRMWARE_ADDR_2;
45
- "/%" VADDR_PRIx "/%08x/%08x] %s\n",
24
+ hwaddr firmware_addr = processor_id <= PROCESSOR_ID_BCM2836
46
+ "/%016" VADDR_PRIx "/%08x/%08x] %s\n",
25
+ ? FIRMWARE_ADDR_2 : FIRMWARE_ADDR_3;
47
cpu->cpu_index, tb->tc.ptr, tb->cs_base, pc,
26
/* load the firmware image (typically kernel.img) */
48
tb->flags, tb->cflags, lookup_symbol(pc));
27
r = load_image_targphys(machine->firmware, firmware_addr,
49
28
ram_size - firmware_addr);
50
@@ -XXX,XX +XXX,XX @@ cpu_tb_exec(CPUState *cpu, TranslationBlock *itb, int *tb_exit)
51
if (qemu_loglevel_mask(CPU_LOG_EXEC)) {
52
vaddr pc = log_pc(cpu, last_tb);
53
if (qemu_log_in_addr_range(pc)) {
54
- qemu_log("Stopped execution of TB chain before %p [%"
55
+ qemu_log("Stopped execution of TB chain before %p [%016"
56
VADDR_PRIx "] %s\n",
57
last_tb->tc.ptr, pc, lookup_symbol(pc));
58
}
59
diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c
60
index XXXXXXX..XXXXXXX 100644
61
--- a/accel/tcg/translate-all.c
62
+++ b/accel/tcg/translate-all.c
63
@@ -XXX,XX +XXX,XX @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr)
64
if (qemu_loglevel_mask(CPU_LOG_EXEC)) {
65
vaddr pc = log_pc(cpu, tb);
66
if (qemu_log_in_addr_range(pc)) {
67
- qemu_log("cpu_io_recompile: rewound execution of TB to %"
68
+ qemu_log("cpu_io_recompile: rewound execution of TB to %016"
69
VADDR_PRIx "\n", pc);
70
}
71
}
29
--
72
--
30
2.20.1
73
2.34.1
31
74
32
75
diff view generated by jsdifflib
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
1
From: Tong Ho <tong.ho@amd.com>
2
2
3
The arm_boot_info structure belong to the machine,
3
Add a check in the bit-set operation to write the backstore
4
move it to RaspiMachineState.
4
only if the affected bit is 0 before.
5
5
6
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
6
With this in place, there will be no need for callers to
7
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
7
do the checking in order to avoid unnecessary writes.
8
Message-id: 20200924111808.77168-4-f4bug@amsat.org
8
9
Signed-off-by: Tong Ho <tong.ho@amd.com>
10
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
11
Reviewed-by: Francisco Iglesias <frasse.iglesias@gmail.com>
12
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
9
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
13
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
10
---
14
---
11
hw/arm/raspi.c | 30 +++++++++++++++---------------
15
hw/nvram/xlnx-efuse.c | 11 +++++++++--
12
1 file changed, 15 insertions(+), 15 deletions(-)
16
1 file changed, 9 insertions(+), 2 deletions(-)
13
17
14
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
18
diff --git a/hw/nvram/xlnx-efuse.c b/hw/nvram/xlnx-efuse.c
15
index XXXXXXX..XXXXXXX 100644
19
index XXXXXXX..XXXXXXX 100644
16
--- a/hw/arm/raspi.c
20
--- a/hw/nvram/xlnx-efuse.c
17
+++ b/hw/arm/raspi.c
21
+++ b/hw/nvram/xlnx-efuse.c
18
@@ -XXX,XX +XXX,XX @@ struct RaspiMachineState {
22
@@ -XXX,XX +XXX,XX @@ static bool efuse_ro_bits_find(XlnxEFuse *s, uint32_t k)
19
MachineState parent_obj;
23
20
/*< public >*/
24
bool xlnx_efuse_set_bit(XlnxEFuse *s, unsigned int bit)
21
BCM283XState soc;
22
+ struct arm_boot_info binfo;
23
};
24
typedef struct RaspiMachineState RaspiMachineState;
25
26
@@ -XXX,XX +XXX,XX @@ static void reset_secondary(ARMCPU *cpu, const struct arm_boot_info *info)
27
static void setup_boot(MachineState *machine, int version, size_t ram_size)
28
{
25
{
29
RaspiMachineState *s = RASPI_MACHINE(machine);
26
+ uint32_t set, *row;
30
- static struct arm_boot_info binfo;
27
+
31
int r;
28
if (efuse_ro_bits_find(s, bit)) {
32
29
g_autofree char *path = object_get_canonical_path(OBJECT(s));
33
- binfo.board_id = MACH_TYPE_BCM2708;
30
34
- binfo.ram_size = ram_size;
31
@@ -XXX,XX +XXX,XX @@ bool xlnx_efuse_set_bit(XlnxEFuse *s, unsigned int bit)
35
- binfo.nb_cpus = machine->smp.cpus;
32
return false;
36
+ s->binfo.board_id = MACH_TYPE_BCM2708;
37
+ s->binfo.ram_size = ram_size;
38
+ s->binfo.nb_cpus = machine->smp.cpus;
39
40
if (version <= 2) {
41
/* The rpi1 and 2 require some custom setup code to run in Secure
42
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
43
* firmware for some cache maintenance operations.
44
* The rpi3 doesn't need this.
45
*/
46
- binfo.board_setup_addr = BOARDSETUP_ADDR;
47
- binfo.write_board_setup = write_board_setup;
48
- binfo.secure_board_setup = true;
49
- binfo.secure_boot = true;
50
+ s->binfo.board_setup_addr = BOARDSETUP_ADDR;
51
+ s->binfo.write_board_setup = write_board_setup;
52
+ s->binfo.secure_board_setup = true;
53
+ s->binfo.secure_boot = true;
54
}
33
}
55
34
56
/* Pi2 and Pi3 requires SMP setup */
35
- s->fuse32[bit / 32] |= 1 << (bit % 32);
57
if (version >= 2) {
36
- efuse_bdrv_sync(s, bit);
58
- binfo.smp_loader_start = SMPBOOT_ADDR;
37
+ /* Avoid back-end write unless there is a real update */
59
+ s->binfo.smp_loader_start = SMPBOOT_ADDR;
38
+ row = &s->fuse32[bit / 32];
60
if (version == 2) {
39
+ set = 1 << (bit % 32);
61
- binfo.write_secondary_boot = write_smpboot;
40
+ if (!(set & *row)) {
62
+ s->binfo.write_secondary_boot = write_smpboot;
41
+ *row |= set;
63
} else {
42
+ efuse_bdrv_sync(s, bit);
64
- binfo.write_secondary_boot = write_smpboot64;
43
+ }
65
+ s->binfo.write_secondary_boot = write_smpboot64;
44
return true;
66
}
67
- binfo.secondary_cpu_reset_hook = reset_secondary;
68
+ s->binfo.secondary_cpu_reset_hook = reset_secondary;
69
}
70
71
/* If the user specified a "firmware" image (e.g. UEFI), we bypass
72
@@ -XXX,XX +XXX,XX @@ static void setup_boot(MachineState *machine, int version, size_t ram_size)
73
exit(1);
74
}
75
76
- binfo.entry = firmware_addr;
77
- binfo.firmware_loaded = true;
78
+ s->binfo.entry = firmware_addr;
79
+ s->binfo.firmware_loaded = true;
80
}
81
82
- arm_load_kernel(&s->soc.cpu[0].core, machine, &binfo);
83
+ arm_load_kernel(&s->soc.cpu[0].core, machine, &s->binfo);
84
}
45
}
85
46
86
static void raspi_machine_init(MachineState *machine)
87
--
47
--
88
2.20.1
48
2.34.1
89
49
90
50
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
Now that we can instantiate different machines based on their
4
board_rev register value, we can have various raspi2 and raspi3.
5
6
In commit fc78a990ec103 we corrected the machine description.
7
Correct the machine names too. For backward compatibility, add
8
an alias to the previous generic name.
9
10
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
11
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
12
Message-id: 20200924111808.77168-6-f4bug@amsat.org
13
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
14
---
15
hw/arm/raspi.c | 6 ++++--
16
1 file changed, 4 insertions(+), 2 deletions(-)
17
18
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
19
index XXXXXXX..XXXXXXX 100644
20
--- a/hw/arm/raspi.c
21
+++ b/hw/arm/raspi.c
22
@@ -XXX,XX +XXX,XX @@ static void raspi2b_machine_class_init(ObjectClass *oc, void *data)
23
MachineClass *mc = MACHINE_CLASS(oc);
24
RaspiMachineClass *rmc = RASPI_MACHINE_CLASS(oc);
25
26
+ mc->alias = "raspi2";
27
rmc->board_rev = 0xa21041;
28
raspi_machine_class_common_init(mc, rmc->board_rev);
29
};
30
@@ -XXX,XX +XXX,XX @@ static void raspi3b_machine_class_init(ObjectClass *oc, void *data)
31
MachineClass *mc = MACHINE_CLASS(oc);
32
RaspiMachineClass *rmc = RASPI_MACHINE_CLASS(oc);
33
34
+ mc->alias = "raspi3";
35
rmc->board_rev = 0xa02082;
36
raspi_machine_class_common_init(mc, rmc->board_rev);
37
};
38
@@ -XXX,XX +XXX,XX @@ static void raspi3b_machine_class_init(ObjectClass *oc, void *data)
39
40
static const TypeInfo raspi_machine_types[] = {
41
{
42
- .name = MACHINE_TYPE_NAME("raspi2"),
43
+ .name = MACHINE_TYPE_NAME("raspi2b"),
44
.parent = TYPE_RASPI_MACHINE,
45
.class_init = raspi2b_machine_class_init,
46
#ifdef TARGET_AARCH64
47
}, {
48
- .name = MACHINE_TYPE_NAME("raspi3"),
49
+ .name = MACHINE_TYPE_NAME("raspi3b"),
50
.parent = TYPE_RASPI_MACHINE,
51
.class_init = raspi3b_machine_class_init,
52
#endif
53
--
54
2.20.1
55
56
diff view generated by jsdifflib
Deleted patch
1
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
2
1
3
As we only support a reduced set of the REV_CODE_PROCESSOR id
4
encoded in the board revision, define the PROCESSOR_ID values
5
as an enum. We can simplify the board_soc_type and cores_count
6
methods.
7
8
Reviewed-by: Luc Michel <luc.michel@greensocs.com>
9
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
10
Message-id: 20200924111808.77168-7-f4bug@amsat.org
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
hw/arm/raspi.c | 45 +++++++++++++++++++++------------------------
14
1 file changed, 21 insertions(+), 24 deletions(-)
15
16
diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c
17
index XXXXXXX..XXXXXXX 100644
18
--- a/hw/arm/raspi.c
19
+++ b/hw/arm/raspi.c
20
@@ -XXX,XX +XXX,XX @@ FIELD(REV_CODE, MANUFACTURER, 16, 4);
21
FIELD(REV_CODE, MEMORY_SIZE, 20, 3);
22
FIELD(REV_CODE, STYLE, 23, 1);
23
24
+typedef enum RaspiProcessorId {
25
+ PROCESSOR_ID_BCM2836 = 1,
26
+ PROCESSOR_ID_BCM2837 = 2,
27
+} RaspiProcessorId;
28
+
29
+static const struct {
30
+ const char *type;
31
+ int cores_count;
32
+} soc_property[] = {
33
+ [PROCESSOR_ID_BCM2836] = {TYPE_BCM2836, BCM283X_NCPUS},
34
+ [PROCESSOR_ID_BCM2837] = {TYPE_BCM2837, BCM283X_NCPUS},
35
+};
36
+
37
static uint64_t board_ram_size(uint32_t board_rev)
38
{
39
assert(FIELD_EX32(board_rev, REV_CODE, STYLE)); /* Only new style */
40
return 256 * MiB << FIELD_EX32(board_rev, REV_CODE, MEMORY_SIZE);
41
}
42
43
-static int board_processor_id(uint32_t board_rev)
44
+static RaspiProcessorId board_processor_id(uint32_t board_rev)
45
{
46
+ int proc_id = FIELD_EX32(board_rev, REV_CODE, PROCESSOR);
47
+
48
assert(FIELD_EX32(board_rev, REV_CODE, STYLE)); /* Only new style */
49
- return FIELD_EX32(board_rev, REV_CODE, PROCESSOR);
50
+ assert(proc_id < ARRAY_SIZE(soc_property) && soc_property[proc_id].type);
51
+
52
+ return proc_id;
53
}
54
55
static int board_version(uint32_t board_rev)
56
@@ -XXX,XX +XXX,XX @@ static int board_version(uint32_t board_rev)
57
58
static const char *board_soc_type(uint32_t board_rev)
59
{
60
- static const char *soc_types[] = {
61
- NULL, TYPE_BCM2836, TYPE_BCM2837,
62
- };
63
- int proc_id = board_processor_id(board_rev);
64
-
65
- if (proc_id >= ARRAY_SIZE(soc_types) || !soc_types[proc_id]) {
66
- error_report("Unsupported processor id '%d' (board revision: 0x%x)",
67
- proc_id, board_rev);
68
- exit(1);
69
- }
70
- return soc_types[proc_id];
71
+ return soc_property[board_processor_id(board_rev)].type;
72
}
73
74
static int cores_count(uint32_t board_rev)
75
{
76
- static const int soc_cores_count[] = {
77
- 0, BCM283X_NCPUS, BCM283X_NCPUS,
78
- };
79
- int proc_id = board_processor_id(board_rev);
80
-
81
- if (proc_id >= ARRAY_SIZE(soc_cores_count) || !soc_cores_count[proc_id]) {
82
- error_report("Unsupported processor id '%d' (board revision: 0x%x)",
83
- proc_id, board_rev);
84
- exit(1);
85
- }
86
- return soc_cores_count[proc_id];
87
+ return soc_property[board_processor_id(board_rev)].cores_count;
88
}
89
90
static const char *board_type(uint32_t board_rev)
91
--
92
2.20.1
93
94
diff view generated by jsdifflib