1
A last arm pullreq before rc0. This is mostly bug fixes,
1
Squashed in a trivial fix for 32-bit hosts:
2
though you could call adding the missing local timer
3
support to bcm2836_control a new feature I suppose --
4
in any case it's a small and localised change.
5
2
6
thanks
3
--- a/target/arm/mve_helper.c
4
+++ b/target/arm/mve_helper.c
5
@@ -XXX,XX +XXX,XX @@ DO_LDAV(vmlsldavxsw, 4, int32_t, true, +=, -=)
6
acc = EVENACC(acc, TO128(n[H##ESIZE(e + 1 * XCHG)] * \
7
m[H##ESIZE(e)])); \
8
} \
9
- acc = int128_add(acc, 1 << 7); \
10
+ acc = int128_add(acc, int128_make64(1 << 7)); \
11
} \
12
} \
13
mve_advance_vpt(env); \
14
7
-- PMM
15
-- PMM
8
16
9
The following changes since commit 7074ab12c81a1b2b1e0e1c40983f56b2c5ccc494:
17
The following changes since commit 53f306f316549d20c76886903181413d20842423:
10
18
11
Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-03-14 16:19:37 +0000)
19
Merge remote-tracking branch 'remotes/ehabkost-gl/tags/x86-next-pull-request' into staging (2021-06-21 11:26:04 +0100)
12
20
13
are available in the Git repository at:
21
are available in the Git repository at:
14
22
15
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20190315
23
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20210624
16
24
17
for you to fetch changes up to 5de56742a3c91de3d646326bec43a989bba83ca4:
25
for you to fetch changes up to 90a76c6316cfe6416fc33814a838fb3928f746ee:
18
26
19
target/arm: Check access permission to ADDVL/ADDPL/RDVL (2019-03-15 11:12:29 +0000)
27
docs/system: arm: Add nRF boards description (2021-06-24 14:58:48 +0100)
20
28
21
----------------------------------------------------------------
29
----------------------------------------------------------------
22
target-arm queue:
30
target-arm queue:
23
* Add missing SVE-enabled check to ADDVL/ADDPL/RDVL
31
* Don't require 'virt' board to be compiled in for ACPI GHES code
24
* virt-acpi-build: use PCIE_MMCFG_BUS to retrieve end_bus_number
32
* docs: Document which architecture extensions we emulate
25
* virt-acpi-build: Fix SMMUv3 GSIV values
33
* Fix bugs in M-profile FPCXT_NS accesses
26
* Allow EL0 to write to arch timer registers, not just read them
34
* First slice of MVE patches
27
* bcm2836_control: Implement local timer
35
* Implement MTE3
36
* docs/system: arm: Add nRF boards description
28
37
29
----------------------------------------------------------------
38
----------------------------------------------------------------
30
Amir Charif (1):
39
Alexandre Iooss (1):
31
target/arm: Check access permission to ADDVL/ADDPL/RDVL
40
docs/system: arm: Add nRF boards description
32
41
33
Dongjiu Geng (1):
42
Peter Collingbourne (1):
34
target/arm: change arch timer registers access permission
43
target/arm: Implement MTE3
35
44
36
Eric Auger (1):
45
Peter Maydell (55):
37
hw/arm/virt-acpi-build: Fix SMMUv3 GSIV values
46
hw/acpi: Provide stub version of acpi_ghes_record_errors()
47
hw/acpi: Provide function acpi_ghes_present()
48
target/arm: Use acpi_ghes_present() to see if we report ACPI memory errors
49
docs/system/arm: Document which architecture extensions we emulate
50
target/arm/translate-vfp.c: Whitespace fixes
51
target/arm: Handle FPU being disabled in FPCXT_NS accesses
52
target/arm: Don't NOCP fault for FPCXT_NS accesses
53
target/arm: Handle writeback in VLDR/VSTR sysreg with no memory access
54
target/arm: Factor FP context update code out into helper function
55
target/arm: Split vfp_access_check() into A and M versions
56
target/arm: Handle FPU check for FPCXT_NS insns via vfp_access_check_m()
57
target/arm: Implement MVE VLDR/VSTR (non-widening forms)
58
target/arm: Implement widening/narrowing MVE VLDR/VSTR insns
59
target/arm: Implement MVE VCLZ
60
target/arm: Implement MVE VCLS
61
target/arm: Implement MVE VREV16, VREV32, VREV64
62
target/arm: Implement MVE VMVN (register)
63
target/arm: Implement MVE VABS
64
target/arm: Implement MVE VNEG
65
tcg: Make gen_dup_i32/i64() public as tcg_gen_dup_i32/i64
66
target/arm: Implement MVE VDUP
67
target/arm: Implement MVE VAND, VBIC, VORR, VORN, VEOR
68
target/arm: Implement MVE VADD, VSUB, VMUL
69
target/arm: Implement MVE VMULH
70
target/arm: Implement MVE VRMULH
71
target/arm: Implement MVE VMAX, VMIN
72
target/arm: Implement MVE VABD
73
target/arm: Implement MVE VHADD, VHSUB
74
target/arm: Implement MVE VMULL
75
target/arm: Implement MVE VMLALDAV
76
target/arm: Implement MVE VMLSLDAV
77
target/arm: Implement MVE VRMLALDAVH, VRMLSLDAVH
78
target/arm: Implement MVE VADD (scalar)
79
target/arm: Implement MVE VSUB, VMUL (scalar)
80
target/arm: Implement MVE VHADD, VHSUB (scalar)
81
target/arm: Implement MVE VBRSR
82
target/arm: Implement MVE VPST
83
target/arm: Implement MVE VQADD and VQSUB
84
target/arm: Implement MVE VQDMULH and VQRDMULH (scalar)
85
target/arm: Implement MVE VQDMULL scalar
86
target/arm: Implement MVE VQDMULH, VQRDMULH (vector)
87
target/arm: Implement MVE VQADD, VQSUB (vector)
88
target/arm: Implement MVE VQSHL (vector)
89
target/arm: Implement MVE VQRSHL
90
target/arm: Implement MVE VSHL insn
91
target/arm: Implement MVE VRSHL
92
target/arm: Implement MVE VQDMLADH and VQRDMLADH
93
target/arm: Implement MVE VQDMLSDH and VQRDMLSDH
94
target/arm: Implement MVE VQDMULL (vector)
95
target/arm: Implement MVE VRHADD
96
target/arm: Implement MVE VADC, VSBC
97
target/arm: Implement MVE VCADD
98
target/arm: Implement MVE VHCADD
99
target/arm: Implement MVE VADDV
100
target/arm: Make VMOV scalar <-> gpreg beatwise for MVE
38
101
39
Wei Yang (1):
102
docs/system/arm/emulation.rst | 103 ++++
40
hw/arm/virt-acpi-build: use PCIE_MMCFG_BUS to retrieve end_bus_number
103
docs/system/arm/nrf.rst | 51 ++
104
docs/system/target-arm.rst | 7 +
105
include/hw/acpi/ghes.h | 9 +
106
include/tcg/tcg-op.h | 8 +
107
include/tcg/tcg.h | 1 -
108
target/arm/helper-mve.h | 357 +++++++++++++
109
target/arm/helper.h | 2 +
110
target/arm/internals.h | 11 +
111
target/arm/translate-a32.h | 3 +
112
target/arm/translate.h | 10 +
113
target/arm/m-nocp.decode | 24 +
114
target/arm/mve.decode | 240 +++++++++
115
target/arm/vfp.decode | 14 -
116
hw/acpi/ghes-stub.c | 22 +
117
hw/acpi/ghes.c | 17 +
118
target/arm/cpu64.c | 2 +-
119
target/arm/kvm64.c | 6 +-
120
target/arm/mte_helper.c | 82 +--
121
target/arm/mve_helper.c | 1160 +++++++++++++++++++++++++++++++++++++++++
122
target/arm/translate-m-nocp.c | 550 +++++++++++++++++++
123
target/arm/translate-mve.c | 759 +++++++++++++++++++++++++++
124
target/arm/translate-vfp.c | 741 +++++++-------------------
125
tcg/tcg-op-gvec.c | 20 +-
126
MAINTAINERS | 1 +
127
hw/acpi/meson.build | 6 +-
128
target/arm/meson.build | 1 +
129
27 files changed, 3578 insertions(+), 629 deletions(-)
130
create mode 100644 docs/system/arm/emulation.rst
131
create mode 100644 docs/system/arm/nrf.rst
132
create mode 100644 target/arm/helper-mve.h
133
create mode 100644 hw/acpi/ghes-stub.c
134
create mode 100644 target/arm/mve_helper.c
41
135
42
Zoltán Baldaszti (1):
43
hw/intc/bcm2836_control: Implement local timer
44
45
include/hw/intc/bcm2836_control.h | 9 ++++
46
hw/arm/virt-acpi-build.c | 6 +--
47
hw/intc/bcm2836_control.c | 101 +++++++++++++++++++++++++++++++++++++-
48
target/arm/helper.c | 30 +++++------
49
target/arm/translate-sve.c | 22 ++++++---
50
5 files changed, 140 insertions(+), 28 deletions(-)
51
diff view generated by jsdifflib
Deleted patch
1
From: Zoltán Baldaszti <bztemail@gmail.com>
2
1
3
The BCM2836 control logic module includes a simple
4
"local timer" which is a programmable down-counter that
5
can generates an interrupt. Implement this functionality.
6
7
Signed-off-by: Zoltán Baldaszti <bztemail@gmail.com>
8
[PMM: wrote commit message; wrapped long line; tweaked
9
some comments to match the final version of the code]
10
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
include/hw/intc/bcm2836_control.h | 9 +++
14
hw/intc/bcm2836_control.c | 101 +++++++++++++++++++++++++++++-
15
2 files changed, 108 insertions(+), 2 deletions(-)
16
17
diff --git a/include/hw/intc/bcm2836_control.h b/include/hw/intc/bcm2836_control.h
18
index XXXXXXX..XXXXXXX 100644
19
--- a/include/hw/intc/bcm2836_control.h
20
+++ b/include/hw/intc/bcm2836_control.h
21
@@ -XXX,XX +XXX,XX @@
22
* Rasperry Pi 2 emulation and refactoring Copyright (c) 2015, Microsoft
23
* Written by Andrew Baumann
24
*
25
+ * ARM Local Timer IRQ Copyright (c) 2019. Zoltán Baldaszti
26
+ * Added basic IRQ_TIMER interrupt support
27
+ *
28
* This code is licensed under the GNU GPLv2 and later.
29
*/
30
31
@@ -XXX,XX +XXX,XX @@
32
#define BCM2836_CONTROL_H
33
34
#include "hw/sysbus.h"
35
+#include "qemu/timer.h"
36
37
/* 4 mailboxes per core, for 16 total */
38
#define BCM2836_NCORES 4
39
@@ -XXX,XX +XXX,XX @@ typedef struct BCM2836ControlState {
40
bool gpu_irq, gpu_fiq;
41
uint8_t timerirqs[BCM2836_NCORES];
42
43
+ /* local timer */
44
+ QEMUTimer timer;
45
+ uint32_t local_timer_control;
46
+ uint8_t route_localtimer;
47
+
48
/* interrupt source registers, post-routing (also input-derived; visible) */
49
uint32_t irqsrc[BCM2836_NCORES];
50
uint32_t fiqsrc[BCM2836_NCORES];
51
diff --git a/hw/intc/bcm2836_control.c b/hw/intc/bcm2836_control.c
52
index XXXXXXX..XXXXXXX 100644
53
--- a/hw/intc/bcm2836_control.c
54
+++ b/hw/intc/bcm2836_control.c
55
@@ -XXX,XX +XXX,XX @@
56
* This code is licensed under the GNU GPLv2 and later.
57
*
58
* At present, only implements interrupt routing, and mailboxes (i.e.,
59
- * not local timer, PMU interrupt, or AXI counters).
60
+ * not PMU interrupt, or AXI counters).
61
+ *
62
+ * ARM Local Timer IRQ Copyright (c) 2019. Zoltán Baldaszti
63
*
64
* Ref:
65
* https://www.raspberrypi.org/documentation/hardware/raspberrypi/bcm2836/QA7_rev3.4.pdf
66
@@ -XXX,XX +XXX,XX @@
67
#include "qemu/log.h"
68
69
#define REG_GPU_ROUTE 0x0c
70
+#define REG_LOCALTIMERROUTING 0x24
71
+#define REG_LOCALTIMERCONTROL 0x34
72
+#define REG_LOCALTIMERACK 0x38
73
#define REG_TIMERCONTROL 0x40
74
#define REG_MBOXCONTROL 0x50
75
#define REG_IRQSRC 0x60
76
@@ -XXX,XX +XXX,XX @@
77
#define IRQ_TIMER 11
78
#define IRQ_MAX IRQ_TIMER
79
80
+#define LOCALTIMER_FREQ 38400000
81
+#define LOCALTIMER_INTFLAG (1 << 31)
82
+#define LOCALTIMER_RELOAD (1 << 30)
83
+#define LOCALTIMER_INTENABLE (1 << 29)
84
+#define LOCALTIMER_ENABLE (1 << 28)
85
+#define LOCALTIMER_VALUE(x) ((x) & 0xfffffff)
86
+
87
static void deliver_local(BCM2836ControlState *s, uint8_t core, uint8_t irq,
88
uint32_t controlreg, uint8_t controlidx)
89
{
90
@@ -XXX,XX +XXX,XX @@ static void bcm2836_control_update(BCM2836ControlState *s)
91
s->fiqsrc[s->route_gpu_fiq] |= (uint32_t)1 << IRQ_GPU;
92
}
93
94
+ /*
95
+ * handle the control module 'local timer' interrupt for one of the
96
+ * cores' IRQ/FIQ; this is distinct from the per-CPU timer
97
+ * interrupts handled below.
98
+ */
99
+ if ((s->local_timer_control & LOCALTIMER_INTENABLE) &&
100
+ (s->local_timer_control & LOCALTIMER_INTFLAG)) {
101
+ if (s->route_localtimer & 4) {
102
+ s->fiqsrc[(s->route_localtimer & 3)] |= (uint32_t)1 << IRQ_TIMER;
103
+ } else {
104
+ s->irqsrc[(s->route_localtimer & 3)] |= (uint32_t)1 << IRQ_TIMER;
105
+ }
106
+ }
107
+
108
for (i = 0; i < BCM2836_NCORES; i++) {
109
/* handle local timer interrupts for this core */
110
if (s->timerirqs[i]) {
111
@@ -XXX,XX +XXX,XX @@ static void bcm2836_control_set_gpu_fiq(void *opaque, int irq, int level)
112
bcm2836_control_update(s);
113
}
114
115
+static void bcm2836_control_local_timer_set_next(void *opaque)
116
+{
117
+ BCM2836ControlState *s = opaque;
118
+ uint64_t next_event;
119
+
120
+ assert(LOCALTIMER_VALUE(s->local_timer_control) > 0);
121
+
122
+ next_event = qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) +
123
+ muldiv64(LOCALTIMER_VALUE(s->local_timer_control),
124
+ NANOSECONDS_PER_SECOND, LOCALTIMER_FREQ);
125
+ timer_mod(&s->timer, next_event);
126
+}
127
+
128
+static void bcm2836_control_local_timer_tick(void *opaque)
129
+{
130
+ BCM2836ControlState *s = opaque;
131
+
132
+ bcm2836_control_local_timer_set_next(s);
133
+
134
+ s->local_timer_control |= LOCALTIMER_INTFLAG;
135
+ bcm2836_control_update(s);
136
+}
137
+
138
+static void bcm2836_control_local_timer_control(void *opaque, uint32_t val)
139
+{
140
+ BCM2836ControlState *s = opaque;
141
+
142
+ s->local_timer_control = val;
143
+ if (val & LOCALTIMER_ENABLE) {
144
+ bcm2836_control_local_timer_set_next(s);
145
+ } else {
146
+ timer_del(&s->timer);
147
+ }
148
+}
149
+
150
+static void bcm2836_control_local_timer_ack(void *opaque, uint32_t val)
151
+{
152
+ BCM2836ControlState *s = opaque;
153
+
154
+ if (val & LOCALTIMER_INTFLAG) {
155
+ s->local_timer_control &= ~LOCALTIMER_INTFLAG;
156
+ }
157
+ if ((val & LOCALTIMER_RELOAD) &&
158
+ (s->local_timer_control & LOCALTIMER_ENABLE)) {
159
+ bcm2836_control_local_timer_set_next(s);
160
+ }
161
+}
162
+
163
static uint64_t bcm2836_control_read(void *opaque, hwaddr offset, unsigned size)
164
{
165
BCM2836ControlState *s = opaque;
166
@@ -XXX,XX +XXX,XX @@ static uint64_t bcm2836_control_read(void *opaque, hwaddr offset, unsigned size)
167
assert(s->route_gpu_fiq < BCM2836_NCORES
168
&& s->route_gpu_irq < BCM2836_NCORES);
169
return ((uint32_t)s->route_gpu_fiq << 2) | s->route_gpu_irq;
170
+ } else if (offset == REG_LOCALTIMERROUTING) {
171
+ return s->route_localtimer;
172
+ } else if (offset == REG_LOCALTIMERCONTROL) {
173
+ return s->local_timer_control;
174
+ } else if (offset == REG_LOCALTIMERACK) {
175
+ return 0;
176
} else if (offset >= REG_TIMERCONTROL && offset < REG_MBOXCONTROL) {
177
return s->timercontrol[(offset - REG_TIMERCONTROL) >> 2];
178
} else if (offset >= REG_MBOXCONTROL && offset < REG_IRQSRC) {
179
@@ -XXX,XX +XXX,XX @@ static void bcm2836_control_write(void *opaque, hwaddr offset,
180
if (offset == REG_GPU_ROUTE) {
181
s->route_gpu_irq = val & 0x3;
182
s->route_gpu_fiq = (val >> 2) & 0x3;
183
+ } else if (offset == REG_LOCALTIMERROUTING) {
184
+ s->route_localtimer = val & 7;
185
+ } else if (offset == REG_LOCALTIMERCONTROL) {
186
+ bcm2836_control_local_timer_control(s, val);
187
+ } else if (offset == REG_LOCALTIMERACK) {
188
+ bcm2836_control_local_timer_ack(s, val);
189
} else if (offset >= REG_TIMERCONTROL && offset < REG_MBOXCONTROL) {
190
s->timercontrol[(offset - REG_TIMERCONTROL) >> 2] = val & 0xff;
191
} else if (offset >= REG_MBOXCONTROL && offset < REG_IRQSRC) {
192
@@ -XXX,XX +XXX,XX @@ static void bcm2836_control_reset(DeviceState *d)
193
194
s->route_gpu_irq = s->route_gpu_fiq = 0;
195
196
+ timer_del(&s->timer);
197
+ s->route_localtimer = 0;
198
+ s->local_timer_control = 0;
199
+
200
for (i = 0; i < BCM2836_NCORES; i++) {
201
s->timercontrol[i] = 0;
202
s->mailboxcontrol[i] = 0;
203
@@ -XXX,XX +XXX,XX @@ static void bcm2836_control_init(Object *obj)
204
/* outputs to CPU cores */
205
qdev_init_gpio_out_named(dev, s->irq, "irq", BCM2836_NCORES);
206
qdev_init_gpio_out_named(dev, s->fiq, "fiq", BCM2836_NCORES);
207
+
208
+ /* create a qemu virtual timer */
209
+ timer_init_ns(&s->timer, QEMU_CLOCK_VIRTUAL,
210
+ bcm2836_control_local_timer_tick, s);
211
}
212
213
static const VMStateDescription vmstate_bcm2836_control = {
214
.name = TYPE_BCM2836_CONTROL,
215
- .version_id = 1,
216
+ .version_id = 2,
217
.minimum_version_id = 1,
218
.fields = (VMStateField[]) {
219
VMSTATE_UINT32_ARRAY(mailboxes, BCM2836ControlState,
220
@@ -XXX,XX +XXX,XX @@ static const VMStateDescription vmstate_bcm2836_control = {
221
VMSTATE_UINT32_ARRAY(timercontrol, BCM2836ControlState, BCM2836_NCORES),
222
VMSTATE_UINT32_ARRAY(mailboxcontrol, BCM2836ControlState,
223
BCM2836_NCORES),
224
+ VMSTATE_TIMER_V(timer, BCM2836ControlState, 2),
225
+ VMSTATE_UINT32_V(local_timer_control, BCM2836ControlState, 2),
226
+ VMSTATE_UINT8_V(route_localtimer, BCM2836ControlState, 2),
227
VMSTATE_END_OF_LIST()
228
}
229
};
230
--
231
2.20.1
232
233
diff view generated by jsdifflib
Deleted patch
1
From: Eric Auger <eric.auger@redhat.com>
2
1
3
The GSIV numbers of the SPI based interrupts is not correct as
4
ARM_SPI_BASE was not added to the irqmap[VIRT_SMMU] value. So
5
this may collide with VIRTIO_MMIO irq window.
6
7
Signed-off-by: Eric Auger <eric.auger@redhat.com>
8
Message-id: 20190312091031.5185-1-eric.auger@redhat.com
9
Reviewed-by: Shannon Zhao <shannon.zhaosl@gmail.com>
10
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
11
---
12
hw/arm/virt-acpi-build.c | 2 +-
13
1 file changed, 1 insertion(+), 1 deletion(-)
14
15
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
16
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/arm/virt-acpi-build.c
18
+++ b/hw/arm/virt-acpi-build.c
19
@@ -XXX,XX +XXX,XX @@ build_iort(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms)
20
its->identifiers[0] = 0; /* MADT translation_id */
21
22
if (vms->iommu == VIRT_IOMMU_SMMUV3) {
23
- int irq = vms->irqmap[VIRT_SMMU];
24
+ int irq = vms->irqmap[VIRT_SMMU] + ARM_SPI_BASE;
25
26
/* SMMUv3 node */
27
smmu_offset = iort_node_offset + node_size;
28
--
29
2.20.1
30
31
diff view generated by jsdifflib
Deleted patch
1
From: Dongjiu Geng <gengdongjiu@huawei.com>
2
1
3
Some generic arch timer registers are Config-RW in the EL0,
4
which means the EL0 exception level can have write permission
5
if it is appropriately configured.
6
7
When VM access registers, QEMU firstly checks whether they have RW
8
permission, then check whether it is appropriately configured.
9
If they are defined to read only in EL0, even though they have been
10
appropriately configured, they still do not have write permission.
11
So need to add the write permission according to ARMV8 spec when
12
define it.
13
14
Signed-off-by: Dongjiu Geng <gengdongjiu@huawei.com>
15
Message-id: 1552395177-12608-1-git-send-email-gengdongjiu@huawei.com
16
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
17
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
18
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
19
---
20
target/arm/helper.c | 30 +++++++++++++++---------------
21
1 file changed, 15 insertions(+), 15 deletions(-)
22
23
diff --git a/target/arm/helper.c b/target/arm/helper.c
24
index XXXXXXX..XXXXXXX 100644
25
--- a/target/arm/helper.c
26
+++ b/target/arm/helper.c
27
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
28
/* per-timer control */
29
{ .name = "CNTP_CTL", .cp = 15, .crn = 14, .crm = 2, .opc1 = 0, .opc2 = 1,
30
.secure = ARM_CP_SECSTATE_NS,
31
- .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL1_RW | PL0_R,
32
+ .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL0_RW,
33
.accessfn = gt_ptimer_access,
34
.fieldoffset = offsetoflow32(CPUARMState,
35
cp15.c14_timer[GTIMER_PHYS].ctl),
36
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
37
{ .name = "CNTP_CTL_S",
38
.cp = 15, .crn = 14, .crm = 2, .opc1 = 0, .opc2 = 1,
39
.secure = ARM_CP_SECSTATE_S,
40
- .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL1_RW | PL0_R,
41
+ .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL0_RW,
42
.accessfn = gt_ptimer_access,
43
.fieldoffset = offsetoflow32(CPUARMState,
44
cp15.c14_timer[GTIMER_SEC].ctl),
45
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
46
},
47
{ .name = "CNTP_CTL_EL0", .state = ARM_CP_STATE_AA64,
48
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 2, .opc2 = 1,
49
- .type = ARM_CP_IO, .access = PL1_RW | PL0_R,
50
+ .type = ARM_CP_IO, .access = PL0_RW,
51
.accessfn = gt_ptimer_access,
52
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_PHYS].ctl),
53
.resetvalue = 0,
54
.writefn = gt_phys_ctl_write, .raw_writefn = raw_write,
55
},
56
{ .name = "CNTV_CTL", .cp = 15, .crn = 14, .crm = 3, .opc1 = 0, .opc2 = 1,
57
- .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL1_RW | PL0_R,
58
+ .type = ARM_CP_IO | ARM_CP_ALIAS, .access = PL0_RW,
59
.accessfn = gt_vtimer_access,
60
.fieldoffset = offsetoflow32(CPUARMState,
61
cp15.c14_timer[GTIMER_VIRT].ctl),
62
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
63
},
64
{ .name = "CNTV_CTL_EL0", .state = ARM_CP_STATE_AA64,
65
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 3, .opc2 = 1,
66
- .type = ARM_CP_IO, .access = PL1_RW | PL0_R,
67
+ .type = ARM_CP_IO, .access = PL0_RW,
68
.accessfn = gt_vtimer_access,
69
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_VIRT].ctl),
70
.resetvalue = 0,
71
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
72
/* TimerValue views: a 32 bit downcounting view of the underlying state */
73
{ .name = "CNTP_TVAL", .cp = 15, .crn = 14, .crm = 2, .opc1 = 0, .opc2 = 0,
74
.secure = ARM_CP_SECSTATE_NS,
75
- .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL1_RW | PL0_R,
76
+ .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL0_RW,
77
.accessfn = gt_ptimer_access,
78
.readfn = gt_phys_tval_read, .writefn = gt_phys_tval_write,
79
},
80
{ .name = "CNTP_TVAL_S",
81
.cp = 15, .crn = 14, .crm = 2, .opc1 = 0, .opc2 = 0,
82
.secure = ARM_CP_SECSTATE_S,
83
- .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL1_RW | PL0_R,
84
+ .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL0_RW,
85
.accessfn = gt_ptimer_access,
86
.readfn = gt_sec_tval_read, .writefn = gt_sec_tval_write,
87
},
88
{ .name = "CNTP_TVAL_EL0", .state = ARM_CP_STATE_AA64,
89
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 2, .opc2 = 0,
90
- .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL1_RW | PL0_R,
91
+ .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL0_RW,
92
.accessfn = gt_ptimer_access, .resetfn = gt_phys_timer_reset,
93
.readfn = gt_phys_tval_read, .writefn = gt_phys_tval_write,
94
},
95
{ .name = "CNTV_TVAL", .cp = 15, .crn = 14, .crm = 3, .opc1 = 0, .opc2 = 0,
96
- .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL1_RW | PL0_R,
97
+ .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL0_RW,
98
.accessfn = gt_vtimer_access,
99
.readfn = gt_virt_tval_read, .writefn = gt_virt_tval_write,
100
},
101
{ .name = "CNTV_TVAL_EL0", .state = ARM_CP_STATE_AA64,
102
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 3, .opc2 = 0,
103
- .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL1_RW | PL0_R,
104
+ .type = ARM_CP_NO_RAW | ARM_CP_IO, .access = PL0_RW,
105
.accessfn = gt_vtimer_access, .resetfn = gt_virt_timer_reset,
106
.readfn = gt_virt_tval_read, .writefn = gt_virt_tval_write,
107
},
108
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
109
/* Comparison value, indicating when the timer goes off */
110
{ .name = "CNTP_CVAL", .cp = 15, .crm = 14, .opc1 = 2,
111
.secure = ARM_CP_SECSTATE_NS,
112
- .access = PL1_RW | PL0_R,
113
+ .access = PL0_RW,
114
.type = ARM_CP_64BIT | ARM_CP_IO | ARM_CP_ALIAS,
115
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_PHYS].cval),
116
.accessfn = gt_ptimer_access,
117
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
118
},
119
{ .name = "CNTP_CVAL_S", .cp = 15, .crm = 14, .opc1 = 2,
120
.secure = ARM_CP_SECSTATE_S,
121
- .access = PL1_RW | PL0_R,
122
+ .access = PL0_RW,
123
.type = ARM_CP_64BIT | ARM_CP_IO | ARM_CP_ALIAS,
124
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_SEC].cval),
125
.accessfn = gt_ptimer_access,
126
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
127
},
128
{ .name = "CNTP_CVAL_EL0", .state = ARM_CP_STATE_AA64,
129
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 2, .opc2 = 2,
130
- .access = PL1_RW | PL0_R,
131
+ .access = PL0_RW,
132
.type = ARM_CP_IO,
133
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_PHYS].cval),
134
.resetvalue = 0, .accessfn = gt_ptimer_access,
135
.writefn = gt_phys_cval_write, .raw_writefn = raw_write,
136
},
137
{ .name = "CNTV_CVAL", .cp = 15, .crm = 14, .opc1 = 3,
138
- .access = PL1_RW | PL0_R,
139
+ .access = PL0_RW,
140
.type = ARM_CP_64BIT | ARM_CP_IO | ARM_CP_ALIAS,
141
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_VIRT].cval),
142
.accessfn = gt_vtimer_access,
143
@@ -XXX,XX +XXX,XX @@ static const ARMCPRegInfo generic_timer_cp_reginfo[] = {
144
},
145
{ .name = "CNTV_CVAL_EL0", .state = ARM_CP_STATE_AA64,
146
.opc0 = 3, .opc1 = 3, .crn = 14, .crm = 3, .opc2 = 2,
147
- .access = PL1_RW | PL0_R,
148
+ .access = PL0_RW,
149
.type = ARM_CP_IO,
150
.fieldoffset = offsetof(CPUARMState, cp15.c14_timer[GTIMER_VIRT].cval),
151
.resetvalue = 0, .accessfn = gt_vtimer_access,
152
--
153
2.20.1
154
155
diff view generated by jsdifflib
Deleted patch
1
From: Wei Yang <richardw.yang@linux.intel.com>
2
1
3
This is more proper to use PCIE_MMCFG_BUS to retrieve end_bus_number.
4
5
Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>
6
Reviewed-by: Eric Auger <eric.auger@redhat.com>
7
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
8
Message-id: 20190312074953.16671-1-richardw.yang@linux.intel.com
9
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
10
---
11
hw/arm/virt-acpi-build.c | 4 ++--
12
1 file changed, 2 insertions(+), 2 deletions(-)
13
14
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
15
index XXXXXXX..XXXXXXX 100644
16
--- a/hw/arm/virt-acpi-build.c
17
+++ b/hw/arm/virt-acpi-build.c
18
@@ -XXX,XX +XXX,XX @@ build_mcfg(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms)
19
/* Only a single allocation so no need to play with segments */
20
mcfg->allocation[0].pci_segment = cpu_to_le16(0);
21
mcfg->allocation[0].start_bus_number = 0;
22
- mcfg->allocation[0].end_bus_number = (memmap[ecam_id].size
23
- / PCIE_MMCFG_SIZE_MIN) - 1;
24
+ mcfg->allocation[0].end_bus_number =
25
+ PCIE_MMCFG_BUS(memmap[ecam_id].size - 1);
26
27
build_header(linker, table_data, (void *)(table_data->data + mcfg_start),
28
"MCFG", table_data->len - mcfg_start, 1, NULL, NULL);
29
--
30
2.20.1
31
32
diff view generated by jsdifflib
Deleted patch
1
From: Amir Charif <amir.charif@cea.fr>
2
1
3
These instructions do not trap when SVE is disabled in EL0,
4
causing them to be executed with wrong size information.
5
6
Signed-off-by: Amir Charif <amir.charif@cea.fr>
7
Message-id: 1552579248-31025-1-git-send-email-amir.charif@cea.fr
8
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
9
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
10
[PMM: added 'target/arm' prefix to subject]
11
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
12
---
13
target/arm/translate-sve.c | 22 ++++++++++++++--------
14
1 file changed, 14 insertions(+), 8 deletions(-)
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_INDEX_rr(DisasContext *s, arg_INDEX_rr *a)
21
22
static bool trans_ADDVL(DisasContext *s, arg_ADDVL *a)
23
{
24
- TCGv_i64 rd = cpu_reg_sp(s, a->rd);
25
- TCGv_i64 rn = cpu_reg_sp(s, a->rn);
26
- tcg_gen_addi_i64(rd, rn, a->imm * vec_full_reg_size(s));
27
+ if (sve_access_check(s)) {
28
+ TCGv_i64 rd = cpu_reg_sp(s, a->rd);
29
+ TCGv_i64 rn = cpu_reg_sp(s, a->rn);
30
+ tcg_gen_addi_i64(rd, rn, a->imm * vec_full_reg_size(s));
31
+ }
32
return true;
33
}
34
35
static bool trans_ADDPL(DisasContext *s, arg_ADDPL *a)
36
{
37
- TCGv_i64 rd = cpu_reg_sp(s, a->rd);
38
- TCGv_i64 rn = cpu_reg_sp(s, a->rn);
39
- tcg_gen_addi_i64(rd, rn, a->imm * pred_full_reg_size(s));
40
+ if (sve_access_check(s)) {
41
+ TCGv_i64 rd = cpu_reg_sp(s, a->rd);
42
+ TCGv_i64 rn = cpu_reg_sp(s, a->rn);
43
+ tcg_gen_addi_i64(rd, rn, a->imm * pred_full_reg_size(s));
44
+ }
45
return true;
46
}
47
48
static bool trans_RDVL(DisasContext *s, arg_RDVL *a)
49
{
50
- TCGv_i64 reg = cpu_reg(s, a->rd);
51
- tcg_gen_movi_i64(reg, a->imm * vec_full_reg_size(s));
52
+ if (sve_access_check(s)) {
53
+ TCGv_i64 reg = cpu_reg(s, a->rd);
54
+ tcg_gen_movi_i64(reg, a->imm * vec_full_reg_size(s));
55
+ }
56
return true;
57
}
58
59
--
60
2.20.1
61
62
diff view generated by jsdifflib