1 | A random mix of items here, nothing very major. | 1 | Not very much here, but several people have fallen over |
---|---|---|---|
2 | the vector operation segfault bug, so let's get the fix | ||
3 | into master. | ||
2 | 4 | ||
3 | thanks | 5 | thanks |
4 | -- PMM | 6 | -- PMM |
5 | 7 | ||
8 | The following changes since commit d418238dca7b4e0b124135827ead3076233052b1: | ||
6 | 9 | ||
7 | The following changes since commit d0dff238a87fa81393ed72754d4dc8b09e50b08b: | 10 | Merge remote-tracking branch 'remotes/rth/tags/pull-rng-20190522' into staging (2019-05-23 12:57:17 +0100) |
8 | 11 | ||
9 | Merge remote-tracking branch 'remotes/juanquintela/tags/migration/20170206' into staging (2017-02-07 15:29:26 +0000) | 12 | are available in the Git repository at: |
10 | 13 | ||
11 | are available in the git repository at: | 14 | https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20190523 |
12 | 15 | ||
13 | git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20170207 | 16 | for you to fetch changes up to 98e4f4fdb8ea05d840f51f47125924c2bb9df2df: |
14 | 17 | ||
15 | for you to fetch changes up to 7727b832886fafbdec7299eb7773dc9071bf4cdd: | 18 | hw/arm/exynos4210: QOM'ify the Exynos4210 SoC (2019-05-23 14:47:44 +0100) |
16 | |||
17 | stellaris: Use the 'unimplemented' device for parts we don't implement (2017-02-07 18:30:00 +0000) | ||
18 | 19 | ||
19 | ---------------------------------------------------------------- | 20 | ---------------------------------------------------------------- |
20 | target-arm: | 21 | target-arm queue: |
21 | * new "unimplemented" device for stubbing out devices in a | 22 | * exynos4210: QOM'ify the Exynos4210 SoC |
22 | system model so accesses can be logged | 23 | * exynos4210: Add DMA support for the Exynos4210 |
23 | * stellaris: document the SoC memory map | 24 | * arm_gicv3: Fix writes to ICC_CTLR_EL3 |
24 | * arm: create instruction syndromes for AArch32 data aborts | 25 | * arm_gicv3: Fix write of ICH_VMCR_EL2.{VBPR0, VBPR1} |
25 | * arm: Correctly handle watchpoints for BE32 CPUs | 26 | * target/arm: Fix vector operation segfault |
26 | * Fix Thumb-1 BE32 execution and disassembly | 27 | * target/arm: Minor improvements to BFXIL, EXTR |
27 | * arm: Add cfgend parameter for ARM CPU selection | ||
28 | * sd: sdhci: check data length during dma_memory_read | ||
29 | * aspeed: add a watchdog controller | ||
30 | * integratorcp: adding vmstate for save/restore | ||
31 | 28 | ||
32 | ---------------------------------------------------------------- | 29 | ---------------------------------------------------------------- |
33 | Cédric Le Goater (2): | 30 | Alistair Francis (1): |
34 | wdt: Add Aspeed watchdog device model | 31 | target/arm: Fix vector operation segfault |
35 | aspeed: add a watchdog controller | ||
36 | 32 | ||
37 | Julian Brown (4): | 33 | Guenter Roeck (1): |
38 | hw/arm/integratorcp: Support specifying features via -cpu | 34 | hw/arm/exynos4210: Add DMA support for the Exynos4210 |
39 | target/arm: Add cfgend parameter for ARM CPU selection. | ||
40 | Fix Thumb-1 BE32 execution and disassembly. | ||
41 | arm: Correctly handle watchpoints for BE32 CPUs | ||
42 | |||
43 | Pavel Dovgalyuk (1): | ||
44 | integratorcp: adding vmstate for save/restore | ||
45 | 35 | ||
46 | Peter Maydell (5): | 36 | Peter Maydell (5): |
47 | target/arm: Abstract out pbit/wbit tests in ARM ldr/str decode | 37 | arm: Move system_clock_scale to armv7m_systick.h |
48 | target/arm: A32, T32: Create Instruction Syndromes for Data Aborts | 38 | arm: Remove unnecessary includes of hw/arm/arm.h |
49 | stellaris: Document memory map and which SoC devices are unimplemented | 39 | arm: Rename hw/arm/arm.h to hw/arm/boot.h |
50 | hw/misc: New "unimplemented" sysbus device | 40 | hw/intc/arm_gicv3: Fix write of ICH_VMCR_EL2.{VBPR0, VBPR1} |
51 | stellaris: Use the 'unimplemented' device for parts we don't implement | 41 | hw/intc/arm_gicv3: Fix writes to ICC_CTLR_EL3 |
52 | 42 | ||
53 | Prasad J Pandit (1): | 43 | Philippe Mathieu-Daudé (3): |
54 | sd: sdhci: check data length during dma_memory_read | 44 | hw/arm/exynos4: Remove unuseful debug code |
45 | hw/arm/exynos4: Use the IEC binary prefix definitions | ||
46 | hw/arm/exynos4210: QOM'ify the Exynos4210 SoC | ||
55 | 47 | ||
56 | hw/misc/Makefile.objs | 2 + | 48 | Richard Henderson (2): |
57 | hw/watchdog/Makefile.objs | 1 + | 49 | target/arm: Use extract2 for EXTR |
58 | include/disas/bfd.h | 7 ++ | 50 | target/arm: Simplify BFXIL expansion |
59 | include/hw/arm/aspeed_soc.h | 2 + | ||
60 | include/hw/misc/unimp.h | 39 +++++++ | ||
61 | include/hw/watchdog/wdt_aspeed.h | 32 ++++++ | ||
62 | include/qom/cpu.h | 3 + | ||
63 | target/arm/arm_ldst.h | 10 +- | ||
64 | target/arm/cpu.h | 7 ++ | ||
65 | target/arm/internals.h | 5 + | ||
66 | target/arm/translate.h | 14 +++ | ||
67 | disas.c | 1 + | ||
68 | exec.c | 1 + | ||
69 | hw/arm/aspeed_soc.c | 13 +++ | ||
70 | hw/arm/integratorcp.c | 78 +++++++++++++- | ||
71 | hw/arm/stellaris.c | 48 +++++++++ | ||
72 | hw/misc/unimp.c | 107 +++++++++++++++++++ | ||
73 | hw/sd/sdhci.c | 2 +- | ||
74 | hw/watchdog/wdt_aspeed.c | 225 +++++++++++++++++++++++++++++++++++++++ | ||
75 | qom/cpu.c | 6 ++ | ||
76 | target/arm/cpu.c | 39 +++++++ | ||
77 | target/arm/op_helper.c | 22 ++++ | ||
78 | target/arm/translate-a64.c | 14 --- | ||
79 | target/arm/translate.c | 193 ++++++++++++++++++++++++--------- | ||
80 | 24 files changed, 801 insertions(+), 70 deletions(-) | ||
81 | create mode 100644 include/hw/misc/unimp.h | ||
82 | create mode 100644 include/hw/watchdog/wdt_aspeed.h | ||
83 | create mode 100644 hw/misc/unimp.c | ||
84 | create mode 100644 hw/watchdog/wdt_aspeed.c | ||
85 | 51 | ||
52 | include/hw/arm/allwinner-a10.h | 2 +- | ||
53 | include/hw/arm/aspeed_soc.h | 1 - | ||
54 | include/hw/arm/bcm2836.h | 1 - | ||
55 | include/hw/arm/{arm.h => boot.h} | 12 +++------ | ||
56 | include/hw/arm/exynos4210.h | 9 +++++-- | ||
57 | include/hw/arm/fsl-imx25.h | 2 +- | ||
58 | include/hw/arm/fsl-imx31.h | 2 +- | ||
59 | include/hw/arm/fsl-imx6.h | 2 +- | ||
60 | include/hw/arm/fsl-imx6ul.h | 2 +- | ||
61 | include/hw/arm/fsl-imx7.h | 2 +- | ||
62 | include/hw/arm/virt.h | 2 +- | ||
63 | include/hw/arm/xlnx-versal.h | 2 +- | ||
64 | include/hw/arm/xlnx-zynqmp.h | 2 +- | ||
65 | include/hw/timer/armv7m_systick.h | 22 ++++++++++++++++ | ||
66 | hw/arm/armsse.c | 2 +- | ||
67 | hw/arm/armv7m.c | 2 +- | ||
68 | hw/arm/aspeed.c | 2 +- | ||
69 | hw/arm/boot.c | 2 +- | ||
70 | hw/arm/collie.c | 2 +- | ||
71 | hw/arm/exynos4210.c | 54 ++++++++++++++++++++++++++++++++++++--- | ||
72 | hw/arm/exynos4_boards.c | 40 ++++++++--------------------- | ||
73 | hw/arm/highbank.c | 2 +- | ||
74 | hw/arm/integratorcp.c | 2 +- | ||
75 | hw/arm/mainstone.c | 2 +- | ||
76 | hw/arm/microbit.c | 2 +- | ||
77 | hw/arm/mps2-tz.c | 2 +- | ||
78 | hw/arm/mps2.c | 2 +- | ||
79 | hw/arm/msf2-soc.c | 1 - | ||
80 | hw/arm/msf2-som.c | 2 +- | ||
81 | hw/arm/musca.c | 2 +- | ||
82 | hw/arm/musicpal.c | 2 +- | ||
83 | hw/arm/netduino2.c | 2 +- | ||
84 | hw/arm/nrf51_soc.c | 2 +- | ||
85 | hw/arm/nseries.c | 2 +- | ||
86 | hw/arm/omap1.c | 2 +- | ||
87 | hw/arm/omap2.c | 2 +- | ||
88 | hw/arm/omap_sx1.c | 2 +- | ||
89 | hw/arm/palm.c | 2 +- | ||
90 | hw/arm/raspi.c | 2 +- | ||
91 | hw/arm/realview.c | 2 +- | ||
92 | hw/arm/spitz.c | 2 +- | ||
93 | hw/arm/stellaris.c | 2 +- | ||
94 | hw/arm/stm32f205_soc.c | 2 +- | ||
95 | hw/arm/strongarm.c | 2 +- | ||
96 | hw/arm/tosa.c | 2 +- | ||
97 | hw/arm/versatilepb.c | 2 +- | ||
98 | hw/arm/vexpress.c | 2 +- | ||
99 | hw/arm/virt.c | 2 +- | ||
100 | hw/arm/xilinx_zynq.c | 2 +- | ||
101 | hw/arm/xlnx-versal.c | 2 +- | ||
102 | hw/arm/z2.c | 2 +- | ||
103 | hw/intc/arm_gicv3_cpuif.c | 6 ++--- | ||
104 | hw/intc/armv7m_nvic.c | 1 - | ||
105 | target/arm/arm-semi.c | 1 - | ||
106 | target/arm/cpu.c | 1 - | ||
107 | target/arm/cpu64.c | 1 - | ||
108 | target/arm/kvm.c | 1 - | ||
109 | target/arm/kvm32.c | 1 - | ||
110 | target/arm/kvm64.c | 1 - | ||
111 | target/arm/translate-a64.c | 44 ++++++++++++++++--------------- | ||
112 | target/arm/translate.c | 4 +-- | ||
113 | 61 files changed, 164 insertions(+), 123 deletions(-) | ||
114 | rename include/hw/arm/{arm.h => boot.h} (96%) | ||
115 | diff view generated by jsdifflib |
1 | Add support for generating the ISS (Instruction Specific Syndrome) | 1 | From: Richard Henderson <richard.henderson@linaro.org> |
---|---|---|---|
2 | for Data Abort exceptions taken from AArch32. These syndromes are | ||
3 | used by hypervisors for example to trap and emulate memory accesses. | ||
4 | 2 | ||
5 | This is the equivalent for AArch32 guests of the work done for AArch64 | 3 | This is, after all, how we implement extract2 in tcg/aarch64. |
6 | guests in commit aaa1f954d4cab243. | ||
7 | 4 | ||
5 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
7 | Message-id: 20190514011129.11330-2-richard.henderson@linaro.org | ||
8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
9 | Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> | ||
10 | --- | 9 | --- |
11 | target/arm/translate.h | 14 ++++ | 10 | target/arm/translate-a64.c | 38 ++++++++++++++++++++------------------ |
12 | target/arm/translate-a64.c | 14 ---- | 11 | 1 file changed, 20 insertions(+), 18 deletions(-) |
13 | target/arm/translate.c | 184 +++++++++++++++++++++++++++++++++------------ | ||
14 | 3 files changed, 149 insertions(+), 63 deletions(-) | ||
15 | 12 | ||
16 | diff --git a/target/arm/translate.h b/target/arm/translate.h | ||
17 | index XXXXXXX..XXXXXXX 100644 | ||
18 | --- a/target/arm/translate.h | ||
19 | +++ b/target/arm/translate.h | ||
20 | @@ -XXX,XX +XXX,XX @@ static inline int default_exception_el(DisasContext *s) | ||
21 | ? 3 : MAX(1, s->current_el); | ||
22 | } | ||
23 | |||
24 | +static void disas_set_insn_syndrome(DisasContext *s, uint32_t syn) | ||
25 | +{ | ||
26 | + /* We don't need to save all of the syndrome so we mask and shift | ||
27 | + * out unneeded bits to help the sleb128 encoder do a better job. | ||
28 | + */ | ||
29 | + syn &= ARM_INSN_START_WORD2_MASK; | ||
30 | + syn >>= ARM_INSN_START_WORD2_SHIFT; | ||
31 | + | ||
32 | + /* We check and clear insn_start_idx to catch multiple updates. */ | ||
33 | + assert(s->insn_start_idx != 0); | ||
34 | + tcg_set_insn_param(s->insn_start_idx, 2, syn); | ||
35 | + s->insn_start_idx = 0; | ||
36 | +} | ||
37 | + | ||
38 | /* target-specific extra values for is_jmp */ | ||
39 | /* These instructions trap after executing, so the A32/T32 decoder must | ||
40 | * defer them until after the conditional execution state has been updated. | ||
41 | diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c | 13 | diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c |
42 | index XXXXXXX..XXXXXXX 100644 | 14 | index XXXXXXX..XXXXXXX 100644 |
43 | --- a/target/arm/translate-a64.c | 15 | --- a/target/arm/translate-a64.c |
44 | +++ b/target/arm/translate-a64.c | 16 | +++ b/target/arm/translate-a64.c |
45 | @@ -XXX,XX +XXX,XX @@ static inline void gen_goto_tb(DisasContext *s, int n, uint64_t dest) | 17 | @@ -XXX,XX +XXX,XX @@ static void disas_extract(DisasContext *s, uint32_t insn) |
46 | } | 18 | } else { |
47 | } | 19 | tcg_gen_ext32u_i64(tcg_rd, cpu_reg(s, rm)); |
48 | 20 | } | |
49 | -static void disas_set_insn_syndrome(DisasContext *s, uint32_t syn) | 21 | - } else if (rm == rn) { /* ROR */ |
50 | -{ | 22 | - tcg_rm = cpu_reg(s, rm); |
51 | - /* We don't need to save all of the syndrome so we mask and shift | 23 | - if (sf) { |
52 | - * out uneeded bits to help the sleb128 encoder do a better job. | 24 | - tcg_gen_rotri_i64(tcg_rd, tcg_rm, imm); |
53 | - */ | 25 | - } else { |
54 | - syn &= ARM_INSN_START_WORD2_MASK; | 26 | - TCGv_i32 tmp = tcg_temp_new_i32(); |
55 | - syn >>= ARM_INSN_START_WORD2_SHIFT; | 27 | - tcg_gen_extrl_i64_i32(tmp, tcg_rm); |
56 | - | 28 | - tcg_gen_rotri_i32(tmp, tmp, imm); |
57 | - /* We check and clear insn_start_idx to catch multiple updates. */ | 29 | - tcg_gen_extu_i32_i64(tcg_rd, tmp); |
58 | - assert(s->insn_start_idx != 0); | 30 | - tcg_temp_free_i32(tmp); |
59 | - tcg_set_insn_param(s->insn_start_idx, 2, syn); | 31 | - } |
60 | - s->insn_start_idx = 0; | 32 | } else { |
61 | -} | 33 | - tcg_rm = read_cpu_reg(s, rm, sf); |
62 | - | 34 | - tcg_rn = read_cpu_reg(s, rn, sf); |
63 | static void unallocated_encoding(DisasContext *s) | 35 | - tcg_gen_shri_i64(tcg_rm, tcg_rm, imm); |
64 | { | 36 | - tcg_gen_shli_i64(tcg_rn, tcg_rn, bitsize - imm); |
65 | /* Unallocated and reserved encodings are uncategorized */ | 37 | - tcg_gen_or_i64(tcg_rd, tcg_rm, tcg_rn); |
66 | diff --git a/target/arm/translate.c b/target/arm/translate.c | 38 | - if (!sf) { |
67 | index XXXXXXX..XXXXXXX 100644 | 39 | - tcg_gen_ext32u_i64(tcg_rd, tcg_rd); |
68 | --- a/target/arm/translate.c | 40 | + tcg_rm = cpu_reg(s, rm); |
69 | +++ b/target/arm/translate.c | 41 | + tcg_rn = cpu_reg(s, rn); |
70 | @@ -XXX,XX +XXX,XX @@ void arm_translate_init(void) | ||
71 | a64_translate_init(); | ||
72 | } | ||
73 | |||
74 | +/* Flags for the disas_set_da_iss info argument: | ||
75 | + * lower bits hold the Rt register number, higher bits are flags. | ||
76 | + */ | ||
77 | +typedef enum ISSInfo { | ||
78 | + ISSNone = 0, | ||
79 | + ISSRegMask = 0x1f, | ||
80 | + ISSInvalid = (1 << 5), | ||
81 | + ISSIsAcqRel = (1 << 6), | ||
82 | + ISSIsWrite = (1 << 7), | ||
83 | + ISSIs16Bit = (1 << 8), | ||
84 | +} ISSInfo; | ||
85 | + | 42 | + |
86 | +/* Save the syndrome information for a Data Abort */ | 43 | + if (sf) { |
87 | +static void disas_set_da_iss(DisasContext *s, TCGMemOp memop, ISSInfo issinfo) | 44 | + /* Specialization to ROR happens in EXTRACT2. */ |
88 | +{ | 45 | + tcg_gen_extract2_i64(tcg_rd, tcg_rm, tcg_rn, imm); |
89 | + uint32_t syn; | 46 | + } else { |
90 | + int sas = memop & MO_SIZE; | 47 | + TCGv_i32 t0 = tcg_temp_new_i32(); |
91 | + bool sse = memop & MO_SIGN; | ||
92 | + bool is_acqrel = issinfo & ISSIsAcqRel; | ||
93 | + bool is_write = issinfo & ISSIsWrite; | ||
94 | + bool is_16bit = issinfo & ISSIs16Bit; | ||
95 | + int srt = issinfo & ISSRegMask; | ||
96 | + | 48 | + |
97 | + if (issinfo & ISSInvalid) { | 49 | + tcg_gen_extrl_i64_i32(t0, tcg_rm); |
98 | + /* Some callsites want to conditionally provide ISS info, | 50 | + if (rm == rn) { |
99 | + * eg "only if this was not a writeback" | 51 | + tcg_gen_rotri_i32(t0, t0, imm); |
100 | + */ | 52 | + } else { |
101 | + return; | 53 | + TCGv_i32 t1 = tcg_temp_new_i32(); |
102 | + } | 54 | + tcg_gen_extrl_i64_i32(t1, tcg_rn); |
103 | + | 55 | + tcg_gen_extract2_i32(t0, t0, t1, imm); |
104 | + if (srt == 15) { | 56 | + tcg_temp_free_i32(t1); |
105 | + /* For AArch32, insns where the src/dest is R15 never generate | 57 | + } |
106 | + * ISS information. Catching that here saves checking at all | 58 | + tcg_gen_extu_i32_i64(tcg_rd, t0); |
107 | + * the call sites. | 59 | + tcg_temp_free_i32(t0); |
108 | + */ | ||
109 | + return; | ||
110 | + } | ||
111 | + | ||
112 | + syn = syn_data_abort_with_iss(0, sas, sse, srt, 0, is_acqrel, | ||
113 | + 0, 0, 0, is_write, 0, is_16bit); | ||
114 | + disas_set_insn_syndrome(s, syn); | ||
115 | +} | ||
116 | + | ||
117 | static inline ARMMMUIdx get_a32_user_mem_index(DisasContext *s) | ||
118 | { | ||
119 | /* Return the mmu_idx to use for A32/T32 "unprivileged load/store" | ||
120 | @@ -XXX,XX +XXX,XX @@ static inline void gen_aa32_ld##SUFF(DisasContext *s, TCGv_i32 val, \ | ||
121 | TCGv_i32 a32, int index) \ | ||
122 | { \ | ||
123 | gen_aa32_ld_i32(s, val, a32, index, OPC | s->be_data); \ | ||
124 | +} \ | ||
125 | +static inline void gen_aa32_ld##SUFF##_iss(DisasContext *s, \ | ||
126 | + TCGv_i32 val, \ | ||
127 | + TCGv_i32 a32, int index, \ | ||
128 | + ISSInfo issinfo) \ | ||
129 | +{ \ | ||
130 | + gen_aa32_ld_i32(s, val, a32, index, OPC | s->be_data); \ | ||
131 | + disas_set_da_iss(s, OPC, issinfo); \ | ||
132 | } | ||
133 | |||
134 | #define DO_GEN_ST(SUFF, OPC) \ | ||
135 | @@ -XXX,XX +XXX,XX @@ static inline void gen_aa32_st##SUFF(DisasContext *s, TCGv_i32 val, \ | ||
136 | TCGv_i32 a32, int index) \ | ||
137 | { \ | ||
138 | gen_aa32_st_i32(s, val, a32, index, OPC | s->be_data); \ | ||
139 | +} \ | ||
140 | +static inline void gen_aa32_st##SUFF##_iss(DisasContext *s, \ | ||
141 | + TCGv_i32 val, \ | ||
142 | + TCGv_i32 a32, int index, \ | ||
143 | + ISSInfo issinfo) \ | ||
144 | +{ \ | ||
145 | + gen_aa32_st_i32(s, val, a32, index, OPC | s->be_data); \ | ||
146 | + disas_set_da_iss(s, OPC, issinfo | ISSIsWrite); \ | ||
147 | } | ||
148 | |||
149 | static inline void gen_aa32_frob64(DisasContext *s, TCGv_i64 val) | ||
150 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
151 | tmp = tcg_temp_new_i32(); | ||
152 | switch (op1) { | ||
153 | case 0: /* lda */ | ||
154 | - gen_aa32_ld32u(s, tmp, addr, | ||
155 | - get_mem_index(s)); | ||
156 | + gen_aa32_ld32u_iss(s, tmp, addr, | ||
157 | + get_mem_index(s), | ||
158 | + rd | ISSIsAcqRel); | ||
159 | break; | ||
160 | case 2: /* ldab */ | ||
161 | - gen_aa32_ld8u(s, tmp, addr, | ||
162 | - get_mem_index(s)); | ||
163 | + gen_aa32_ld8u_iss(s, tmp, addr, | ||
164 | + get_mem_index(s), | ||
165 | + rd | ISSIsAcqRel); | ||
166 | break; | ||
167 | case 3: /* ldah */ | ||
168 | - gen_aa32_ld16u(s, tmp, addr, | ||
169 | - get_mem_index(s)); | ||
170 | + gen_aa32_ld16u_iss(s, tmp, addr, | ||
171 | + get_mem_index(s), | ||
172 | + rd | ISSIsAcqRel); | ||
173 | break; | ||
174 | default: | ||
175 | abort(); | ||
176 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
177 | tmp = load_reg(s, rm); | ||
178 | switch (op1) { | ||
179 | case 0: /* stl */ | ||
180 | - gen_aa32_st32(s, tmp, addr, | ||
181 | - get_mem_index(s)); | ||
182 | + gen_aa32_st32_iss(s, tmp, addr, | ||
183 | + get_mem_index(s), | ||
184 | + rm | ISSIsAcqRel); | ||
185 | break; | ||
186 | case 2: /* stlb */ | ||
187 | - gen_aa32_st8(s, tmp, addr, | ||
188 | - get_mem_index(s)); | ||
189 | + gen_aa32_st8_iss(s, tmp, addr, | ||
190 | + get_mem_index(s), | ||
191 | + rm | ISSIsAcqRel); | ||
192 | break; | ||
193 | case 3: /* stlh */ | ||
194 | - gen_aa32_st16(s, tmp, addr, | ||
195 | - get_mem_index(s)); | ||
196 | + gen_aa32_st16_iss(s, tmp, addr, | ||
197 | + get_mem_index(s), | ||
198 | + rm | ISSIsAcqRel); | ||
199 | break; | ||
200 | default: | ||
201 | abort(); | ||
202 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
203 | bool wbit = insn & (1 << 21); | ||
204 | bool pbit = insn & (1 << 24); | ||
205 | bool doubleword = false; | ||
206 | + ISSInfo issinfo; | ||
207 | + | ||
208 | /* Misc load/store */ | ||
209 | rn = (insn >> 16) & 0xf; | ||
210 | rd = (insn >> 12) & 0xf; | ||
211 | |||
212 | + /* ISS not valid if writeback */ | ||
213 | + issinfo = (pbit & !wbit) ? rd : ISSInvalid; | ||
214 | + | ||
215 | if (!load && (sh & 2)) { | ||
216 | /* doubleword */ | ||
217 | ARCH(5TE); | ||
218 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
219 | tmp = tcg_temp_new_i32(); | ||
220 | switch (sh) { | ||
221 | case 1: | ||
222 | - gen_aa32_ld16u(s, tmp, addr, get_mem_index(s)); | ||
223 | + gen_aa32_ld16u_iss(s, tmp, addr, get_mem_index(s), | ||
224 | + issinfo); | ||
225 | break; | ||
226 | case 2: | ||
227 | - gen_aa32_ld8s(s, tmp, addr, get_mem_index(s)); | ||
228 | + gen_aa32_ld8s_iss(s, tmp, addr, get_mem_index(s), | ||
229 | + issinfo); | ||
230 | break; | ||
231 | default: | ||
232 | case 3: | ||
233 | - gen_aa32_ld16s(s, tmp, addr, get_mem_index(s)); | ||
234 | + gen_aa32_ld16s_iss(s, tmp, addr, get_mem_index(s), | ||
235 | + issinfo); | ||
236 | break; | ||
237 | } | ||
238 | } else { | ||
239 | /* store */ | ||
240 | tmp = load_reg(s, rd); | ||
241 | - gen_aa32_st16(s, tmp, addr, get_mem_index(s)); | ||
242 | + gen_aa32_st16_iss(s, tmp, addr, get_mem_index(s), issinfo); | ||
243 | tcg_temp_free_i32(tmp); | ||
244 | } | ||
245 | /* Perform base writeback before the loaded value to | ||
246 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
247 | /* load */ | ||
248 | tmp = tcg_temp_new_i32(); | ||
249 | if (insn & (1 << 22)) { | ||
250 | - gen_aa32_ld8u(s, tmp, tmp2, i); | ||
251 | + gen_aa32_ld8u_iss(s, tmp, tmp2, i, rd); | ||
252 | } else { | ||
253 | - gen_aa32_ld32u(s, tmp, tmp2, i); | ||
254 | + gen_aa32_ld32u_iss(s, tmp, tmp2, i, rd); | ||
255 | } | ||
256 | } else { | ||
257 | /* store */ | ||
258 | tmp = load_reg(s, rd); | ||
259 | if (insn & (1 << 22)) { | ||
260 | - gen_aa32_st8(s, tmp, tmp2, i); | ||
261 | + gen_aa32_st8_iss(s, tmp, tmp2, i, rd); | ||
262 | } else { | ||
263 | - gen_aa32_st32(s, tmp, tmp2, i); | ||
264 | + gen_aa32_st32_iss(s, tmp, tmp2, i, rd); | ||
265 | } | ||
266 | tcg_temp_free_i32(tmp); | ||
267 | } | ||
268 | @@ -XXX,XX +XXX,XX @@ static int disas_thumb2_insn(CPUARMState *env, DisasContext *s, uint16_t insn_hw | ||
269 | tmp = tcg_temp_new_i32(); | ||
270 | switch (op) { | ||
271 | case 0: /* ldab */ | ||
272 | - gen_aa32_ld8u(s, tmp, addr, get_mem_index(s)); | ||
273 | + gen_aa32_ld8u_iss(s, tmp, addr, get_mem_index(s), | ||
274 | + rs | ISSIsAcqRel); | ||
275 | break; | ||
276 | case 1: /* ldah */ | ||
277 | - gen_aa32_ld16u(s, tmp, addr, get_mem_index(s)); | ||
278 | + gen_aa32_ld16u_iss(s, tmp, addr, get_mem_index(s), | ||
279 | + rs | ISSIsAcqRel); | ||
280 | break; | ||
281 | case 2: /* lda */ | ||
282 | - gen_aa32_ld32u(s, tmp, addr, get_mem_index(s)); | ||
283 | + gen_aa32_ld32u_iss(s, tmp, addr, get_mem_index(s), | ||
284 | + rs | ISSIsAcqRel); | ||
285 | break; | ||
286 | default: | ||
287 | abort(); | ||
288 | @@ -XXX,XX +XXX,XX @@ static int disas_thumb2_insn(CPUARMState *env, DisasContext *s, uint16_t insn_hw | ||
289 | tmp = load_reg(s, rs); | ||
290 | switch (op) { | ||
291 | case 0: /* stlb */ | ||
292 | - gen_aa32_st8(s, tmp, addr, get_mem_index(s)); | ||
293 | + gen_aa32_st8_iss(s, tmp, addr, get_mem_index(s), | ||
294 | + rs | ISSIsAcqRel); | ||
295 | break; | ||
296 | case 1: /* stlh */ | ||
297 | - gen_aa32_st16(s, tmp, addr, get_mem_index(s)); | ||
298 | + gen_aa32_st16_iss(s, tmp, addr, get_mem_index(s), | ||
299 | + rs | ISSIsAcqRel); | ||
300 | break; | ||
301 | case 2: /* stl */ | ||
302 | - gen_aa32_st32(s, tmp, addr, get_mem_index(s)); | ||
303 | + gen_aa32_st32_iss(s, tmp, addr, get_mem_index(s), | ||
304 | + rs | ISSIsAcqRel); | ||
305 | break; | ||
306 | default: | ||
307 | abort(); | ||
308 | @@ -XXX,XX +XXX,XX @@ static int disas_thumb2_insn(CPUARMState *env, DisasContext *s, uint16_t insn_hw | ||
309 | int postinc = 0; | ||
310 | int writeback = 0; | ||
311 | int memidx; | ||
312 | + ISSInfo issinfo; | ||
313 | + | ||
314 | if ((insn & 0x01100000) == 0x01000000) { | ||
315 | if (disas_neon_ls_insn(s, insn)) { | ||
316 | goto illegal_op; | ||
317 | @@ -XXX,XX +XXX,XX @@ static int disas_thumb2_insn(CPUARMState *env, DisasContext *s, uint16_t insn_hw | ||
318 | } | ||
319 | } | 60 | } |
320 | } | 61 | } |
321 | + | 62 | } |
322 | + issinfo = writeback ? ISSInvalid : rs; | ||
323 | + | ||
324 | if (insn & (1 << 20)) { | ||
325 | /* Load. */ | ||
326 | tmp = tcg_temp_new_i32(); | ||
327 | switch (op) { | ||
328 | case 0: | ||
329 | - gen_aa32_ld8u(s, tmp, addr, memidx); | ||
330 | + gen_aa32_ld8u_iss(s, tmp, addr, memidx, issinfo); | ||
331 | break; | ||
332 | case 4: | ||
333 | - gen_aa32_ld8s(s, tmp, addr, memidx); | ||
334 | + gen_aa32_ld8s_iss(s, tmp, addr, memidx, issinfo); | ||
335 | break; | ||
336 | case 1: | ||
337 | - gen_aa32_ld16u(s, tmp, addr, memidx); | ||
338 | + gen_aa32_ld16u_iss(s, tmp, addr, memidx, issinfo); | ||
339 | break; | ||
340 | case 5: | ||
341 | - gen_aa32_ld16s(s, tmp, addr, memidx); | ||
342 | + gen_aa32_ld16s_iss(s, tmp, addr, memidx, issinfo); | ||
343 | break; | ||
344 | case 2: | ||
345 | - gen_aa32_ld32u(s, tmp, addr, memidx); | ||
346 | + gen_aa32_ld32u_iss(s, tmp, addr, memidx, issinfo); | ||
347 | break; | ||
348 | default: | ||
349 | tcg_temp_free_i32(tmp); | ||
350 | @@ -XXX,XX +XXX,XX @@ static int disas_thumb2_insn(CPUARMState *env, DisasContext *s, uint16_t insn_hw | ||
351 | tmp = load_reg(s, rs); | ||
352 | switch (op) { | ||
353 | case 0: | ||
354 | - gen_aa32_st8(s, tmp, addr, memidx); | ||
355 | + gen_aa32_st8_iss(s, tmp, addr, memidx, issinfo); | ||
356 | break; | ||
357 | case 1: | ||
358 | - gen_aa32_st16(s, tmp, addr, memidx); | ||
359 | + gen_aa32_st16_iss(s, tmp, addr, memidx, issinfo); | ||
360 | break; | ||
361 | case 2: | ||
362 | - gen_aa32_st32(s, tmp, addr, memidx); | ||
363 | + gen_aa32_st32_iss(s, tmp, addr, memidx, issinfo); | ||
364 | break; | ||
365 | default: | ||
366 | tcg_temp_free_i32(tmp); | ||
367 | @@ -XXX,XX +XXX,XX @@ static void disas_thumb_insn(CPUARMState *env, DisasContext *s) | ||
368 | addr = tcg_temp_new_i32(); | ||
369 | tcg_gen_movi_i32(addr, val); | ||
370 | tmp = tcg_temp_new_i32(); | ||
371 | - gen_aa32_ld32u(s, tmp, addr, get_mem_index(s)); | ||
372 | + gen_aa32_ld32u_iss(s, tmp, addr, get_mem_index(s), | ||
373 | + rd | ISSIs16Bit); | ||
374 | tcg_temp_free_i32(addr); | ||
375 | store_reg(s, rd, tmp); | ||
376 | break; | ||
377 | @@ -XXX,XX +XXX,XX @@ static void disas_thumb_insn(CPUARMState *env, DisasContext *s) | ||
378 | |||
379 | switch (op) { | ||
380 | case 0: /* str */ | ||
381 | - gen_aa32_st32(s, tmp, addr, get_mem_index(s)); | ||
382 | + gen_aa32_st32_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
383 | break; | ||
384 | case 1: /* strh */ | ||
385 | - gen_aa32_st16(s, tmp, addr, get_mem_index(s)); | ||
386 | + gen_aa32_st16_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
387 | break; | ||
388 | case 2: /* strb */ | ||
389 | - gen_aa32_st8(s, tmp, addr, get_mem_index(s)); | ||
390 | + gen_aa32_st8_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
391 | break; | ||
392 | case 3: /* ldrsb */ | ||
393 | - gen_aa32_ld8s(s, tmp, addr, get_mem_index(s)); | ||
394 | + gen_aa32_ld8s_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
395 | break; | ||
396 | case 4: /* ldr */ | ||
397 | - gen_aa32_ld32u(s, tmp, addr, get_mem_index(s)); | ||
398 | + gen_aa32_ld32u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
399 | break; | ||
400 | case 5: /* ldrh */ | ||
401 | - gen_aa32_ld16u(s, tmp, addr, get_mem_index(s)); | ||
402 | + gen_aa32_ld16u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
403 | break; | ||
404 | case 6: /* ldrb */ | ||
405 | - gen_aa32_ld8u(s, tmp, addr, get_mem_index(s)); | ||
406 | + gen_aa32_ld8u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
407 | break; | ||
408 | case 7: /* ldrsh */ | ||
409 | - gen_aa32_ld16s(s, tmp, addr, get_mem_index(s)); | ||
410 | + gen_aa32_ld16s_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
411 | break; | ||
412 | } | ||
413 | if (op >= 3) { /* load */ | ||
414 | @@ -XXX,XX +XXX,XX @@ static void disas_thumb_insn(CPUARMState *env, DisasContext *s) | ||
415 | if (insn & (1 << 11)) { | ||
416 | /* load */ | ||
417 | tmp = tcg_temp_new_i32(); | ||
418 | - gen_aa32_ld8u(s, tmp, addr, get_mem_index(s)); | ||
419 | + gen_aa32_ld8u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
420 | store_reg(s, rd, tmp); | ||
421 | } else { | ||
422 | /* store */ | ||
423 | tmp = load_reg(s, rd); | ||
424 | - gen_aa32_st8(s, tmp, addr, get_mem_index(s)); | ||
425 | + gen_aa32_st8_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
426 | tcg_temp_free_i32(tmp); | ||
427 | } | ||
428 | tcg_temp_free_i32(addr); | ||
429 | @@ -XXX,XX +XXX,XX @@ static void disas_thumb_insn(CPUARMState *env, DisasContext *s) | ||
430 | if (insn & (1 << 11)) { | ||
431 | /* load */ | ||
432 | tmp = tcg_temp_new_i32(); | ||
433 | - gen_aa32_ld16u(s, tmp, addr, get_mem_index(s)); | ||
434 | + gen_aa32_ld16u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
435 | store_reg(s, rd, tmp); | ||
436 | } else { | ||
437 | /* store */ | ||
438 | tmp = load_reg(s, rd); | ||
439 | - gen_aa32_st16(s, tmp, addr, get_mem_index(s)); | ||
440 | + gen_aa32_st16_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
441 | tcg_temp_free_i32(tmp); | ||
442 | } | ||
443 | tcg_temp_free_i32(addr); | ||
444 | @@ -XXX,XX +XXX,XX @@ static void disas_thumb_insn(CPUARMState *env, DisasContext *s) | ||
445 | if (insn & (1 << 11)) { | ||
446 | /* load */ | ||
447 | tmp = tcg_temp_new_i32(); | ||
448 | - gen_aa32_ld32u(s, tmp, addr, get_mem_index(s)); | ||
449 | + gen_aa32_ld32u_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
450 | store_reg(s, rd, tmp); | ||
451 | } else { | ||
452 | /* store */ | ||
453 | tmp = load_reg(s, rd); | ||
454 | - gen_aa32_st32(s, tmp, addr, get_mem_index(s)); | ||
455 | + gen_aa32_st32_iss(s, tmp, addr, get_mem_index(s), rd | ISSIs16Bit); | ||
456 | tcg_temp_free_i32(tmp); | ||
457 | } | ||
458 | tcg_temp_free_i32(addr); | ||
459 | @@ -XXX,XX +XXX,XX @@ void gen_intermediate_code(CPUARMState *env, TranslationBlock *tb) | ||
460 | store_cpu_field(tmp, condexec_bits); | ||
461 | } | ||
462 | do { | ||
463 | + dc->insn_start_idx = tcg_op_buf_count(); | ||
464 | tcg_gen_insn_start(dc->pc, | ||
465 | (dc->condexec_cond << 4) | (dc->condexec_mask >> 1), | ||
466 | 0); | ||
467 | -- | 63 | -- |
468 | 2.7.4 | 64 | 2.20.1 |
469 | 65 | ||
470 | 66 | diff view generated by jsdifflib |
1 | From: Julian Brown <julian@codesourcery.com> | 1 | From: Richard Henderson <richard.henderson@linaro.org> |
---|---|---|---|
2 | 2 | ||
3 | Since the integratorcp board creates the CPU object directly | 3 | The mask implied by the extract is redundant with the one |
4 | rather than via cpu_arm_init(), we have to call the CPU | 4 | implied by the deposit. Also, fix spelling of BFXIL. |
5 | class parse_features() method ourselves if we want to | ||
6 | support the user passing features via the -cpu command | ||
7 | line argument as well as just the cpu name. Do so. | ||
8 | 5 | ||
9 | Signed-off-by: Julian Brown <julian@codesourcery.com> | 6 | Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> |
10 | [PMM: split out into its own patch] | 7 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> |
11 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | 8 | Message-id: 20190514011129.11330-3-richard.henderson@linaro.org |
12 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 9 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
13 | --- | 10 | --- |
14 | hw/arm/integratorcp.c | 19 +++++++++++++++++-- | 11 | target/arm/translate-a64.c | 6 +++--- |
15 | 1 file changed, 17 insertions(+), 2 deletions(-) | 12 | 1 file changed, 3 insertions(+), 3 deletions(-) |
16 | 13 | ||
17 | diff --git a/hw/arm/integratorcp.c b/hw/arm/integratorcp.c | 14 | diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c |
18 | index XXXXXXX..XXXXXXX 100644 | 15 | index XXXXXXX..XXXXXXX 100644 |
19 | --- a/hw/arm/integratorcp.c | 16 | --- a/target/arm/translate-a64.c |
20 | +++ b/hw/arm/integratorcp.c | 17 | +++ b/target/arm/translate-a64.c |
21 | @@ -XXX,XX +XXX,XX @@ static void integratorcp_init(MachineState *machine) | 18 | @@ -XXX,XX +XXX,XX @@ static void disas_bitfield(DisasContext *s, uint32_t insn) |
22 | const char *kernel_filename = machine->kernel_filename; | 19 | tcg_gen_extract_i64(tcg_rd, tcg_tmp, ri, len); |
23 | const char *kernel_cmdline = machine->kernel_cmdline; | 20 | return; |
24 | const char *initrd_filename = machine->initrd_filename; | 21 | } |
25 | + char **cpustr; | 22 | - /* opc == 1, BXFIL fall through to deposit */ |
26 | ObjectClass *cpu_oc; | 23 | - tcg_gen_extract_i64(tcg_tmp, tcg_tmp, ri, len); |
27 | + CPUClass *cc; | 24 | + /* opc == 1, BFXIL fall through to deposit */ |
28 | Object *cpuobj; | 25 | + tcg_gen_shri_i64(tcg_tmp, tcg_tmp, ri); |
29 | ARMCPU *cpu; | 26 | pos = 0; |
30 | + const char *typename; | 27 | } else { |
31 | MemoryRegion *address_space_mem = get_system_memory(); | 28 | /* Handle the ri > si case with a deposit |
32 | MemoryRegion *ram = g_new(MemoryRegion, 1); | 29 | @@ -XXX,XX +XXX,XX @@ static void disas_bitfield(DisasContext *s, uint32_t insn) |
33 | MemoryRegion *ram_alias = g_new(MemoryRegion, 1); | 30 | len = ri; |
34 | qemu_irq pic[32]; | ||
35 | DeviceState *dev, *sic, *icp; | ||
36 | int i; | ||
37 | + Error *err = NULL; | ||
38 | |||
39 | if (!cpu_model) { | ||
40 | cpu_model = "arm926"; | ||
41 | } | 31 | } |
42 | 32 | ||
43 | - cpu_oc = cpu_class_by_name(TYPE_ARM_CPU, cpu_model); | 33 | - if (opc == 1) { /* BFM, BXFIL */ |
44 | + cpustr = g_strsplit(cpu_model, ",", 2); | 34 | + if (opc == 1) { /* BFM, BFXIL */ |
45 | + | 35 | tcg_gen_deposit_i64(tcg_rd, tcg_rd, tcg_tmp, pos, len); |
46 | + cpu_oc = cpu_class_by_name(TYPE_ARM_CPU, cpustr[0]); | 36 | } else { |
47 | if (!cpu_oc) { | 37 | /* SBFM or UBFM: We start with zero, and we haven't modified |
48 | fprintf(stderr, "Unable to find CPU definition\n"); | ||
49 | exit(1); | ||
50 | } | ||
51 | + typename = object_class_get_name(cpu_oc); | ||
52 | + | ||
53 | + cc = CPU_CLASS(cpu_oc); | ||
54 | + cc->parse_features(typename, cpustr[1], &err); | ||
55 | + g_strfreev(cpustr); | ||
56 | + if (err) { | ||
57 | + error_report_err(err); | ||
58 | + exit(1); | ||
59 | + } | ||
60 | |||
61 | - cpuobj = object_new(object_class_get_name(cpu_oc)); | ||
62 | + cpuobj = object_new(typename); | ||
63 | |||
64 | /* By default ARM1176 CPUs have EL3 enabled. This board does not | ||
65 | * currently support EL3 so the CPU EL3 property is disabled before | ||
66 | -- | 38 | -- |
67 | 2.7.4 | 39 | 2.20.1 |
68 | 40 | ||
69 | 41 | diff view generated by jsdifflib |
1 | In the ARM ldr/str decode path, rather than directly testing | 1 | From: Alistair Francis <alistair.francis@wdc.com> |
---|---|---|---|
2 | "insn & (1 << 21)" and "insn & (1 << 24)", abstract these | ||
3 | bits out into wbit and pbit local flags. (We will want to | ||
4 | do more tests against them to determine whether we need to | ||
5 | provide syndrome information.) | ||
6 | 2 | ||
3 | Commit 89e68b575 "target/arm: Use vector operations for saturation" | ||
4 | causes this abort() when booting QEMU ARM with a Cortex-A15: | ||
5 | |||
6 | 0 0x00007ffff4c2382f in raise () at /usr/lib/libc.so.6 | ||
7 | 1 0x00007ffff4c0e672 in abort () at /usr/lib/libc.so.6 | ||
8 | 2 0x00005555559c1839 in disas_neon_data_insn (insn=<optimized out>, s=<optimized out>) at ./target/arm/translate.c:6673 | ||
9 | 3 0x00005555559c1839 in disas_neon_data_insn (s=<optimized out>, insn=<optimized out>) at ./target/arm/translate.c:6386 | ||
10 | 4 0x00005555559cd8a4 in disas_arm_insn (insn=4081107068, s=0x7fffe59a9510) at ./target/arm/translate.c:9289 | ||
11 | 5 0x00005555559cd8a4 in arm_tr_translate_insn (dcbase=0x7fffe59a9510, cpu=<optimized out>) at ./target/arm/translate.c:13612 | ||
12 | 6 0x00005555558d1d39 in translator_loop (ops=0x5555561cc580 <arm_translator_ops>, db=0x7fffe59a9510, cpu=0x55555686a2f0, tb=<optimized out>, max_insns=<optimized out>) at ./accel/tcg/translator.c:96 | ||
13 | 7 0x00005555559d10d4 in gen_intermediate_code (cpu=cpu@entry=0x55555686a2f0, tb=tb@entry=0x7fffd7840080 <code_gen_buffer+126091347>, max_insns=max_insns@entry=512) at ./target/arm/translate.c:13901 | ||
14 | 8 0x00005555558d06b9 in tb_gen_code (cpu=cpu@entry=0x55555686a2f0, pc=3067096216, cs_base=0, flags=192, cflags=-16252928, cflags@entry=524288) at ./accel/tcg/translate-all.c:1736 | ||
15 | 9 0x00005555558ce467 in tb_find (cf_mask=524288, tb_exit=1, last_tb=0x7fffd783e640 <code_gen_buffer+126084627>, cpu=0x1) at ./accel/tcg/cpu-exec.c:407 | ||
16 | 10 0x00005555558ce467 in cpu_exec (cpu=cpu@entry=0x55555686a2f0) at ./accel/tcg/cpu-exec.c:728 | ||
17 | 11 0x000055555588b0cf in tcg_cpu_exec (cpu=0x55555686a2f0) at ./cpus.c:1431 | ||
18 | 12 0x000055555588d223 in qemu_tcg_cpu_thread_fn (arg=0x55555686a2f0) at ./cpus.c:1735 | ||
19 | 13 0x000055555588d223 in qemu_tcg_cpu_thread_fn (arg=arg@entry=0x55555686a2f0) at ./cpus.c:1709 | ||
20 | 14 0x0000555555d2629a in qemu_thread_start (args=<optimized out>) at ./util/qemu-thread-posix.c:502 | ||
21 | 15 0x00007ffff4db8a92 in start_thread () at /usr/lib/libpthread. | ||
22 | |||
23 | This patch ensures that we don't hit the abort() in the second switch | ||
24 | case in disas_neon_data_insn() as we will return from the first case. | ||
25 | |||
26 | Signed-off-by: Alistair Francis <alistair.francis@wdc.com> | ||
27 | Reviewed-by: Richard Henderson <richard.henderson@linaro.org> | ||
28 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
29 | Reviewed-by: Alex Bennée <alex.bennee@linaro.org> | ||
30 | Tested-by: Alex Bennée <alex.bennee@linaro.org> | ||
31 | Message-id: ad91b397f360b2fc7f4087e476f7df5b04d42ddb.1558021877.git.alistair.francis@wdc.com | ||
7 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 32 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
8 | Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> | ||
9 | --- | 33 | --- |
10 | target/arm/translate.c | 9 ++++++--- | 34 | target/arm/translate.c | 4 ++-- |
11 | 1 file changed, 6 insertions(+), 3 deletions(-) | 35 | 1 file changed, 2 insertions(+), 2 deletions(-) |
12 | 36 | ||
13 | diff --git a/target/arm/translate.c b/target/arm/translate.c | 37 | diff --git a/target/arm/translate.c b/target/arm/translate.c |
14 | index XXXXXXX..XXXXXXX 100644 | 38 | index XXXXXXX..XXXXXXX 100644 |
15 | --- a/target/arm/translate.c | 39 | --- a/target/arm/translate.c |
16 | +++ b/target/arm/translate.c | 40 | +++ b/target/arm/translate.c |
17 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | 41 | @@ -XXX,XX +XXX,XX @@ static int disas_neon_data_insn(DisasContext *s, uint32_t insn) |
18 | } else { | 42 | tcg_gen_gvec_4(rd_ofs, offsetof(CPUARMState, vfp.qc), |
19 | int address_offset; | 43 | rn_ofs, rm_ofs, vec_size, vec_size, |
20 | bool load = insn & (1 << 20); | 44 | (u ? uqadd_op : sqadd_op) + size); |
21 | + bool wbit = insn & (1 << 21); | 45 | - break; |
22 | + bool pbit = insn & (1 << 24); | 46 | + return 0; |
23 | bool doubleword = false; | 47 | |
24 | /* Misc load/store */ | 48 | case NEON_3R_VQSUB: |
25 | rn = (insn >> 16) & 0xf; | 49 | tcg_gen_gvec_4(rd_ofs, offsetof(CPUARMState, vfp.qc), |
26 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | 50 | rn_ofs, rm_ofs, vec_size, vec_size, |
27 | } | 51 | (u ? uqsub_op : sqsub_op) + size); |
28 | 52 | - break; | |
29 | addr = load_reg(s, rn); | 53 | + return 0; |
30 | - if (insn & (1 << 24)) | 54 | |
31 | + if (pbit) { | 55 | case NEON_3R_VMUL: /* VMUL */ |
32 | gen_add_datah_offset(s, insn, 0, addr); | 56 | if (u) { |
33 | + } | ||
34 | address_offset = 0; | ||
35 | |||
36 | if (doubleword) { | ||
37 | @@ -XXX,XX +XXX,XX @@ static void disas_arm_insn(DisasContext *s, unsigned int insn) | ||
38 | ensure correct behavior with overlapping index registers. | ||
39 | ldrd with base writeback is undefined if the | ||
40 | destination and index registers overlap. */ | ||
41 | - if (!(insn & (1 << 24))) { | ||
42 | + if (!pbit) { | ||
43 | gen_add_datah_offset(s, insn, address_offset, addr); | ||
44 | store_reg(s, rn, addr); | ||
45 | - } else if (insn & (1 << 21)) { | ||
46 | + } else if (wbit) { | ||
47 | if (address_offset) | ||
48 | tcg_gen_addi_i32(addr, addr, address_offset); | ||
49 | store_reg(s, rn, addr); | ||
50 | -- | 57 | -- |
51 | 2.7.4 | 58 | 2.20.1 |
52 | 59 | ||
53 | 60 | diff view generated by jsdifflib |
1 | Create a new "unimplemented" sysbus device, which simply accepts | 1 | The system_clock_scale global is used only by the armv7m systick |
---|---|---|---|
2 | all read and write accesses, and implements them as read-as-zero, | 2 | device; move the extern declaration to the armv7m_systick.h header, |
3 | write-ignored, with logging of the access as LOG_UNIMP. | 3 | and expand the comment to explain what it is and that it should |
4 | 4 | ideally be replaced with a different approach. | |
5 | This is useful for stubbing out bits of an SoC or board model | ||
6 | which haven't been written yet. | ||
7 | 5 | ||
8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 6 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
9 | Reviewed-by: Alex Bennée <alex.bennee@linaro.org> | 7 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
10 | Message-id: 1484247815-15279-3-git-send-email-peter.maydell@linaro.org | 8 | Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
9 | Message-id: 20190516163857.6430-2-peter.maydell@linaro.org | ||
11 | --- | 10 | --- |
12 | hw/misc/Makefile.objs | 2 + | 11 | include/hw/arm/arm.h | 4 ---- |
13 | include/hw/misc/unimp.h | 39 ++++++++++++++++++ | 12 | include/hw/timer/armv7m_systick.h | 22 ++++++++++++++++++++++ |
14 | hw/misc/unimp.c | 107 ++++++++++++++++++++++++++++++++++++++++++++++++ | 13 | 2 files changed, 22 insertions(+), 4 deletions(-) |
15 | 3 files changed, 148 insertions(+) | ||
16 | create mode 100644 include/hw/misc/unimp.h | ||
17 | create mode 100644 hw/misc/unimp.c | ||
18 | 14 | ||
19 | diff --git a/hw/misc/Makefile.objs b/hw/misc/Makefile.objs | 15 | diff --git a/include/hw/arm/arm.h b/include/hw/arm/arm.h |
20 | index XXXXXXX..XXXXXXX 100644 | 16 | index XXXXXXX..XXXXXXX 100644 |
21 | --- a/hw/misc/Makefile.objs | 17 | --- a/include/hw/arm/arm.h |
22 | +++ b/hw/misc/Makefile.objs | 18 | +++ b/include/hw/arm/arm.h |
23 | @@ -XXX,XX +XXX,XX @@ common-obj-$(CONFIG_SGA) += sga.o | 19 | @@ -XXX,XX +XXX,XX @@ void arm_write_secure_board_setup_dummy_smc(ARMCPU *cpu, |
24 | common-obj-$(CONFIG_ISA_TESTDEV) += pc-testdev.o | 20 | const struct arm_boot_info *info, |
25 | common-obj-$(CONFIG_PCI_TESTDEV) += pci-testdev.o | 21 | hwaddr mvbar_addr); |
26 | 22 | ||
27 | +common-obj-y += unimp.o | 23 | -/* Multiplication factor to convert from system clock ticks to qemu timer |
24 | - ticks. */ | ||
25 | -extern int system_clock_scale; | ||
26 | - | ||
27 | #endif /* HW_ARM_H */ | ||
28 | diff --git a/include/hw/timer/armv7m_systick.h b/include/hw/timer/armv7m_systick.h | ||
29 | index XXXXXXX..XXXXXXX 100644 | ||
30 | --- a/include/hw/timer/armv7m_systick.h | ||
31 | +++ b/include/hw/timer/armv7m_systick.h | ||
32 | @@ -XXX,XX +XXX,XX @@ typedef struct SysTickState { | ||
33 | qemu_irq irq; | ||
34 | } SysTickState; | ||
35 | |||
36 | +/* | ||
37 | + * Multiplication factor to convert from system clock ticks to qemu timer | ||
38 | + * ticks. This should be set (by board code, usually) to a value | ||
39 | + * equal to NANOSECONDS_PER_SECOND / frq, where frq is the clock frequency | ||
40 | + * in Hz of the CPU. | ||
41 | + * | ||
42 | + * This value is used by the systick device when it is running in | ||
43 | + * its "use the CPU clock" mode (ie when SYST_CSR.CLKSOURCE == 1) to | ||
44 | + * set how fast the timer should tick. | ||
45 | + * | ||
46 | + * TODO: we should refactor this so that rather than using a global | ||
47 | + * we use a device property or something similar. This is complicated | ||
48 | + * because (a) the property would need to be plumbed through from the | ||
49 | + * board code down through various layers to the systick device | ||
50 | + * and (b) the property needs to be modifiable after realize, because | ||
51 | + * the stellaris board uses this to implement the behaviour where the | ||
52 | + * guest can reprogram the PLL registers to downclock the CPU, and the | ||
53 | + * systick device needs to react accordingly. Possibly this should | ||
54 | + * be deferred until we have a good API for modelling clock trees. | ||
55 | + */ | ||
56 | +extern int system_clock_scale; | ||
28 | + | 57 | + |
29 | obj-$(CONFIG_VMPORT) += vmport.o | 58 | #endif |
30 | |||
31 | # ARM devices | ||
32 | diff --git a/include/hw/misc/unimp.h b/include/hw/misc/unimp.h | ||
33 | new file mode 100644 | ||
34 | index XXXXXXX..XXXXXXX | ||
35 | --- /dev/null | ||
36 | +++ b/include/hw/misc/unimp.h | ||
37 | @@ -XXX,XX +XXX,XX @@ | ||
38 | +/* | ||
39 | + * "Unimplemented" device | ||
40 | + * | ||
41 | + * Copyright Linaro Limited, 2017 | ||
42 | + * Written by Peter Maydell | ||
43 | + */ | ||
44 | + | ||
45 | +#ifndef HW_MISC_UNIMP_H | ||
46 | +#define HW_MISC_UNIMP_H | ||
47 | + | ||
48 | +#define TYPE_UNIMPLEMENTED_DEVICE "unimplemented-device" | ||
49 | + | ||
50 | +/** | ||
51 | + * create_unimplemented_device: create and map a dummy device | ||
52 | + * @name: name of the device for debug logging | ||
53 | + * @base: base address of the device's MMIO region | ||
54 | + * @size: size of the device's MMIO region | ||
55 | + * | ||
56 | + * This utility function creates and maps an instance of unimplemented-device, | ||
57 | + * which is a dummy device which simply logs all guest accesses to | ||
58 | + * it via the qemu_log LOG_UNIMP debug log. | ||
59 | + * The device is mapped at priority -1000, which means that you can | ||
60 | + * use it to cover a large region and then map other devices on top of it | ||
61 | + * if necessary. | ||
62 | + */ | ||
63 | +static inline void create_unimplemented_device(const char *name, | ||
64 | + hwaddr base, | ||
65 | + hwaddr size) | ||
66 | +{ | ||
67 | + DeviceState *dev = qdev_create(NULL, TYPE_UNIMPLEMENTED_DEVICE); | ||
68 | + | ||
69 | + qdev_prop_set_string(dev, "name", name); | ||
70 | + qdev_prop_set_uint64(dev, "size", size); | ||
71 | + qdev_init_nofail(dev); | ||
72 | + | ||
73 | + sysbus_mmio_map_overlap(SYS_BUS_DEVICE(dev), 0, base, -1000); | ||
74 | +} | ||
75 | + | ||
76 | +#endif | ||
77 | diff --git a/hw/misc/unimp.c b/hw/misc/unimp.c | ||
78 | new file mode 100644 | ||
79 | index XXXXXXX..XXXXXXX | ||
80 | --- /dev/null | ||
81 | +++ b/hw/misc/unimp.c | ||
82 | @@ -XXX,XX +XXX,XX @@ | ||
83 | +/* "Unimplemented" device | ||
84 | + * | ||
85 | + * This is a dummy device which accepts and logs all accesses. | ||
86 | + * It's useful for stubbing out regions of an SoC or board | ||
87 | + * map which correspond to devices that have not yet been | ||
88 | + * implemented. This is often sufficient to placate initial | ||
89 | + * guest device driver probing such that the system will | ||
90 | + * come up. | ||
91 | + * | ||
92 | + * Copyright Linaro Limited, 2017 | ||
93 | + * Written by Peter Maydell | ||
94 | + */ | ||
95 | + | ||
96 | +#include "qemu/osdep.h" | ||
97 | +#include "hw/hw.h" | ||
98 | +#include "hw/sysbus.h" | ||
99 | +#include "hw/misc/unimp.h" | ||
100 | +#include "qemu/log.h" | ||
101 | +#include "qapi/error.h" | ||
102 | + | ||
103 | +#define UNIMPLEMENTED_DEVICE(obj) \ | ||
104 | + OBJECT_CHECK(UnimplementedDeviceState, (obj), TYPE_UNIMPLEMENTED_DEVICE) | ||
105 | + | ||
106 | +typedef struct { | ||
107 | + SysBusDevice parent_obj; | ||
108 | + MemoryRegion iomem; | ||
109 | + char *name; | ||
110 | + uint64_t size; | ||
111 | +} UnimplementedDeviceState; | ||
112 | + | ||
113 | +static uint64_t unimp_read(void *opaque, hwaddr offset, unsigned size) | ||
114 | +{ | ||
115 | + UnimplementedDeviceState *s = UNIMPLEMENTED_DEVICE(opaque); | ||
116 | + | ||
117 | + qemu_log_mask(LOG_UNIMP, "%s: unimplemented device read " | ||
118 | + "(size %d, offset 0x%" HWADDR_PRIx ")\n", | ||
119 | + s->name, size, offset); | ||
120 | + return 0; | ||
121 | +} | ||
122 | + | ||
123 | +static void unimp_write(void *opaque, hwaddr offset, | ||
124 | + uint64_t value, unsigned size) | ||
125 | +{ | ||
126 | + UnimplementedDeviceState *s = UNIMPLEMENTED_DEVICE(opaque); | ||
127 | + | ||
128 | + qemu_log_mask(LOG_UNIMP, "%s: unimplemented device write " | ||
129 | + "(size %d, value 0x%" PRIx64 | ||
130 | + ", offset 0x%" HWADDR_PRIx ")\n", | ||
131 | + s->name, size, value, offset); | ||
132 | +} | ||
133 | + | ||
134 | +static const MemoryRegionOps unimp_ops = { | ||
135 | + .read = unimp_read, | ||
136 | + .write = unimp_write, | ||
137 | + .impl.min_access_size = 1, | ||
138 | + .impl.max_access_size = 8, | ||
139 | + .valid.min_access_size = 1, | ||
140 | + .valid.max_access_size = 8, | ||
141 | + .endianness = DEVICE_NATIVE_ENDIAN, | ||
142 | +}; | ||
143 | + | ||
144 | +static void unimp_realize(DeviceState *dev, Error **errp) | ||
145 | +{ | ||
146 | + UnimplementedDeviceState *s = UNIMPLEMENTED_DEVICE(dev); | ||
147 | + | ||
148 | + if (s->size == 0) { | ||
149 | + error_setg(errp, "property 'size' not specified or zero"); | ||
150 | + return; | ||
151 | + } | ||
152 | + | ||
153 | + if (s->name == NULL) { | ||
154 | + error_setg(errp, "property 'name' not specified"); | ||
155 | + return; | ||
156 | + } | ||
157 | + | ||
158 | + memory_region_init_io(&s->iomem, OBJECT(s), &unimp_ops, s, | ||
159 | + s->name, s->size); | ||
160 | + sysbus_init_mmio(SYS_BUS_DEVICE(s), &s->iomem); | ||
161 | +} | ||
162 | + | ||
163 | +static Property unimp_properties[] = { | ||
164 | + DEFINE_PROP_UINT64("size", UnimplementedDeviceState, size, 0), | ||
165 | + DEFINE_PROP_STRING("name", UnimplementedDeviceState, name), | ||
166 | + DEFINE_PROP_END_OF_LIST(), | ||
167 | +}; | ||
168 | + | ||
169 | +static void unimp_class_init(ObjectClass *klass, void *data) | ||
170 | +{ | ||
171 | + DeviceClass *dc = DEVICE_CLASS(klass); | ||
172 | + | ||
173 | + dc->realize = unimp_realize; | ||
174 | + dc->props = unimp_properties; | ||
175 | +} | ||
176 | + | ||
177 | +static const TypeInfo unimp_info = { | ||
178 | + .name = TYPE_UNIMPLEMENTED_DEVICE, | ||
179 | + .parent = TYPE_SYS_BUS_DEVICE, | ||
180 | + .instance_size = sizeof(UnimplementedDeviceState), | ||
181 | + .class_init = unimp_class_init, | ||
182 | +}; | ||
183 | + | ||
184 | +static void unimp_register_types(void) | ||
185 | +{ | ||
186 | + type_register_static(&unimp_info); | ||
187 | +} | ||
188 | + | ||
189 | +type_init(unimp_register_types) | ||
190 | -- | 59 | -- |
191 | 2.7.4 | 60 | 2.20.1 |
192 | 61 | ||
193 | 62 | diff view generated by jsdifflib |
1 | From: Julian Brown <julian@codesourcery.com> | 1 | The hw/arm/arm.h header now only includes declarations relating |
---|---|---|---|
2 | to boot.c code, so it is only needed by Arm board or SoC code. | ||
3 | Remove some unnecessary inclusions of it from target/arm files | ||
4 | and from hw/intc/armv7m_nvic.c. | ||
2 | 5 | ||
3 | In BE32 mode, sub-word size watchpoints can fail to trigger because the | 6 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
4 | address of the access is adjusted in the opcode helpers before being | 7 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
5 | compared with the watchpoint registers. This patch reverses the address | 8 | Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
6 | adjustment before performing the comparison with the help of a new CPUClass | 9 | Message-id: 20190516163857.6430-3-peter.maydell@linaro.org |
7 | hook. | 10 | --- |
11 | hw/intc/armv7m_nvic.c | 1 - | ||
12 | target/arm/arm-semi.c | 1 - | ||
13 | target/arm/cpu.c | 1 - | ||
14 | target/arm/cpu64.c | 1 - | ||
15 | target/arm/kvm.c | 1 - | ||
16 | target/arm/kvm32.c | 1 - | ||
17 | target/arm/kvm64.c | 1 - | ||
18 | 7 files changed, 7 deletions(-) | ||
8 | 19 | ||
9 | This version of the patch augments and tidies up comments a little. | 20 | diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c |
10 | |||
11 | Signed-off-by: Julian Brown <julian@codesourcery.com> | ||
12 | Message-id: caaf64ffc72f6ae183015337b7afdbd4b8989cb6.1484929304.git.julian@codesourcery.com | ||
13 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
14 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | ||
15 | --- | ||
16 | include/qom/cpu.h | 3 +++ | ||
17 | target/arm/internals.h | 5 +++++ | ||
18 | exec.c | 1 + | ||
19 | qom/cpu.c | 6 ++++++ | ||
20 | target/arm/cpu.c | 3 +++ | ||
21 | target/arm/op_helper.c | 22 ++++++++++++++++++++++ | ||
22 | 6 files changed, 40 insertions(+) | ||
23 | |||
24 | diff --git a/include/qom/cpu.h b/include/qom/cpu.h | ||
25 | index XXXXXXX..XXXXXXX 100644 | 21 | index XXXXXXX..XXXXXXX 100644 |
26 | --- a/include/qom/cpu.h | 22 | --- a/hw/intc/armv7m_nvic.c |
27 | +++ b/include/qom/cpu.h | 23 | +++ b/hw/intc/armv7m_nvic.c |
28 | @@ -XXX,XX +XXX,XX @@ struct TranslationBlock; | 24 | @@ -XXX,XX +XXX,XX @@ |
29 | * @cpu_exec_exit: Callback for cpu_exec cleanup. | 25 | #include "cpu.h" |
30 | * @cpu_exec_interrupt: Callback for processing interrupts in cpu_exec. | 26 | #include "hw/sysbus.h" |
31 | * @disas_set_info: Setup architecture specific components of disassembly info | 27 | #include "qemu/timer.h" |
32 | + * @adjust_watchpoint_address: Perform a target-specific adjustment to an | 28 | -#include "hw/arm/arm.h" |
33 | + * address before attempting to match it against watchpoints. | 29 | #include "hw/intc/armv7m_nvic.h" |
34 | * | 30 | #include "target/arm/cpu.h" |
35 | * Represents a CPU family or model. | 31 | #include "exec/exec-all.h" |
36 | */ | 32 | diff --git a/target/arm/arm-semi.c b/target/arm/arm-semi.c |
37 | @@ -XXX,XX +XXX,XX @@ typedef struct CPUClass { | ||
38 | bool (*cpu_exec_interrupt)(CPUState *cpu, int interrupt_request); | ||
39 | |||
40 | void (*disas_set_info)(CPUState *cpu, disassemble_info *info); | ||
41 | + vaddr (*adjust_watchpoint_address)(CPUState *cpu, vaddr addr, int len); | ||
42 | } CPUClass; | ||
43 | |||
44 | #ifdef HOST_WORDS_BIGENDIAN | ||
45 | diff --git a/target/arm/internals.h b/target/arm/internals.h | ||
46 | index XXXXXXX..XXXXXXX 100644 | 33 | index XXXXXXX..XXXXXXX 100644 |
47 | --- a/target/arm/internals.h | 34 | --- a/target/arm/arm-semi.c |
48 | +++ b/target/arm/internals.h | 35 | +++ b/target/arm/arm-semi.c |
49 | @@ -XXX,XX +XXX,XX @@ void hw_breakpoint_update_all(ARMCPU *cpu); | 36 | @@ -XXX,XX +XXX,XX @@ |
50 | /* Callback function for checking if a watchpoint should trigger. */ | 37 | #else |
51 | bool arm_debug_check_watchpoint(CPUState *cs, CPUWatchpoint *wp); | 38 | #include "qemu-common.h" |
52 | 39 | #include "exec/gdbstub.h" | |
53 | +/* Adjust addresses (in BE32 mode) before testing against watchpoint | 40 | -#include "hw/arm/arm.h" |
54 | + * addresses. | 41 | #include "qemu/cutils.h" |
55 | + */ | 42 | #endif |
56 | +vaddr arm_adjust_watchpoint_address(CPUState *cs, vaddr addr, int len); | 43 | |
57 | + | ||
58 | /* Callback function for when a watchpoint or breakpoint triggers. */ | ||
59 | void arm_debug_excp_handler(CPUState *cs); | ||
60 | |||
61 | diff --git a/exec.c b/exec.c | ||
62 | index XXXXXXX..XXXXXXX 100644 | ||
63 | --- a/exec.c | ||
64 | +++ b/exec.c | ||
65 | @@ -XXX,XX +XXX,XX @@ static void check_watchpoint(int offset, int len, MemTxAttrs attrs, int flags) | ||
66 | return; | ||
67 | } | ||
68 | vaddr = (cpu->mem_io_vaddr & TARGET_PAGE_MASK) + offset; | ||
69 | + vaddr = cc->adjust_watchpoint_address(cpu, vaddr, len); | ||
70 | QTAILQ_FOREACH(wp, &cpu->watchpoints, entry) { | ||
71 | if (cpu_watchpoint_address_matches(wp, vaddr, len) | ||
72 | && (wp->flags & flags)) { | ||
73 | diff --git a/qom/cpu.c b/qom/cpu.c | ||
74 | index XXXXXXX..XXXXXXX 100644 | ||
75 | --- a/qom/cpu.c | ||
76 | +++ b/qom/cpu.c | ||
77 | @@ -XXX,XX +XXX,XX @@ static int64_t cpu_common_get_arch_id(CPUState *cpu) | ||
78 | return cpu->cpu_index; | ||
79 | } | ||
80 | |||
81 | +static vaddr cpu_adjust_watchpoint_address(CPUState *cpu, vaddr addr, int len) | ||
82 | +{ | ||
83 | + return addr; | ||
84 | +} | ||
85 | + | ||
86 | static void cpu_class_init(ObjectClass *klass, void *data) | ||
87 | { | ||
88 | DeviceClass *dc = DEVICE_CLASS(klass); | ||
89 | @@ -XXX,XX +XXX,XX @@ static void cpu_class_init(ObjectClass *klass, void *data) | ||
90 | k->cpu_exec_enter = cpu_common_noop; | ||
91 | k->cpu_exec_exit = cpu_common_noop; | ||
92 | k->cpu_exec_interrupt = cpu_common_exec_interrupt; | ||
93 | + k->adjust_watchpoint_address = cpu_adjust_watchpoint_address; | ||
94 | set_bit(DEVICE_CATEGORY_CPU, dc->categories); | ||
95 | dc->realize = cpu_common_realizefn; | ||
96 | dc->unrealize = cpu_common_unrealizefn; | ||
97 | diff --git a/target/arm/cpu.c b/target/arm/cpu.c | 44 | diff --git a/target/arm/cpu.c b/target/arm/cpu.c |
98 | index XXXXXXX..XXXXXXX 100644 | 45 | index XXXXXXX..XXXXXXX 100644 |
99 | --- a/target/arm/cpu.c | 46 | --- a/target/arm/cpu.c |
100 | +++ b/target/arm/cpu.c | 47 | +++ b/target/arm/cpu.c |
101 | @@ -XXX,XX +XXX,XX @@ static void arm_cpu_class_init(ObjectClass *oc, void *data) | 48 | @@ -XXX,XX +XXX,XX @@ |
102 | cc->gdb_stop_before_watchpoint = true; | 49 | #if !defined(CONFIG_USER_ONLY) |
103 | cc->debug_excp_handler = arm_debug_excp_handler; | 50 | #include "hw/loader.h" |
104 | cc->debug_check_watchpoint = arm_debug_check_watchpoint; | 51 | #endif |
105 | +#if !defined(CONFIG_USER_ONLY) | 52 | -#include "hw/arm/arm.h" |
106 | + cc->adjust_watchpoint_address = arm_adjust_watchpoint_address; | 53 | #include "sysemu/sysemu.h" |
107 | +#endif | 54 | #include "sysemu/hw_accel.h" |
108 | 55 | #include "kvm_arm.h" | |
109 | cc->disas_set_info = arm_disas_set_info; | 56 | diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c |
110 | } | ||
111 | diff --git a/target/arm/op_helper.c b/target/arm/op_helper.c | ||
112 | index XXXXXXX..XXXXXXX 100644 | 57 | index XXXXXXX..XXXXXXX 100644 |
113 | --- a/target/arm/op_helper.c | 58 | --- a/target/arm/cpu64.c |
114 | +++ b/target/arm/op_helper.c | 59 | +++ b/target/arm/cpu64.c |
115 | @@ -XXX,XX +XXX,XX @@ bool arm_debug_check_watchpoint(CPUState *cs, CPUWatchpoint *wp) | 60 | @@ -XXX,XX +XXX,XX @@ |
116 | return check_watchpoints(cpu); | 61 | #if !defined(CONFIG_USER_ONLY) |
117 | } | 62 | #include "hw/loader.h" |
118 | 63 | #endif | |
119 | +vaddr arm_adjust_watchpoint_address(CPUState *cs, vaddr addr, int len) | 64 | -#include "hw/arm/arm.h" |
120 | +{ | 65 | #include "sysemu/sysemu.h" |
121 | + ARMCPU *cpu = ARM_CPU(cs); | 66 | #include "sysemu/kvm.h" |
122 | + CPUARMState *env = &cpu->env; | 67 | #include "kvm_arm.h" |
123 | + | 68 | diff --git a/target/arm/kvm.c b/target/arm/kvm.c |
124 | + /* In BE32 system mode, target memory is stored byteswapped (on a | 69 | index XXXXXXX..XXXXXXX 100644 |
125 | + * little-endian host system), and by the time we reach here (via an | 70 | --- a/target/arm/kvm.c |
126 | + * opcode helper) the addresses of subword accesses have been adjusted | 71 | +++ b/target/arm/kvm.c |
127 | + * to account for that, which means that watchpoints will not match. | 72 | @@ -XXX,XX +XXX,XX @@ |
128 | + * Undo the adjustment here. | 73 | #include "cpu.h" |
129 | + */ | 74 | #include "trace.h" |
130 | + if (arm_sctlr_b(env)) { | 75 | #include "internals.h" |
131 | + if (len == 1) { | 76 | -#include "hw/arm/arm.h" |
132 | + addr ^= 3; | 77 | #include "hw/pci/pci.h" |
133 | + } else if (len == 2) { | 78 | #include "exec/memattrs.h" |
134 | + addr ^= 2; | 79 | #include "exec/address-spaces.h" |
135 | + } | 80 | diff --git a/target/arm/kvm32.c b/target/arm/kvm32.c |
136 | + } | 81 | index XXXXXXX..XXXXXXX 100644 |
137 | + | 82 | --- a/target/arm/kvm32.c |
138 | + return addr; | 83 | +++ b/target/arm/kvm32.c |
139 | +} | 84 | @@ -XXX,XX +XXX,XX @@ |
140 | + | 85 | #include "sysemu/kvm.h" |
141 | void arm_debug_excp_handler(CPUState *cs) | 86 | #include "kvm_arm.h" |
142 | { | 87 | #include "internals.h" |
143 | /* Called by core code when a watchpoint or breakpoint fires; | 88 | -#include "hw/arm/arm.h" |
89 | #include "qemu/log.h" | ||
90 | |||
91 | static inline void set_feature(uint64_t *features, int feature) | ||
92 | diff --git a/target/arm/kvm64.c b/target/arm/kvm64.c | ||
93 | index XXXXXXX..XXXXXXX 100644 | ||
94 | --- a/target/arm/kvm64.c | ||
95 | +++ b/target/arm/kvm64.c | ||
96 | @@ -XXX,XX +XXX,XX @@ | ||
97 | #include "sysemu/kvm.h" | ||
98 | #include "kvm_arm.h" | ||
99 | #include "internals.h" | ||
100 | -#include "hw/arm/arm.h" | ||
101 | |||
102 | static bool have_guest_debug; | ||
103 | |||
144 | -- | 104 | -- |
145 | 2.7.4 | 105 | 2.20.1 |
146 | 106 | ||
147 | 107 | diff view generated by jsdifflib |
1 | Use the 'unimplemented' dummy device to cover regions of the | 1 | The header file hw/arm/arm.h now includes only declarations |
---|---|---|---|
2 | SoC device memory map which we don't have proper device | 2 | relating to hw/arm/boot.c functionality. Rename it accordingly, |
3 | implementations for yet. | 3 | and adjust its header comment. |
4 | |||
5 | The bulk of this commit was created via | ||
6 | perl -pi -e 's|hw/arm/arm.h|hw/arm/boot.h|' hw/arm/*.c include/hw/arm/*.h | ||
7 | |||
8 | In a few cases we can just delete the #include: | ||
9 | hw/arm/msf2-soc.c, include/hw/arm/aspeed_soc.h and | ||
10 | include/hw/arm/bcm2836.h did not require it. | ||
4 | 11 | ||
5 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 12 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
6 | Reviewed-by: Alex Bennée <alex.bennee@linaro.org> | 13 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
7 | Message-id: 1484247815-15279-4-git-send-email-peter.maydell@linaro.org | 14 | Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
15 | Message-id: 20190516163857.6430-4-peter.maydell@linaro.org | ||
8 | --- | 16 | --- |
9 | hw/arm/stellaris.c | 14 ++++++++++++++ | 17 | include/hw/arm/allwinner-a10.h | 2 +- |
10 | 1 file changed, 14 insertions(+) | 18 | include/hw/arm/aspeed_soc.h | 1 - |
19 | include/hw/arm/bcm2836.h | 1 - | ||
20 | include/hw/arm/{arm.h => boot.h} | 8 ++++---- | ||
21 | include/hw/arm/fsl-imx25.h | 2 +- | ||
22 | include/hw/arm/fsl-imx31.h | 2 +- | ||
23 | include/hw/arm/fsl-imx6.h | 2 +- | ||
24 | include/hw/arm/fsl-imx6ul.h | 2 +- | ||
25 | include/hw/arm/fsl-imx7.h | 2 +- | ||
26 | include/hw/arm/virt.h | 2 +- | ||
27 | include/hw/arm/xlnx-versal.h | 2 +- | ||
28 | include/hw/arm/xlnx-zynqmp.h | 2 +- | ||
29 | hw/arm/armsse.c | 2 +- | ||
30 | hw/arm/armv7m.c | 2 +- | ||
31 | hw/arm/aspeed.c | 2 +- | ||
32 | hw/arm/boot.c | 2 +- | ||
33 | hw/arm/collie.c | 2 +- | ||
34 | hw/arm/exynos4210.c | 2 +- | ||
35 | hw/arm/exynos4_boards.c | 2 +- | ||
36 | hw/arm/highbank.c | 2 +- | ||
37 | hw/arm/integratorcp.c | 2 +- | ||
38 | hw/arm/mainstone.c | 2 +- | ||
39 | hw/arm/microbit.c | 2 +- | ||
40 | hw/arm/mps2-tz.c | 2 +- | ||
41 | hw/arm/mps2.c | 2 +- | ||
42 | hw/arm/msf2-soc.c | 1 - | ||
43 | hw/arm/msf2-som.c | 2 +- | ||
44 | hw/arm/musca.c | 2 +- | ||
45 | hw/arm/musicpal.c | 2 +- | ||
46 | hw/arm/netduino2.c | 2 +- | ||
47 | hw/arm/nrf51_soc.c | 2 +- | ||
48 | hw/arm/nseries.c | 2 +- | ||
49 | hw/arm/omap1.c | 2 +- | ||
50 | hw/arm/omap2.c | 2 +- | ||
51 | hw/arm/omap_sx1.c | 2 +- | ||
52 | hw/arm/palm.c | 2 +- | ||
53 | hw/arm/raspi.c | 2 +- | ||
54 | hw/arm/realview.c | 2 +- | ||
55 | hw/arm/spitz.c | 2 +- | ||
56 | hw/arm/stellaris.c | 2 +- | ||
57 | hw/arm/stm32f205_soc.c | 2 +- | ||
58 | hw/arm/strongarm.c | 2 +- | ||
59 | hw/arm/tosa.c | 2 +- | ||
60 | hw/arm/versatilepb.c | 2 +- | ||
61 | hw/arm/vexpress.c | 2 +- | ||
62 | hw/arm/virt.c | 2 +- | ||
63 | hw/arm/xilinx_zynq.c | 2 +- | ||
64 | hw/arm/xlnx-versal.c | 2 +- | ||
65 | hw/arm/z2.c | 2 +- | ||
66 | 49 files changed, 49 insertions(+), 52 deletions(-) | ||
67 | rename include/hw/arm/{arm.h => boot.h} (98%) | ||
11 | 68 | ||
69 | diff --git a/include/hw/arm/allwinner-a10.h b/include/hw/arm/allwinner-a10.h | ||
70 | index XXXXXXX..XXXXXXX 100644 | ||
71 | --- a/include/hw/arm/allwinner-a10.h | ||
72 | +++ b/include/hw/arm/allwinner-a10.h | ||
73 | @@ -XXX,XX +XXX,XX @@ | ||
74 | #include "qemu-common.h" | ||
75 | #include "qemu/error-report.h" | ||
76 | #include "hw/char/serial.h" | ||
77 | -#include "hw/arm/arm.h" | ||
78 | +#include "hw/arm/boot.h" | ||
79 | #include "hw/timer/allwinner-a10-pit.h" | ||
80 | #include "hw/intc/allwinner-a10-pic.h" | ||
81 | #include "hw/net/allwinner_emac.h" | ||
82 | diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h | ||
83 | index XXXXXXX..XXXXXXX 100644 | ||
84 | --- a/include/hw/arm/aspeed_soc.h | ||
85 | +++ b/include/hw/arm/aspeed_soc.h | ||
86 | @@ -XXX,XX +XXX,XX @@ | ||
87 | #ifndef ASPEED_SOC_H | ||
88 | #define ASPEED_SOC_H | ||
89 | |||
90 | -#include "hw/arm/arm.h" | ||
91 | #include "hw/intc/aspeed_vic.h" | ||
92 | #include "hw/misc/aspeed_scu.h" | ||
93 | #include "hw/misc/aspeed_sdmc.h" | ||
94 | diff --git a/include/hw/arm/bcm2836.h b/include/hw/arm/bcm2836.h | ||
95 | index XXXXXXX..XXXXXXX 100644 | ||
96 | --- a/include/hw/arm/bcm2836.h | ||
97 | +++ b/include/hw/arm/bcm2836.h | ||
98 | @@ -XXX,XX +XXX,XX @@ | ||
99 | #ifndef BCM2836_H | ||
100 | #define BCM2836_H | ||
101 | |||
102 | -#include "hw/arm/arm.h" | ||
103 | #include "hw/arm/bcm2835_peripherals.h" | ||
104 | #include "hw/intc/bcm2836_control.h" | ||
105 | |||
106 | diff --git a/include/hw/arm/arm.h b/include/hw/arm/boot.h | ||
107 | similarity index 98% | ||
108 | rename from include/hw/arm/arm.h | ||
109 | rename to include/hw/arm/boot.h | ||
110 | index XXXXXXX..XXXXXXX 100644 | ||
111 | --- a/include/hw/arm/arm.h | ||
112 | +++ b/include/hw/arm/boot.h | ||
113 | @@ -XXX,XX +XXX,XX @@ | ||
114 | /* | ||
115 | - * Misc ARM declarations | ||
116 | + * ARM kernel loader. | ||
117 | * | ||
118 | * Copyright (c) 2006 CodeSourcery. | ||
119 | * Written by Paul Brook | ||
120 | @@ -XXX,XX +XXX,XX @@ | ||
121 | * | ||
122 | */ | ||
123 | |||
124 | -#ifndef HW_ARM_H | ||
125 | -#define HW_ARM_H | ||
126 | +#ifndef HW_ARM_BOOT_H | ||
127 | +#define HW_ARM_BOOT_H | ||
128 | |||
129 | #include "exec/memory.h" | ||
130 | #include "target/arm/cpu-qom.h" | ||
131 | @@ -XXX,XX +XXX,XX @@ void arm_write_secure_board_setup_dummy_smc(ARMCPU *cpu, | ||
132 | const struct arm_boot_info *info, | ||
133 | hwaddr mvbar_addr); | ||
134 | |||
135 | -#endif /* HW_ARM_H */ | ||
136 | +#endif /* HW_ARM_BOOT_H */ | ||
137 | diff --git a/include/hw/arm/fsl-imx25.h b/include/hw/arm/fsl-imx25.h | ||
138 | index XXXXXXX..XXXXXXX 100644 | ||
139 | --- a/include/hw/arm/fsl-imx25.h | ||
140 | +++ b/include/hw/arm/fsl-imx25.h | ||
141 | @@ -XXX,XX +XXX,XX @@ | ||
142 | #ifndef FSL_IMX25_H | ||
143 | #define FSL_IMX25_H | ||
144 | |||
145 | -#include "hw/arm/arm.h" | ||
146 | +#include "hw/arm/boot.h" | ||
147 | #include "hw/intc/imx_avic.h" | ||
148 | #include "hw/misc/imx25_ccm.h" | ||
149 | #include "hw/char/imx_serial.h" | ||
150 | diff --git a/include/hw/arm/fsl-imx31.h b/include/hw/arm/fsl-imx31.h | ||
151 | index XXXXXXX..XXXXXXX 100644 | ||
152 | --- a/include/hw/arm/fsl-imx31.h | ||
153 | +++ b/include/hw/arm/fsl-imx31.h | ||
154 | @@ -XXX,XX +XXX,XX @@ | ||
155 | #ifndef FSL_IMX31_H | ||
156 | #define FSL_IMX31_H | ||
157 | |||
158 | -#include "hw/arm/arm.h" | ||
159 | +#include "hw/arm/boot.h" | ||
160 | #include "hw/intc/imx_avic.h" | ||
161 | #include "hw/misc/imx31_ccm.h" | ||
162 | #include "hw/char/imx_serial.h" | ||
163 | diff --git a/include/hw/arm/fsl-imx6.h b/include/hw/arm/fsl-imx6.h | ||
164 | index XXXXXXX..XXXXXXX 100644 | ||
165 | --- a/include/hw/arm/fsl-imx6.h | ||
166 | +++ b/include/hw/arm/fsl-imx6.h | ||
167 | @@ -XXX,XX +XXX,XX @@ | ||
168 | #ifndef FSL_IMX6_H | ||
169 | #define FSL_IMX6_H | ||
170 | |||
171 | -#include "hw/arm/arm.h" | ||
172 | +#include "hw/arm/boot.h" | ||
173 | #include "hw/cpu/a9mpcore.h" | ||
174 | #include "hw/misc/imx6_ccm.h" | ||
175 | #include "hw/misc/imx6_src.h" | ||
176 | diff --git a/include/hw/arm/fsl-imx6ul.h b/include/hw/arm/fsl-imx6ul.h | ||
177 | index XXXXXXX..XXXXXXX 100644 | ||
178 | --- a/include/hw/arm/fsl-imx6ul.h | ||
179 | +++ b/include/hw/arm/fsl-imx6ul.h | ||
180 | @@ -XXX,XX +XXX,XX @@ | ||
181 | #ifndef FSL_IMX6UL_H | ||
182 | #define FSL_IMX6UL_H | ||
183 | |||
184 | -#include "hw/arm/arm.h" | ||
185 | +#include "hw/arm/boot.h" | ||
186 | #include "hw/cpu/a15mpcore.h" | ||
187 | #include "hw/misc/imx6ul_ccm.h" | ||
188 | #include "hw/misc/imx6_src.h" | ||
189 | diff --git a/include/hw/arm/fsl-imx7.h b/include/hw/arm/fsl-imx7.h | ||
190 | index XXXXXXX..XXXXXXX 100644 | ||
191 | --- a/include/hw/arm/fsl-imx7.h | ||
192 | +++ b/include/hw/arm/fsl-imx7.h | ||
193 | @@ -XXX,XX +XXX,XX @@ | ||
194 | #ifndef FSL_IMX7_H | ||
195 | #define FSL_IMX7_H | ||
196 | |||
197 | -#include "hw/arm/arm.h" | ||
198 | +#include "hw/arm/boot.h" | ||
199 | #include "hw/cpu/a15mpcore.h" | ||
200 | #include "hw/intc/imx_gpcv2.h" | ||
201 | #include "hw/misc/imx7_ccm.h" | ||
202 | diff --git a/include/hw/arm/virt.h b/include/hw/arm/virt.h | ||
203 | index XXXXXXX..XXXXXXX 100644 | ||
204 | --- a/include/hw/arm/virt.h | ||
205 | +++ b/include/hw/arm/virt.h | ||
206 | @@ -XXX,XX +XXX,XX @@ | ||
207 | #include "exec/hwaddr.h" | ||
208 | #include "qemu/notify.h" | ||
209 | #include "hw/boards.h" | ||
210 | -#include "hw/arm/arm.h" | ||
211 | +#include "hw/arm/boot.h" | ||
212 | #include "hw/block/flash.h" | ||
213 | #include "sysemu/kvm.h" | ||
214 | #include "hw/intc/arm_gicv3_common.h" | ||
215 | diff --git a/include/hw/arm/xlnx-versal.h b/include/hw/arm/xlnx-versal.h | ||
216 | index XXXXXXX..XXXXXXX 100644 | ||
217 | --- a/include/hw/arm/xlnx-versal.h | ||
218 | +++ b/include/hw/arm/xlnx-versal.h | ||
219 | @@ -XXX,XX +XXX,XX @@ | ||
220 | #define XLNX_VERSAL_H | ||
221 | |||
222 | #include "hw/sysbus.h" | ||
223 | -#include "hw/arm/arm.h" | ||
224 | +#include "hw/arm/boot.h" | ||
225 | #include "hw/intc/arm_gicv3.h" | ||
226 | |||
227 | #define TYPE_XLNX_VERSAL "xlnx-versal" | ||
228 | diff --git a/include/hw/arm/xlnx-zynqmp.h b/include/hw/arm/xlnx-zynqmp.h | ||
229 | index XXXXXXX..XXXXXXX 100644 | ||
230 | --- a/include/hw/arm/xlnx-zynqmp.h | ||
231 | +++ b/include/hw/arm/xlnx-zynqmp.h | ||
232 | @@ -XXX,XX +XXX,XX @@ | ||
233 | #ifndef XLNX_ZYNQMP_H | ||
234 | |||
235 | #include "qemu-common.h" | ||
236 | -#include "hw/arm/arm.h" | ||
237 | +#include "hw/arm/boot.h" | ||
238 | #include "hw/intc/arm_gic.h" | ||
239 | #include "hw/net/cadence_gem.h" | ||
240 | #include "hw/char/cadence_uart.h" | ||
241 | diff --git a/hw/arm/armsse.c b/hw/arm/armsse.c | ||
242 | index XXXXXXX..XXXXXXX 100644 | ||
243 | --- a/hw/arm/armsse.c | ||
244 | +++ b/hw/arm/armsse.c | ||
245 | @@ -XXX,XX +XXX,XX @@ | ||
246 | #include "hw/sysbus.h" | ||
247 | #include "hw/registerfields.h" | ||
248 | #include "hw/arm/armsse.h" | ||
249 | -#include "hw/arm/arm.h" | ||
250 | +#include "hw/arm/boot.h" | ||
251 | |||
252 | /* Format of the System Information block SYS_CONFIG register */ | ||
253 | typedef enum SysConfigFormat { | ||
254 | diff --git a/hw/arm/armv7m.c b/hw/arm/armv7m.c | ||
255 | index XXXXXXX..XXXXXXX 100644 | ||
256 | --- a/hw/arm/armv7m.c | ||
257 | +++ b/hw/arm/armv7m.c | ||
258 | @@ -XXX,XX +XXX,XX @@ | ||
259 | #include "qemu-common.h" | ||
260 | #include "cpu.h" | ||
261 | #include "hw/sysbus.h" | ||
262 | -#include "hw/arm/arm.h" | ||
263 | +#include "hw/arm/boot.h" | ||
264 | #include "hw/loader.h" | ||
265 | #include "elf.h" | ||
266 | #include "sysemu/qtest.h" | ||
267 | diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c | ||
268 | index XXXXXXX..XXXXXXX 100644 | ||
269 | --- a/hw/arm/aspeed.c | ||
270 | +++ b/hw/arm/aspeed.c | ||
271 | @@ -XXX,XX +XXX,XX @@ | ||
272 | #include "qemu-common.h" | ||
273 | #include "cpu.h" | ||
274 | #include "exec/address-spaces.h" | ||
275 | -#include "hw/arm/arm.h" | ||
276 | +#include "hw/arm/boot.h" | ||
277 | #include "hw/arm/aspeed.h" | ||
278 | #include "hw/arm/aspeed_soc.h" | ||
279 | #include "hw/boards.h" | ||
280 | diff --git a/hw/arm/boot.c b/hw/arm/boot.c | ||
281 | index XXXXXXX..XXXXXXX 100644 | ||
282 | --- a/hw/arm/boot.c | ||
283 | +++ b/hw/arm/boot.c | ||
284 | @@ -XXX,XX +XXX,XX @@ | ||
285 | #include "qapi/error.h" | ||
286 | #include <libfdt.h> | ||
287 | #include "hw/hw.h" | ||
288 | -#include "hw/arm/arm.h" | ||
289 | +#include "hw/arm/boot.h" | ||
290 | #include "hw/arm/linux-boot-if.h" | ||
291 | #include "sysemu/kvm.h" | ||
292 | #include "sysemu/sysemu.h" | ||
293 | diff --git a/hw/arm/collie.c b/hw/arm/collie.c | ||
294 | index XXXXXXX..XXXXXXX 100644 | ||
295 | --- a/hw/arm/collie.c | ||
296 | +++ b/hw/arm/collie.c | ||
297 | @@ -XXX,XX +XXX,XX @@ | ||
298 | #include "hw/sysbus.h" | ||
299 | #include "hw/boards.h" | ||
300 | #include "strongarm.h" | ||
301 | -#include "hw/arm/arm.h" | ||
302 | +#include "hw/arm/boot.h" | ||
303 | #include "hw/block/flash.h" | ||
304 | #include "exec/address-spaces.h" | ||
305 | #include "cpu.h" | ||
306 | diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c | ||
307 | index XXXXXXX..XXXXXXX 100644 | ||
308 | --- a/hw/arm/exynos4210.c | ||
309 | +++ b/hw/arm/exynos4210.c | ||
310 | @@ -XXX,XX +XXX,XX @@ | ||
311 | #include "hw/boards.h" | ||
312 | #include "sysemu/sysemu.h" | ||
313 | #include "hw/sysbus.h" | ||
314 | -#include "hw/arm/arm.h" | ||
315 | +#include "hw/arm/boot.h" | ||
316 | #include "hw/loader.h" | ||
317 | #include "hw/arm/exynos4210.h" | ||
318 | #include "hw/sd/sdhci.h" | ||
319 | diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c | ||
320 | index XXXXXXX..XXXXXXX 100644 | ||
321 | --- a/hw/arm/exynos4_boards.c | ||
322 | +++ b/hw/arm/exynos4_boards.c | ||
323 | @@ -XXX,XX +XXX,XX @@ | ||
324 | #include "sysemu/sysemu.h" | ||
325 | #include "hw/sysbus.h" | ||
326 | #include "net/net.h" | ||
327 | -#include "hw/arm/arm.h" | ||
328 | +#include "hw/arm/boot.h" | ||
329 | #include "exec/address-spaces.h" | ||
330 | #include "hw/arm/exynos4210.h" | ||
331 | #include "hw/net/lan9118.h" | ||
332 | diff --git a/hw/arm/highbank.c b/hw/arm/highbank.c | ||
333 | index XXXXXXX..XXXXXXX 100644 | ||
334 | --- a/hw/arm/highbank.c | ||
335 | +++ b/hw/arm/highbank.c | ||
336 | @@ -XXX,XX +XXX,XX @@ | ||
337 | #include "qemu/osdep.h" | ||
338 | #include "qapi/error.h" | ||
339 | #include "hw/sysbus.h" | ||
340 | -#include "hw/arm/arm.h" | ||
341 | +#include "hw/arm/boot.h" | ||
342 | #include "hw/loader.h" | ||
343 | #include "net/net.h" | ||
344 | #include "sysemu/kvm.h" | ||
345 | diff --git a/hw/arm/integratorcp.c b/hw/arm/integratorcp.c | ||
346 | index XXXXXXX..XXXXXXX 100644 | ||
347 | --- a/hw/arm/integratorcp.c | ||
348 | +++ b/hw/arm/integratorcp.c | ||
349 | @@ -XXX,XX +XXX,XX @@ | ||
350 | #include "cpu.h" | ||
351 | #include "hw/sysbus.h" | ||
352 | #include "hw/boards.h" | ||
353 | -#include "hw/arm/arm.h" | ||
354 | +#include "hw/arm/boot.h" | ||
355 | #include "hw/misc/arm_integrator_debug.h" | ||
356 | #include "hw/net/smc91c111.h" | ||
357 | #include "net/net.h" | ||
358 | diff --git a/hw/arm/mainstone.c b/hw/arm/mainstone.c | ||
359 | index XXXXXXX..XXXXXXX 100644 | ||
360 | --- a/hw/arm/mainstone.c | ||
361 | +++ b/hw/arm/mainstone.c | ||
362 | @@ -XXX,XX +XXX,XX @@ | ||
363 | #include "qapi/error.h" | ||
364 | #include "hw/hw.h" | ||
365 | #include "hw/arm/pxa.h" | ||
366 | -#include "hw/arm/arm.h" | ||
367 | +#include "hw/arm/boot.h" | ||
368 | #include "net/net.h" | ||
369 | #include "hw/net/smc91c111.h" | ||
370 | #include "hw/boards.h" | ||
371 | diff --git a/hw/arm/microbit.c b/hw/arm/microbit.c | ||
372 | index XXXXXXX..XXXXXXX 100644 | ||
373 | --- a/hw/arm/microbit.c | ||
374 | +++ b/hw/arm/microbit.c | ||
375 | @@ -XXX,XX +XXX,XX @@ | ||
376 | #include "qemu/osdep.h" | ||
377 | #include "qapi/error.h" | ||
378 | #include "hw/boards.h" | ||
379 | -#include "hw/arm/arm.h" | ||
380 | +#include "hw/arm/boot.h" | ||
381 | #include "sysemu/sysemu.h" | ||
382 | #include "exec/address-spaces.h" | ||
383 | |||
384 | diff --git a/hw/arm/mps2-tz.c b/hw/arm/mps2-tz.c | ||
385 | index XXXXXXX..XXXXXXX 100644 | ||
386 | --- a/hw/arm/mps2-tz.c | ||
387 | +++ b/hw/arm/mps2-tz.c | ||
388 | @@ -XXX,XX +XXX,XX @@ | ||
389 | #include "qemu/osdep.h" | ||
390 | #include "qapi/error.h" | ||
391 | #include "qemu/error-report.h" | ||
392 | -#include "hw/arm/arm.h" | ||
393 | +#include "hw/arm/boot.h" | ||
394 | #include "hw/arm/armv7m.h" | ||
395 | #include "hw/or-irq.h" | ||
396 | #include "hw/boards.h" | ||
397 | diff --git a/hw/arm/mps2.c b/hw/arm/mps2.c | ||
398 | index XXXXXXX..XXXXXXX 100644 | ||
399 | --- a/hw/arm/mps2.c | ||
400 | +++ b/hw/arm/mps2.c | ||
401 | @@ -XXX,XX +XXX,XX @@ | ||
402 | #include "qemu/osdep.h" | ||
403 | #include "qapi/error.h" | ||
404 | #include "qemu/error-report.h" | ||
405 | -#include "hw/arm/arm.h" | ||
406 | +#include "hw/arm/boot.h" | ||
407 | #include "hw/arm/armv7m.h" | ||
408 | #include "hw/or-irq.h" | ||
409 | #include "hw/boards.h" | ||
410 | diff --git a/hw/arm/msf2-soc.c b/hw/arm/msf2-soc.c | ||
411 | index XXXXXXX..XXXXXXX 100644 | ||
412 | --- a/hw/arm/msf2-soc.c | ||
413 | +++ b/hw/arm/msf2-soc.c | ||
414 | @@ -XXX,XX +XXX,XX @@ | ||
415 | #include "qemu/units.h" | ||
416 | #include "qapi/error.h" | ||
417 | #include "qemu-common.h" | ||
418 | -#include "hw/arm/arm.h" | ||
419 | #include "exec/address-spaces.h" | ||
420 | #include "hw/char/serial.h" | ||
421 | #include "hw/boards.h" | ||
422 | diff --git a/hw/arm/msf2-som.c b/hw/arm/msf2-som.c | ||
423 | index XXXXXXX..XXXXXXX 100644 | ||
424 | --- a/hw/arm/msf2-som.c | ||
425 | +++ b/hw/arm/msf2-som.c | ||
426 | @@ -XXX,XX +XXX,XX @@ | ||
427 | #include "qapi/error.h" | ||
428 | #include "qemu/error-report.h" | ||
429 | #include "hw/boards.h" | ||
430 | -#include "hw/arm/arm.h" | ||
431 | +#include "hw/arm/boot.h" | ||
432 | #include "exec/address-spaces.h" | ||
433 | #include "hw/arm/msf2-soc.h" | ||
434 | #include "cpu.h" | ||
435 | diff --git a/hw/arm/musca.c b/hw/arm/musca.c | ||
436 | index XXXXXXX..XXXXXXX 100644 | ||
437 | --- a/hw/arm/musca.c | ||
438 | +++ b/hw/arm/musca.c | ||
439 | @@ -XXX,XX +XXX,XX @@ | ||
440 | #include "qapi/error.h" | ||
441 | #include "exec/address-spaces.h" | ||
442 | #include "sysemu/sysemu.h" | ||
443 | -#include "hw/arm/arm.h" | ||
444 | +#include "hw/arm/boot.h" | ||
445 | #include "hw/arm/armsse.h" | ||
446 | #include "hw/boards.h" | ||
447 | #include "hw/char/pl011.h" | ||
448 | diff --git a/hw/arm/musicpal.c b/hw/arm/musicpal.c | ||
449 | index XXXXXXX..XXXXXXX 100644 | ||
450 | --- a/hw/arm/musicpal.c | ||
451 | +++ b/hw/arm/musicpal.c | ||
452 | @@ -XXX,XX +XXX,XX @@ | ||
453 | #include "qemu-common.h" | ||
454 | #include "cpu.h" | ||
455 | #include "hw/sysbus.h" | ||
456 | -#include "hw/arm/arm.h" | ||
457 | +#include "hw/arm/boot.h" | ||
458 | #include "net/net.h" | ||
459 | #include "sysemu/sysemu.h" | ||
460 | #include "hw/boards.h" | ||
461 | diff --git a/hw/arm/netduino2.c b/hw/arm/netduino2.c | ||
462 | index XXXXXXX..XXXXXXX 100644 | ||
463 | --- a/hw/arm/netduino2.c | ||
464 | +++ b/hw/arm/netduino2.c | ||
465 | @@ -XXX,XX +XXX,XX @@ | ||
466 | #include "hw/boards.h" | ||
467 | #include "qemu/error-report.h" | ||
468 | #include "hw/arm/stm32f205_soc.h" | ||
469 | -#include "hw/arm/arm.h" | ||
470 | +#include "hw/arm/boot.h" | ||
471 | |||
472 | static void netduino2_init(MachineState *machine) | ||
473 | { | ||
474 | diff --git a/hw/arm/nrf51_soc.c b/hw/arm/nrf51_soc.c | ||
475 | index XXXXXXX..XXXXXXX 100644 | ||
476 | --- a/hw/arm/nrf51_soc.c | ||
477 | +++ b/hw/arm/nrf51_soc.c | ||
478 | @@ -XXX,XX +XXX,XX @@ | ||
479 | #include "qemu/osdep.h" | ||
480 | #include "qapi/error.h" | ||
481 | #include "qemu-common.h" | ||
482 | -#include "hw/arm/arm.h" | ||
483 | +#include "hw/arm/boot.h" | ||
484 | #include "hw/sysbus.h" | ||
485 | #include "hw/boards.h" | ||
486 | #include "hw/misc/unimp.h" | ||
487 | diff --git a/hw/arm/nseries.c b/hw/arm/nseries.c | ||
488 | index XXXXXXX..XXXXXXX 100644 | ||
489 | --- a/hw/arm/nseries.c | ||
490 | +++ b/hw/arm/nseries.c | ||
491 | @@ -XXX,XX +XXX,XX @@ | ||
492 | #include "qemu/bswap.h" | ||
493 | #include "sysemu/sysemu.h" | ||
494 | #include "hw/arm/omap.h" | ||
495 | -#include "hw/arm/arm.h" | ||
496 | +#include "hw/arm/boot.h" | ||
497 | #include "hw/irq.h" | ||
498 | #include "ui/console.h" | ||
499 | #include "hw/boards.h" | ||
500 | diff --git a/hw/arm/omap1.c b/hw/arm/omap1.c | ||
501 | index XXXXXXX..XXXXXXX 100644 | ||
502 | --- a/hw/arm/omap1.c | ||
503 | +++ b/hw/arm/omap1.c | ||
504 | @@ -XXX,XX +XXX,XX @@ | ||
505 | #include "cpu.h" | ||
506 | #include "hw/boards.h" | ||
507 | #include "hw/hw.h" | ||
508 | -#include "hw/arm/arm.h" | ||
509 | +#include "hw/arm/boot.h" | ||
510 | #include "hw/arm/omap.h" | ||
511 | #include "sysemu/sysemu.h" | ||
512 | #include "hw/arm/soc_dma.h" | ||
513 | diff --git a/hw/arm/omap2.c b/hw/arm/omap2.c | ||
514 | index XXXXXXX..XXXXXXX 100644 | ||
515 | --- a/hw/arm/omap2.c | ||
516 | +++ b/hw/arm/omap2.c | ||
517 | @@ -XXX,XX +XXX,XX @@ | ||
518 | #include "sysemu/qtest.h" | ||
519 | #include "hw/boards.h" | ||
520 | #include "hw/hw.h" | ||
521 | -#include "hw/arm/arm.h" | ||
522 | +#include "hw/arm/boot.h" | ||
523 | #include "hw/arm/omap.h" | ||
524 | #include "sysemu/sysemu.h" | ||
525 | #include "qemu/timer.h" | ||
526 | diff --git a/hw/arm/omap_sx1.c b/hw/arm/omap_sx1.c | ||
527 | index XXXXXXX..XXXXXXX 100644 | ||
528 | --- a/hw/arm/omap_sx1.c | ||
529 | +++ b/hw/arm/omap_sx1.c | ||
530 | @@ -XXX,XX +XXX,XX @@ | ||
531 | #include "ui/console.h" | ||
532 | #include "hw/arm/omap.h" | ||
533 | #include "hw/boards.h" | ||
534 | -#include "hw/arm/arm.h" | ||
535 | +#include "hw/arm/boot.h" | ||
536 | #include "hw/block/flash.h" | ||
537 | #include "sysemu/qtest.h" | ||
538 | #include "exec/address-spaces.h" | ||
539 | diff --git a/hw/arm/palm.c b/hw/arm/palm.c | ||
540 | index XXXXXXX..XXXXXXX 100644 | ||
541 | --- a/hw/arm/palm.c | ||
542 | +++ b/hw/arm/palm.c | ||
543 | @@ -XXX,XX +XXX,XX @@ | ||
544 | #include "ui/console.h" | ||
545 | #include "hw/arm/omap.h" | ||
546 | #include "hw/boards.h" | ||
547 | -#include "hw/arm/arm.h" | ||
548 | +#include "hw/arm/boot.h" | ||
549 | #include "hw/input/tsc2xxx.h" | ||
550 | #include "hw/loader.h" | ||
551 | #include "exec/address-spaces.h" | ||
552 | diff --git a/hw/arm/raspi.c b/hw/arm/raspi.c | ||
553 | index XXXXXXX..XXXXXXX 100644 | ||
554 | --- a/hw/arm/raspi.c | ||
555 | +++ b/hw/arm/raspi.c | ||
556 | @@ -XXX,XX +XXX,XX @@ | ||
557 | #include "qemu/error-report.h" | ||
558 | #include "hw/boards.h" | ||
559 | #include "hw/loader.h" | ||
560 | -#include "hw/arm/arm.h" | ||
561 | +#include "hw/arm/boot.h" | ||
562 | #include "sysemu/sysemu.h" | ||
563 | |||
564 | #define SMPBOOT_ADDR 0x300 /* this should leave enough space for ATAGS */ | ||
565 | diff --git a/hw/arm/realview.c b/hw/arm/realview.c | ||
566 | index XXXXXXX..XXXXXXX 100644 | ||
567 | --- a/hw/arm/realview.c | ||
568 | +++ b/hw/arm/realview.c | ||
569 | @@ -XXX,XX +XXX,XX @@ | ||
570 | #include "qemu-common.h" | ||
571 | #include "cpu.h" | ||
572 | #include "hw/sysbus.h" | ||
573 | -#include "hw/arm/arm.h" | ||
574 | +#include "hw/arm/boot.h" | ||
575 | #include "hw/arm/primecell.h" | ||
576 | #include "hw/net/lan9118.h" | ||
577 | #include "hw/net/smc91c111.h" | ||
578 | diff --git a/hw/arm/spitz.c b/hw/arm/spitz.c | ||
579 | index XXXXXXX..XXXXXXX 100644 | ||
580 | --- a/hw/arm/spitz.c | ||
581 | +++ b/hw/arm/spitz.c | ||
582 | @@ -XXX,XX +XXX,XX @@ | ||
583 | #include "qapi/error.h" | ||
584 | #include "hw/hw.h" | ||
585 | #include "hw/arm/pxa.h" | ||
586 | -#include "hw/arm/arm.h" | ||
587 | +#include "hw/arm/boot.h" | ||
588 | #include "sysemu/sysemu.h" | ||
589 | #include "hw/pcmcia.h" | ||
590 | #include "hw/i2c/i2c.h" | ||
12 | diff --git a/hw/arm/stellaris.c b/hw/arm/stellaris.c | 591 | diff --git a/hw/arm/stellaris.c b/hw/arm/stellaris.c |
13 | index XXXXXXX..XXXXXXX 100644 | 592 | index XXXXXXX..XXXXXXX 100644 |
14 | --- a/hw/arm/stellaris.c | 593 | --- a/hw/arm/stellaris.c |
15 | +++ b/hw/arm/stellaris.c | 594 | +++ b/hw/arm/stellaris.c |
16 | @@ -XXX,XX +XXX,XX @@ | 595 | @@ -XXX,XX +XXX,XX @@ |
17 | #include "exec/address-spaces.h" | 596 | #include "qapi/error.h" |
18 | #include "sysemu/sysemu.h" | 597 | #include "hw/sysbus.h" |
19 | #include "hw/char/pl011.h" | 598 | #include "hw/ssi/ssi.h" |
20 | +#include "hw/misc/unimp.h" | 599 | -#include "hw/arm/arm.h" |
21 | 600 | +#include "hw/arm/boot.h" | |
22 | #define GPIO_A 0 | 601 | #include "qemu/timer.h" |
23 | #define GPIO_B 1 | 602 | #include "hw/i2c/i2c.h" |
24 | @@ -XXX,XX +XXX,XX @@ static void stellaris_init(const char *kernel_filename, const char *cpu_model, | 603 | #include "net/net.h" |
25 | } | 604 | diff --git a/hw/arm/stm32f205_soc.c b/hw/arm/stm32f205_soc.c |
26 | } | 605 | index XXXXXXX..XXXXXXX 100644 |
27 | } | 606 | --- a/hw/arm/stm32f205_soc.c |
28 | + | 607 | +++ b/hw/arm/stm32f205_soc.c |
29 | + /* Add dummy regions for the devices we don't implement yet, | 608 | @@ -XXX,XX +XXX,XX @@ |
30 | + * so guest accesses don't cause unlogged crashes. | 609 | #include "qemu/osdep.h" |
31 | + */ | 610 | #include "qapi/error.h" |
32 | + create_unimplemented_device("wdtimer", 0x40000000, 0x1000); | 611 | #include "qemu-common.h" |
33 | + create_unimplemented_device("i2c-0", 0x40002000, 0x1000); | 612 | -#include "hw/arm/arm.h" |
34 | + create_unimplemented_device("i2c-2", 0x40021000, 0x1000); | 613 | +#include "hw/arm/boot.h" |
35 | + create_unimplemented_device("PWM", 0x40028000, 0x1000); | 614 | #include "exec/address-spaces.h" |
36 | + create_unimplemented_device("QEI-0", 0x4002c000, 0x1000); | 615 | #include "hw/arm/stm32f205_soc.h" |
37 | + create_unimplemented_device("QEI-1", 0x4002d000, 0x1000); | 616 | |
38 | + create_unimplemented_device("analogue-comparator", 0x4003c000, 0x1000); | 617 | diff --git a/hw/arm/strongarm.c b/hw/arm/strongarm.c |
39 | + create_unimplemented_device("hibernation", 0x400fc000, 0x1000); | 618 | index XXXXXXX..XXXXXXX 100644 |
40 | + create_unimplemented_device("flash-control", 0x400fd000, 0x1000); | 619 | --- a/hw/arm/strongarm.c |
41 | } | 620 | +++ b/hw/arm/strongarm.c |
42 | 621 | @@ -XXX,XX +XXX,XX @@ | |
43 | /* FIXME: Figure out how to generate these from stellaris_boards. */ | 622 | #include "hw/sysbus.h" |
623 | #include "strongarm.h" | ||
624 | #include "qemu/error-report.h" | ||
625 | -#include "hw/arm/arm.h" | ||
626 | +#include "hw/arm/boot.h" | ||
627 | #include "chardev/char-fe.h" | ||
628 | #include "chardev/char-serial.h" | ||
629 | #include "sysemu/sysemu.h" | ||
630 | diff --git a/hw/arm/tosa.c b/hw/arm/tosa.c | ||
631 | index XXXXXXX..XXXXXXX 100644 | ||
632 | --- a/hw/arm/tosa.c | ||
633 | +++ b/hw/arm/tosa.c | ||
634 | @@ -XXX,XX +XXX,XX @@ | ||
635 | #include "qapi/error.h" | ||
636 | #include "hw/hw.h" | ||
637 | #include "hw/arm/pxa.h" | ||
638 | -#include "hw/arm/arm.h" | ||
639 | +#include "hw/arm/boot.h" | ||
640 | #include "hw/arm/sharpsl.h" | ||
641 | #include "hw/pcmcia.h" | ||
642 | #include "hw/boards.h" | ||
643 | diff --git a/hw/arm/versatilepb.c b/hw/arm/versatilepb.c | ||
644 | index XXXXXXX..XXXXXXX 100644 | ||
645 | --- a/hw/arm/versatilepb.c | ||
646 | +++ b/hw/arm/versatilepb.c | ||
647 | @@ -XXX,XX +XXX,XX @@ | ||
648 | #include "qemu-common.h" | ||
649 | #include "cpu.h" | ||
650 | #include "hw/sysbus.h" | ||
651 | -#include "hw/arm/arm.h" | ||
652 | +#include "hw/arm/boot.h" | ||
653 | #include "hw/net/smc91c111.h" | ||
654 | #include "net/net.h" | ||
655 | #include "sysemu/sysemu.h" | ||
656 | diff --git a/hw/arm/vexpress.c b/hw/arm/vexpress.c | ||
657 | index XXXXXXX..XXXXXXX 100644 | ||
658 | --- a/hw/arm/vexpress.c | ||
659 | +++ b/hw/arm/vexpress.c | ||
660 | @@ -XXX,XX +XXX,XX @@ | ||
661 | #include "qemu-common.h" | ||
662 | #include "cpu.h" | ||
663 | #include "hw/sysbus.h" | ||
664 | -#include "hw/arm/arm.h" | ||
665 | +#include "hw/arm/boot.h" | ||
666 | #include "hw/arm/primecell.h" | ||
667 | #include "hw/net/lan9118.h" | ||
668 | #include "hw/i2c/i2c.h" | ||
669 | diff --git a/hw/arm/virt.c b/hw/arm/virt.c | ||
670 | index XXXXXXX..XXXXXXX 100644 | ||
671 | --- a/hw/arm/virt.c | ||
672 | +++ b/hw/arm/virt.c | ||
673 | @@ -XXX,XX +XXX,XX @@ | ||
674 | #include "qemu/option.h" | ||
675 | #include "qapi/error.h" | ||
676 | #include "hw/sysbus.h" | ||
677 | -#include "hw/arm/arm.h" | ||
678 | +#include "hw/arm/boot.h" | ||
679 | #include "hw/arm/primecell.h" | ||
680 | #include "hw/arm/virt.h" | ||
681 | #include "hw/block/flash.h" | ||
682 | diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c | ||
683 | index XXXXXXX..XXXXXXX 100644 | ||
684 | --- a/hw/arm/xilinx_zynq.c | ||
685 | +++ b/hw/arm/xilinx_zynq.c | ||
686 | @@ -XXX,XX +XXX,XX @@ | ||
687 | #include "qemu-common.h" | ||
688 | #include "cpu.h" | ||
689 | #include "hw/sysbus.h" | ||
690 | -#include "hw/arm/arm.h" | ||
691 | +#include "hw/arm/boot.h" | ||
692 | #include "net/net.h" | ||
693 | #include "exec/address-spaces.h" | ||
694 | #include "sysemu/sysemu.h" | ||
695 | diff --git a/hw/arm/xlnx-versal.c b/hw/arm/xlnx-versal.c | ||
696 | index XXXXXXX..XXXXXXX 100644 | ||
697 | --- a/hw/arm/xlnx-versal.c | ||
698 | +++ b/hw/arm/xlnx-versal.c | ||
699 | @@ -XXX,XX +XXX,XX @@ | ||
700 | #include "net/net.h" | ||
701 | #include "sysemu/sysemu.h" | ||
702 | #include "sysemu/kvm.h" | ||
703 | -#include "hw/arm/arm.h" | ||
704 | +#include "hw/arm/boot.h" | ||
705 | #include "kvm_arm.h" | ||
706 | #include "hw/misc/unimp.h" | ||
707 | #include "hw/intc/arm_gicv3_common.h" | ||
708 | diff --git a/hw/arm/z2.c b/hw/arm/z2.c | ||
709 | index XXXXXXX..XXXXXXX 100644 | ||
710 | --- a/hw/arm/z2.c | ||
711 | +++ b/hw/arm/z2.c | ||
712 | @@ -XXX,XX +XXX,XX @@ | ||
713 | #include "qemu/osdep.h" | ||
714 | #include "hw/hw.h" | ||
715 | #include "hw/arm/pxa.h" | ||
716 | -#include "hw/arm/arm.h" | ||
717 | +#include "hw/arm/boot.h" | ||
718 | #include "hw/i2c/i2c.h" | ||
719 | #include "hw/ssi/ssi.h" | ||
720 | #include "hw/boards.h" | ||
44 | -- | 721 | -- |
45 | 2.7.4 | 722 | 2.20.1 |
46 | 723 | ||
47 | 724 | diff view generated by jsdifflib |
1 | Add a comment documenting the memory map of the SoC devices and which | 1 | In ich_vmcr_write() we enforce "writes of BPR fields to less than |
---|---|---|---|
2 | are not implemented. | 2 | their minimum sets them to the minimum" by doing a "read vbpr and |
3 | write it back" operation. A typo here meant that we weren't handling | ||
4 | writes to these fields correctly, because we were reading from VBPR0 | ||
5 | but writing to VBPR1. | ||
3 | 6 | ||
4 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 7 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
5 | Message-id: 1484247815-15279-2-git-send-email-peter.maydell@linaro.org | 8 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
9 | Message-id: 20190520162809.2677-4-peter.maydell@linaro.org | ||
6 | --- | 10 | --- |
7 | hw/arm/stellaris.c | 34 ++++++++++++++++++++++++++++++++++ | 11 | hw/intc/arm_gicv3_cpuif.c | 2 +- |
8 | 1 file changed, 34 insertions(+) | 12 | 1 file changed, 1 insertion(+), 1 deletion(-) |
9 | 13 | ||
10 | diff --git a/hw/arm/stellaris.c b/hw/arm/stellaris.c | 14 | diff --git a/hw/intc/arm_gicv3_cpuif.c b/hw/intc/arm_gicv3_cpuif.c |
11 | index XXXXXXX..XXXXXXX 100644 | 15 | index XXXXXXX..XXXXXXX 100644 |
12 | --- a/hw/arm/stellaris.c | 16 | --- a/hw/intc/arm_gicv3_cpuif.c |
13 | +++ b/hw/arm/stellaris.c | 17 | +++ b/hw/intc/arm_gicv3_cpuif.c |
14 | @@ -XXX,XX +XXX,XX @@ static void stellaris_init(const char *kernel_filename, const char *cpu_model, | 18 | @@ -XXX,XX +XXX,XX @@ static void ich_vmcr_write(CPUARMState *env, const ARMCPRegInfo *ri, |
15 | 0x40024000, 0x40025000, 0x40026000}; | 19 | /* Enforce "writing BPRs to less than minimum sets them to the minimum" |
16 | static const int gpio_irq[7] = {0, 1, 2, 3, 4, 30, 31}; | 20 | * by reading and writing back the fields. |
17 | 21 | */ | |
18 | + /* Memory map of SoC devices, from | 22 | - write_vbpr(cs, GICV3_G1, read_vbpr(cs, GICV3_G0)); |
19 | + * Stellaris LM3S6965 Microcontroller Data Sheet (rev I) | 23 | + write_vbpr(cs, GICV3_G0, read_vbpr(cs, GICV3_G0)); |
20 | + * http://www.ti.com/lit/ds/symlink/lm3s6965.pdf | 24 | write_vbpr(cs, GICV3_G1, read_vbpr(cs, GICV3_G1)); |
21 | + * | 25 | |
22 | + * 40000000 wdtimer (unimplemented) | 26 | gicv3_cpuif_virt_update(cs); |
23 | + * 40002000 i2c (unimplemented) | ||
24 | + * 40004000 GPIO | ||
25 | + * 40005000 GPIO | ||
26 | + * 40006000 GPIO | ||
27 | + * 40007000 GPIO | ||
28 | + * 40008000 SSI | ||
29 | + * 4000c000 UART | ||
30 | + * 4000d000 UART | ||
31 | + * 4000e000 UART | ||
32 | + * 40020000 i2c | ||
33 | + * 40021000 i2c (unimplemented) | ||
34 | + * 40024000 GPIO | ||
35 | + * 40025000 GPIO | ||
36 | + * 40026000 GPIO | ||
37 | + * 40028000 PWM (unimplemented) | ||
38 | + * 4002c000 QEI (unimplemented) | ||
39 | + * 4002d000 QEI (unimplemented) | ||
40 | + * 40030000 gptimer | ||
41 | + * 40031000 gptimer | ||
42 | + * 40032000 gptimer | ||
43 | + * 40033000 gptimer | ||
44 | + * 40038000 ADC | ||
45 | + * 4003c000 analogue comparator (unimplemented) | ||
46 | + * 40048000 ethernet | ||
47 | + * 400fc000 hibernation module (unimplemented) | ||
48 | + * 400fd000 flash memory control (unimplemented) | ||
49 | + * 400fe000 system control | ||
50 | + */ | ||
51 | + | ||
52 | DeviceState *gpio_dev[7], *nvic; | ||
53 | qemu_irq gpio_in[7][8]; | ||
54 | qemu_irq gpio_out[7][8]; | ||
55 | -- | 27 | -- |
56 | 2.7.4 | 28 | 2.20.1 |
57 | 29 | ||
58 | 30 | diff view generated by jsdifflib |
1 | From: Julian Brown <julian@codesourcery.com> | 1 | The ICC_CTLR_EL3 register includes some bits which are aliases |
---|---|---|---|
2 | of bits in the ICC_CTLR_EL1(S) and (NS) registers. QEMU chooses | ||
3 | to keep those bits in the cs->icc_ctlr_el1[] struct fields. | ||
4 | Unfortunately a missing '~' in the code to update the bits | ||
5 | in those fields meant that writing to ICC_CTLR_EL3 would corrupt | ||
6 | the ICC_CLTR_EL1 register values. | ||
2 | 7 | ||
3 | Add a new "cfgend" property which selects whether the CPU resets into | 8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
4 | big-endian mode or not. This setting affects whether we reset with | 9 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
5 | SCTLR_B (ARMv6 and earlier) or SCTLR_EE (ARMv7 and later) set. | 10 | Message-id: 20190520162809.2677-5-peter.maydell@linaro.org |
11 | --- | ||
12 | hw/intc/arm_gicv3_cpuif.c | 4 ++-- | ||
13 | 1 file changed, 2 insertions(+), 2 deletions(-) | ||
6 | 14 | ||
7 | Signed-off-by: Julian Brown <julian@codesourcery.com> | 15 | diff --git a/hw/intc/arm_gicv3_cpuif.c b/hw/intc/arm_gicv3_cpuif.c |
8 | Message-id: 11420d1c49636c1790e60578ee996e51f0f0b835.1484929304.git.julian@codesourcery.com | ||
9 | [PMM: use error_report_err() rather than error_report(); | ||
10 | move the integratorcp changes to their own patch; | ||
11 | drop an unnecessary extra #include; | ||
12 | rephrase commit message accordingly; | ||
13 | move setting of reset_sctlr above registration of cpregs | ||
14 | so it actually has an effect] | ||
15 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
16 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | ||
17 | --- | ||
18 | target/arm/cpu.h | 7 +++++++ | ||
19 | target/arm/cpu.c | 13 +++++++++++++ | ||
20 | 2 files changed, 20 insertions(+) | ||
21 | |||
22 | diff --git a/target/arm/cpu.h b/target/arm/cpu.h | ||
23 | index XXXXXXX..XXXXXXX 100644 | 16 | index XXXXXXX..XXXXXXX 100644 |
24 | --- a/target/arm/cpu.h | 17 | --- a/hw/intc/arm_gicv3_cpuif.c |
25 | +++ b/target/arm/cpu.h | 18 | +++ b/hw/intc/arm_gicv3_cpuif.c |
26 | @@ -XXX,XX +XXX,XX @@ struct ARMCPU { | 19 | @@ -XXX,XX +XXX,XX @@ static void icc_ctlr_el3_write(CPUARMState *env, const ARMCPRegInfo *ri, |
27 | int gic_vpribits; /* number of virtual priority bits */ | 20 | trace_gicv3_icc_ctlr_el3_write(gicv3_redist_affid(cs), value); |
28 | int gic_vprebits; /* number of virtual preemption bits */ | 21 | |
29 | 22 | /* *_EL1NS and *_EL1S bits are aliases into the ICC_CTLR_EL1 bits. */ | |
30 | + /* Whether the cfgend input is high (i.e. this CPU should reset into | 23 | - cs->icc_ctlr_el1[GICV3_NS] &= (ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE); |
31 | + * big-endian mode). This setting isn't used directly: instead it modifies | 24 | + cs->icc_ctlr_el1[GICV3_NS] &= ~(ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE); |
32 | + * the reset_sctlr value to have SCTLR_B or SCTLR_EE set, depending on the | 25 | if (value & ICC_CTLR_EL3_EOIMODE_EL1NS) { |
33 | + * architecture version. | 26 | cs->icc_ctlr_el1[GICV3_NS] |= ICC_CTLR_EL1_EOIMODE; |
34 | + */ | ||
35 | + bool cfgend; | ||
36 | + | ||
37 | ARMELChangeHook *el_change_hook; | ||
38 | void *el_change_hook_opaque; | ||
39 | }; | ||
40 | diff --git a/target/arm/cpu.c b/target/arm/cpu.c | ||
41 | index XXXXXXX..XXXXXXX 100644 | ||
42 | --- a/target/arm/cpu.c | ||
43 | +++ b/target/arm/cpu.c | ||
44 | @@ -XXX,XX +XXX,XX @@ static Property arm_cpu_has_el2_property = | ||
45 | static Property arm_cpu_has_el3_property = | ||
46 | DEFINE_PROP_BOOL("has_el3", ARMCPU, has_el3, true); | ||
47 | |||
48 | +static Property arm_cpu_cfgend_property = | ||
49 | + DEFINE_PROP_BOOL("cfgend", ARMCPU, cfgend, false); | ||
50 | + | ||
51 | /* use property name "pmu" to match other archs and virt tools */ | ||
52 | static Property arm_cpu_has_pmu_property = | ||
53 | DEFINE_PROP_BOOL("pmu", ARMCPU, has_pmu, true); | ||
54 | @@ -XXX,XX +XXX,XX @@ static void arm_cpu_post_init(Object *obj) | ||
55 | } | ||
56 | } | 27 | } |
57 | 28 | @@ -XXX,XX +XXX,XX @@ static void icc_ctlr_el3_write(CPUARMState *env, const ARMCPRegInfo *ri, | |
58 | + qdev_property_add_static(DEVICE(obj), &arm_cpu_cfgend_property, | 29 | cs->icc_ctlr_el1[GICV3_NS] |= ICC_CTLR_EL1_CBPR; |
59 | + &error_abort); | ||
60 | } | ||
61 | |||
62 | static void arm_cpu_finalizefn(Object *obj) | ||
63 | @@ -XXX,XX +XXX,XX @@ static void arm_cpu_realizefn(DeviceState *dev, Error **errp) | ||
64 | cpu->reset_sctlr |= (1 << 13); | ||
65 | } | 30 | } |
66 | 31 | ||
67 | + if (cpu->cfgend) { | 32 | - cs->icc_ctlr_el1[GICV3_S] &= (ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE); |
68 | + if (arm_feature(&cpu->env, ARM_FEATURE_V7)) { | 33 | + cs->icc_ctlr_el1[GICV3_S] &= ~(ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE); |
69 | + cpu->reset_sctlr |= SCTLR_EE; | 34 | if (value & ICC_CTLR_EL3_EOIMODE_EL1S) { |
70 | + } else { | 35 | cs->icc_ctlr_el1[GICV3_S] |= ICC_CTLR_EL1_EOIMODE; |
71 | + cpu->reset_sctlr |= SCTLR_B; | 36 | } |
72 | + } | ||
73 | + } | ||
74 | + | ||
75 | if (!cpu->has_el3) { | ||
76 | /* If the has_el3 CPU property is disabled then we need to disable the | ||
77 | * feature. | ||
78 | -- | 37 | -- |
79 | 2.7.4 | 38 | 2.20.1 |
80 | 39 | ||
81 | 40 | diff view generated by jsdifflib |
1 | From: Prasad J Pandit <pjp@fedoraproject.org> | 1 | From: Philippe Mathieu-Daudé <philmd@redhat.com> |
---|---|---|---|
2 | 2 | ||
3 | While doing multi block SDMA transfer in routine | 3 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
4 | 'sdhci_sdma_transfer_multi_blocks', the 's->fifo_buffer' starting | 4 | Reviewed-by: Alistair Francis <alistair.francis@wdc.com> |
5 | index 'begin' and data length 's->data_count' could end up to be same. | 5 | Message-id: 20190520214342.13709-2-philmd@redhat.com |
6 | This could lead to an OOB access issue. Correct transfer data length | ||
7 | to avoid it. | ||
8 | |||
9 | Cc: qemu-stable@nongnu.org | ||
10 | Reported-by: Jiang Xin <jiangxin1@huawei.com> | ||
11 | Signed-off-by: Prasad J Pandit <pjp@fedoraproject.org> | ||
12 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
13 | Message-id: 20170130064736.9236-1-ppandit@redhat.com | ||
14 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 6 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
15 | --- | 7 | --- |
16 | hw/sd/sdhci.c | 2 +- | 8 | hw/arm/exynos4_boards.c | 24 ------------------------ |
17 | 1 file changed, 1 insertion(+), 1 deletion(-) | 9 | 1 file changed, 24 deletions(-) |
18 | 10 | ||
19 | diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c | 11 | diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c |
20 | index XXXXXXX..XXXXXXX 100644 | 12 | index XXXXXXX..XXXXXXX 100644 |
21 | --- a/hw/sd/sdhci.c | 13 | --- a/hw/arm/exynos4_boards.c |
22 | +++ b/hw/sd/sdhci.c | 14 | +++ b/hw/arm/exynos4_boards.c |
23 | @@ -XXX,XX +XXX,XX @@ static void sdhci_sdma_transfer_multi_blocks(SDHCIState *s) | 15 | @@ -XXX,XX +XXX,XX @@ |
24 | boundary_count -= block_size - begin; | 16 | #include "hw/net/lan9118.h" |
25 | } | 17 | #include "hw/boards.h" |
26 | dma_memory_read(&address_space_memory, s->sdmasysad, | 18 | |
27 | - &s->fifo_buffer[begin], s->data_count); | 19 | -#undef DEBUG |
28 | + &s->fifo_buffer[begin], s->data_count - begin); | 20 | - |
29 | s->sdmasysad += s->data_count - begin; | 21 | -//#define DEBUG |
30 | if (s->data_count == block_size) { | 22 | - |
31 | for (n = 0; n < block_size; n++) { | 23 | -#ifdef DEBUG |
24 | - #undef PRINT_DEBUG | ||
25 | - #define PRINT_DEBUG(fmt, args...) \ | ||
26 | - do { \ | ||
27 | - fprintf(stderr, " [%s:%d] "fmt, __func__, __LINE__, ##args); \ | ||
28 | - } while (0) | ||
29 | -#else | ||
30 | - #define PRINT_DEBUG(fmt, args...) do {} while (0) | ||
31 | -#endif | ||
32 | - | ||
33 | #define SMDK_LAN9118_BASE_ADDR 0x05000000 | ||
34 | |||
35 | typedef enum Exynos4BoardType { | ||
36 | @@ -XXX,XX +XXX,XX @@ exynos4_boards_init_common(MachineState *machine, | ||
37 | exynos4_board_binfo.gic_cpu_if_addr = | ||
38 | EXYNOS4210_SMP_PRIVATE_BASE_ADDR + 0x100; | ||
39 | |||
40 | - PRINT_DEBUG("\n ram_size: %luMiB [0x%08lx]\n" | ||
41 | - " kernel_filename: %s\n" | ||
42 | - " kernel_cmdline: %s\n" | ||
43 | - " initrd_filename: %s\n", | ||
44 | - exynos4_board_ram_size[board_type] / 1048576, | ||
45 | - exynos4_board_ram_size[board_type], | ||
46 | - machine->kernel_filename, | ||
47 | - machine->kernel_cmdline, | ||
48 | - machine->initrd_filename); | ||
49 | - | ||
50 | exynos4_boards_init_ram(s, get_system_memory(), | ||
51 | exynos4_board_ram_size[board_type]); | ||
52 | |||
32 | -- | 53 | -- |
33 | 2.7.4 | 54 | 2.20.1 |
34 | 55 | ||
35 | 56 | diff view generated by jsdifflib |
1 | From: Pavel Dovgalyuk <Pavel.Dovgaluk@ispras.ru> | 1 | From: Philippe Mathieu-Daudé <philmd@redhat.com> |
---|---|---|---|
2 | 2 | ||
3 | VMState added by this patch preserves correct | 3 | It eases code review, unit is explicit. |
4 | loading of the integratorcp device state. | ||
5 | 4 | ||
6 | Signed-off-by: Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru> | 5 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
7 | Message-id: 20170131114310.6768.79416.stgit@PASHA-ISP | 6 | Reviewed-by: Alistair Francis <alistair.francis@wdc.com> |
8 | [PMM: removed unnecessary minimum_version_id_old lines] | 7 | Message-id: 20190520214342.13709-3-philmd@redhat.com |
9 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
10 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
11 | --- | 9 | --- |
12 | hw/arm/integratorcp.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++ | 10 | hw/arm/exynos4_boards.c | 5 +++-- |
13 | 1 file changed, 59 insertions(+) | 11 | 1 file changed, 3 insertions(+), 2 deletions(-) |
14 | 12 | ||
15 | diff --git a/hw/arm/integratorcp.c b/hw/arm/integratorcp.c | 13 | diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c |
16 | index XXXXXXX..XXXXXXX 100644 | 14 | index XXXXXXX..XXXXXXX 100644 |
17 | --- a/hw/arm/integratorcp.c | 15 | --- a/hw/arm/exynos4_boards.c |
18 | +++ b/hw/arm/integratorcp.c | 16 | +++ b/hw/arm/exynos4_boards.c |
19 | @@ -XXX,XX +XXX,XX @@ static uint8_t integrator_spd[128] = { | 17 | @@ -XXX,XX +XXX,XX @@ |
20 | 0xe, 4, 0x1c, 1, 2, 0x20, 0xc0, 0, 0, 0, 0, 0x30, 0x28, 0x30, 0x28, 0x40 | 18 | */ |
19 | |||
20 | #include "qemu/osdep.h" | ||
21 | +#include "qemu/units.h" | ||
22 | #include "qapi/error.h" | ||
23 | #include "qemu/error-report.h" | ||
24 | #include "qemu-common.h" | ||
25 | @@ -XXX,XX +XXX,XX @@ static int exynos4_board_smp_bootreg_addr[EXYNOS4_NUM_OF_BOARDS] = { | ||
21 | }; | 26 | }; |
22 | 27 | ||
23 | +static const VMStateDescription vmstate_integratorcm = { | 28 | static unsigned long exynos4_board_ram_size[EXYNOS4_NUM_OF_BOARDS] = { |
24 | + .name = "integratorcm", | 29 | - [EXYNOS4_BOARD_NURI] = 0x40000000, |
25 | + .version_id = 1, | 30 | - [EXYNOS4_BOARD_SMDKC210] = 0x40000000, |
26 | + .minimum_version_id = 1, | 31 | + [EXYNOS4_BOARD_NURI] = 1 * GiB, |
27 | + .fields = (VMStateField[]) { | 32 | + [EXYNOS4_BOARD_SMDKC210] = 1 * GiB, |
28 | + VMSTATE_UINT32(cm_osc, IntegratorCMState), | ||
29 | + VMSTATE_UINT32(cm_ctrl, IntegratorCMState), | ||
30 | + VMSTATE_UINT32(cm_lock, IntegratorCMState), | ||
31 | + VMSTATE_UINT32(cm_auxosc, IntegratorCMState), | ||
32 | + VMSTATE_UINT32(cm_sdram, IntegratorCMState), | ||
33 | + VMSTATE_UINT32(cm_init, IntegratorCMState), | ||
34 | + VMSTATE_UINT32(cm_flags, IntegratorCMState), | ||
35 | + VMSTATE_UINT32(cm_nvflags, IntegratorCMState), | ||
36 | + VMSTATE_UINT32(int_level, IntegratorCMState), | ||
37 | + VMSTATE_UINT32(irq_enabled, IntegratorCMState), | ||
38 | + VMSTATE_UINT32(fiq_enabled, IntegratorCMState), | ||
39 | + VMSTATE_END_OF_LIST() | ||
40 | + } | ||
41 | +}; | ||
42 | + | ||
43 | static uint64_t integratorcm_read(void *opaque, hwaddr offset, | ||
44 | unsigned size) | ||
45 | { | ||
46 | @@ -XXX,XX +XXX,XX @@ typedef struct icp_pic_state { | ||
47 | qemu_irq parent_fiq; | ||
48 | } icp_pic_state; | ||
49 | |||
50 | +static const VMStateDescription vmstate_icp_pic = { | ||
51 | + .name = "icp_pic", | ||
52 | + .version_id = 1, | ||
53 | + .minimum_version_id = 1, | ||
54 | + .fields = (VMStateField[]) { | ||
55 | + VMSTATE_UINT32(level, icp_pic_state), | ||
56 | + VMSTATE_UINT32(irq_enabled, icp_pic_state), | ||
57 | + VMSTATE_UINT32(fiq_enabled, icp_pic_state), | ||
58 | + VMSTATE_END_OF_LIST() | ||
59 | + } | ||
60 | +}; | ||
61 | + | ||
62 | static void icp_pic_update(icp_pic_state *s) | ||
63 | { | ||
64 | uint32_t flags; | ||
65 | @@ -XXX,XX +XXX,XX @@ typedef struct ICPCtrlRegsState { | ||
66 | #define ICP_INTREG_WPROT (1 << 0) | ||
67 | #define ICP_INTREG_CARDIN (1 << 3) | ||
68 | |||
69 | +static const VMStateDescription vmstate_icp_control = { | ||
70 | + .name = "icp_control", | ||
71 | + .version_id = 1, | ||
72 | + .minimum_version_id = 1, | ||
73 | + .fields = (VMStateField[]) { | ||
74 | + VMSTATE_UINT32(intreg_state, ICPCtrlRegsState), | ||
75 | + VMSTATE_END_OF_LIST() | ||
76 | + } | ||
77 | +}; | ||
78 | + | ||
79 | static uint64_t icp_control_read(void *opaque, hwaddr offset, | ||
80 | unsigned size) | ||
81 | { | ||
82 | @@ -XXX,XX +XXX,XX @@ static void core_class_init(ObjectClass *klass, void *data) | ||
83 | |||
84 | dc->props = core_properties; | ||
85 | dc->realize = integratorcm_realize; | ||
86 | + dc->vmsd = &vmstate_integratorcm; | ||
87 | +} | ||
88 | + | ||
89 | +static void icp_pic_class_init(ObjectClass *klass, void *data) | ||
90 | +{ | ||
91 | + DeviceClass *dc = DEVICE_CLASS(klass); | ||
92 | + | ||
93 | + dc->vmsd = &vmstate_icp_pic; | ||
94 | +} | ||
95 | + | ||
96 | +static void icp_control_class_init(ObjectClass *klass, void *data) | ||
97 | +{ | ||
98 | + DeviceClass *dc = DEVICE_CLASS(klass); | ||
99 | + | ||
100 | + dc->vmsd = &vmstate_icp_control; | ||
101 | } | ||
102 | |||
103 | static const TypeInfo core_info = { | ||
104 | @@ -XXX,XX +XXX,XX @@ static const TypeInfo icp_pic_info = { | ||
105 | .parent = TYPE_SYS_BUS_DEVICE, | ||
106 | .instance_size = sizeof(icp_pic_state), | ||
107 | .instance_init = icp_pic_init, | ||
108 | + .class_init = icp_pic_class_init, | ||
109 | }; | 33 | }; |
110 | 34 | ||
111 | static const TypeInfo icp_ctrl_regs_info = { | 35 | static struct arm_boot_info exynos4_board_binfo = { |
112 | @@ -XXX,XX +XXX,XX @@ static const TypeInfo icp_ctrl_regs_info = { | ||
113 | .parent = TYPE_SYS_BUS_DEVICE, | ||
114 | .instance_size = sizeof(ICPCtrlRegsState), | ||
115 | .instance_init = icp_control_init, | ||
116 | + .class_init = icp_control_class_init, | ||
117 | }; | ||
118 | |||
119 | static void integratorcp_register_types(void) | ||
120 | -- | 36 | -- |
121 | 2.7.4 | 37 | 2.20.1 |
122 | 38 | ||
123 | 39 | diff view generated by jsdifflib |
1 | From: Julian Brown <julian@codesourcery.com> | 1 | From: Guenter Roeck <linux@roeck-us.net> |
---|---|---|---|
2 | 2 | ||
3 | Thumb-1 code has some issues in BE32 mode (as currently implemented). In | 3 | QEMU already supports pl330. Instantiate it for Exynos4210. |
4 | short, since bytes are swapped within words at load time for BE32 | ||
5 | executables, this also swaps pairs of adjacent Thumb-1 instructions. | ||
6 | 4 | ||
7 | This patch un-swaps those pairs of instructions again, both for execution, | 5 | Relevant part of Linux arch/arm/boot/dts/exynos4.dtsi: |
8 | and for disassembly. (The previous version of the patch always read four | ||
9 | bytes in arm_read_memory_func and then extracted the proper two bytes, | ||
10 | in a probably misguided attempt to match the behaviour of actual hardware | ||
11 | as described by e.g. the ARM9TDMI TRM, section 3.3 "Endian effects for | ||
12 | instruction fetches". It's less complicated to just read the correct | ||
13 | two bytes though.) | ||
14 | 6 | ||
15 | Signed-off-by: Julian Brown <julian@codesourcery.com> | 7 | / { |
16 | Message-id: ca20462a044848000370318a8bd41dd0a4ed273f.1484929304.git.julian@codesourcery.com | 8 | soc: soc { |
9 | amba { | ||
10 | pdma0: pdma@12680000 { | ||
11 | compatible = "arm,pl330", "arm,primecell"; | ||
12 | reg = <0x12680000 0x1000>; | ||
13 | interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>; | ||
14 | clocks = <&clock CLK_PDMA0>; | ||
15 | clock-names = "apb_pclk"; | ||
16 | #dma-cells = <1>; | ||
17 | #dma-channels = <8>; | ||
18 | #dma-requests = <32>; | ||
19 | }; | ||
20 | pdma1: pdma@12690000 { | ||
21 | compatible = "arm,pl330", "arm,primecell"; | ||
22 | reg = <0x12690000 0x1000>; | ||
23 | interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>; | ||
24 | clocks = <&clock CLK_PDMA1>; | ||
25 | clock-names = "apb_pclk"; | ||
26 | #dma-cells = <1>; | ||
27 | #dma-channels = <8>; | ||
28 | #dma-requests = <32>; | ||
29 | }; | ||
30 | mdma1: mdma@12850000 { | ||
31 | compatible = "arm,pl330", "arm,primecell"; | ||
32 | reg = <0x12850000 0x1000>; | ||
33 | interrupts = <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>; | ||
34 | clocks = <&clock CLK_MDMA>; | ||
35 | clock-names = "apb_pclk"; | ||
36 | #dma-cells = <1>; | ||
37 | #dma-channels = <8>; | ||
38 | #dma-requests = <1>; | ||
39 | }; | ||
40 | }; | ||
41 | }; | ||
42 | }; | ||
43 | |||
44 | Signed-off-by: Guenter Roeck <linux@roeck-us.net> | ||
45 | Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
46 | Reviewed-by: Alistair Francis <alistair.francis@wdc.com> | ||
47 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
48 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
49 | Message-id: 20190520214342.13709-4-philmd@redhat.com | ||
50 | [PMD: Do not set default qdev properties, create the controllers in the SoC | ||
51 | rather than the board (Peter Maydell), add dtsi in commit message] | ||
52 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
17 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | 53 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> |
18 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 54 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
19 | --- | 55 | --- |
20 | include/disas/bfd.h | 7 +++++++ | 56 | hw/arm/exynos4210.c | 26 ++++++++++++++++++++++++++ |
21 | target/arm/arm_ldst.h | 10 +++++++++- | 57 | 1 file changed, 26 insertions(+) |
22 | disas.c | 1 + | ||
23 | target/arm/cpu.c | 23 +++++++++++++++++++++++ | ||
24 | 4 files changed, 40 insertions(+), 1 deletion(-) | ||
25 | 58 | ||
26 | diff --git a/include/disas/bfd.h b/include/disas/bfd.h | 59 | diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c |
27 | index XXXXXXX..XXXXXXX 100644 | 60 | index XXXXXXX..XXXXXXX 100644 |
28 | --- a/include/disas/bfd.h | 61 | --- a/hw/arm/exynos4210.c |
29 | +++ b/include/disas/bfd.h | 62 | +++ b/hw/arm/exynos4210.c |
30 | @@ -XXX,XX +XXX,XX @@ typedef struct disassemble_info { | 63 | @@ -XXX,XX +XXX,XX @@ |
31 | The bottom 16 bits are for the internal use of the disassembler. */ | 64 | /* EHCI */ |
32 | unsigned long flags; | 65 | #define EXYNOS4210_EHCI_BASE_ADDR 0x12580000 |
33 | #define INSN_HAS_RELOC 0x80000000 | 66 | |
34 | +#define INSN_ARM_BE32 0x00010000 | 67 | +/* DMA */ |
35 | PTR private_data; | 68 | +#define EXYNOS4210_PL330_BASE0_ADDR 0x12680000 |
36 | 69 | +#define EXYNOS4210_PL330_BASE1_ADDR 0x12690000 | |
37 | /* Function used to get bytes to disassemble. MEMADDR is the | 70 | +#define EXYNOS4210_PL330_BASE2_ADDR 0x12850000 |
38 | @@ -XXX,XX +XXX,XX @@ typedef struct disassemble_info { | ||
39 | (bfd_vma memaddr, bfd_byte *myaddr, int length, | ||
40 | struct disassemble_info *info); | ||
41 | |||
42 | + /* A place to stash the real read_memory_func if read_memory_func wants to | ||
43 | + do some funky address arithmetic or similar (e.g. for ARM BE32 mode). */ | ||
44 | + int (*read_memory_inner_func) | ||
45 | + (bfd_vma memaddr, bfd_byte *myaddr, int length, | ||
46 | + struct disassemble_info *info); | ||
47 | + | 71 | + |
48 | /* Function which should be called if we get an error that we can't | 72 | static uint8_t chipid_and_omr[] = { 0x11, 0x02, 0x21, 0x43, |
49 | recover from. STATUS is the errno value from read_memory_func and | 73 | 0x09, 0x00, 0x00, 0x00 }; |
50 | MEMADDR is the address that we were trying to read. INFO is a | 74 | |
51 | diff --git a/target/arm/arm_ldst.h b/target/arm/arm_ldst.h | 75 | @@ -XXX,XX +XXX,XX @@ static uint64_t exynos4210_calc_affinity(int cpu) |
52 | index XXXXXXX..XXXXXXX 100644 | 76 | return (0x9 << ARM_AFF1_SHIFT) | cpu; |
53 | --- a/target/arm/arm_ldst.h | ||
54 | +++ b/target/arm/arm_ldst.h | ||
55 | @@ -XXX,XX +XXX,XX @@ static inline uint32_t arm_ldl_code(CPUARMState *env, target_ulong addr, | ||
56 | static inline uint16_t arm_lduw_code(CPUARMState *env, target_ulong addr, | ||
57 | bool sctlr_b) | ||
58 | { | ||
59 | - uint16_t insn = cpu_lduw_code(env, addr); | ||
60 | + uint16_t insn; | ||
61 | +#ifndef CONFIG_USER_ONLY | ||
62 | + /* In big-endian (BE32) mode, adjacent Thumb instructions have been swapped | ||
63 | + within each word. Undo that now. */ | ||
64 | + if (sctlr_b) { | ||
65 | + addr ^= 2; | ||
66 | + } | ||
67 | +#endif | ||
68 | + insn = cpu_lduw_code(env, addr); | ||
69 | if (bswap_code(sctlr_b)) { | ||
70 | return bswap16(insn); | ||
71 | } | ||
72 | diff --git a/disas.c b/disas.c | ||
73 | index XXXXXXX..XXXXXXX 100644 | ||
74 | --- a/disas.c | ||
75 | +++ b/disas.c | ||
76 | @@ -XXX,XX +XXX,XX @@ void target_disas(FILE *out, CPUState *cpu, target_ulong code, | ||
77 | |||
78 | s.cpu = cpu; | ||
79 | s.info.read_memory_func = target_read_memory; | ||
80 | + s.info.read_memory_inner_func = NULL; | ||
81 | s.info.buffer_vma = code; | ||
82 | s.info.buffer_length = size; | ||
83 | s.info.print_address_func = generic_print_address; | ||
84 | diff --git a/target/arm/cpu.c b/target/arm/cpu.c | ||
85 | index XXXXXXX..XXXXXXX 100644 | ||
86 | --- a/target/arm/cpu.c | ||
87 | +++ b/target/arm/cpu.c | ||
88 | @@ -XXX,XX +XXX,XX @@ print_insn_thumb1(bfd_vma pc, disassemble_info *info) | ||
89 | return print_insn_arm(pc | 1, info); | ||
90 | } | 77 | } |
91 | 78 | ||
92 | +static int arm_read_memory_func(bfd_vma memaddr, bfd_byte *b, | 79 | +static void pl330_create(uint32_t base, qemu_irq irq, int nreq) |
93 | + int length, struct disassemble_info *info) | ||
94 | +{ | 80 | +{ |
95 | + assert(info->read_memory_inner_func); | 81 | + SysBusDevice *busdev; |
96 | + assert((info->flags & INSN_ARM_BE32) == 0 || length == 2 || length == 4); | 82 | + DeviceState *dev; |
97 | + | 83 | + |
98 | + if ((info->flags & INSN_ARM_BE32) != 0 && length == 2) { | 84 | + dev = qdev_create(NULL, "pl330"); |
99 | + assert(info->endian == BFD_ENDIAN_LITTLE); | 85 | + qdev_prop_set_uint8(dev, "num_periph_req", nreq); |
100 | + return info->read_memory_inner_func(memaddr ^ 2, (bfd_byte *)b, 2, | 86 | + qdev_init_nofail(dev); |
101 | + info); | 87 | + busdev = SYS_BUS_DEVICE(dev); |
102 | + } else { | 88 | + sysbus_mmio_map(busdev, 0, base); |
103 | + return info->read_memory_inner_func(memaddr, b, length, info); | 89 | + sysbus_connect_irq(busdev, 0, irq); |
104 | + } | ||
105 | +} | 90 | +} |
106 | + | 91 | + |
107 | static void arm_disas_set_info(CPUState *cpu, disassemble_info *info) | 92 | Exynos4210State *exynos4210_init(MemoryRegion *system_mem) |
108 | { | 93 | { |
109 | ARMCPU *ac = ARM_CPU(cpu); | 94 | Exynos4210State *s = g_new0(Exynos4210State, 1); |
110 | @@ -XXX,XX +XXX,XX @@ static void arm_disas_set_info(CPUState *cpu, disassemble_info *info) | 95 | @@ -XXX,XX +XXX,XX @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem) |
111 | info->endian = BFD_ENDIAN_BIG; | 96 | sysbus_create_simple(TYPE_EXYNOS4210_EHCI, EXYNOS4210_EHCI_BASE_ADDR, |
112 | #endif | 97 | s->irq_table[exynos4210_get_irq(28, 3)]); |
113 | } | 98 | |
114 | + if (info->read_memory_inner_func == NULL) { | 99 | + /*** DMA controllers ***/ |
115 | + info->read_memory_inner_func = info->read_memory_func; | 100 | + pl330_create(EXYNOS4210_PL330_BASE0_ADDR, |
116 | + info->read_memory_func = arm_read_memory_func; | 101 | + qemu_irq_invert(s->irq_table[exynos4210_get_irq(35, 1)]), 32); |
117 | + } | 102 | + pl330_create(EXYNOS4210_PL330_BASE1_ADDR, |
118 | + info->flags &= ~INSN_ARM_BE32; | 103 | + qemu_irq_invert(s->irq_table[exynos4210_get_irq(36, 1)]), 32); |
119 | + if (arm_sctlr_b(env)) { | 104 | + pl330_create(EXYNOS4210_PL330_BASE2_ADDR, |
120 | + info->flags |= INSN_ARM_BE32; | 105 | + qemu_irq_invert(s->irq_table[exynos4210_get_irq(34, 1)]), 1); |
121 | + } | 106 | + |
107 | return s; | ||
122 | } | 108 | } |
123 | |||
124 | static void arm_cpu_initfn(Object *obj) | ||
125 | -- | 109 | -- |
126 | 2.7.4 | 110 | 2.20.1 |
127 | 111 | ||
128 | 112 | diff view generated by jsdifflib |
1 | From: Cédric Le Goater <clg@kaod.org> | 1 | From: Philippe Mathieu-Daudé <philmd@redhat.com> |
---|---|---|---|
2 | 2 | ||
3 | The Aspeed SoC includes a set of watchdog timers using 32-bit | 3 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> |
4 | decrement counters, which can be based either on the APB clock or | 4 | Reviewed-by: Alistair Francis <alistair.francis@wdc.com> |
5 | a 1 MHz clock. | 5 | Message-id: 20190520214342.13709-5-philmd@redhat.com |
6 | |||
7 | The watchdog timer is designed to prevent system deadlock and, in | ||
8 | general, it should be restarted before timeout. When a timeout occurs, | ||
9 | different types of signals can be generated, ARM reset, SOC reset, | ||
10 | System reset, CPU Interrupt, external signal or boot from alternate | ||
11 | block. The current model only performs the system reset function as | ||
12 | this is used by U-Boot and Linux. | ||
13 | |||
14 | Signed-off-by: Joel Stanley <joel@jms.id.au> | ||
15 | Message-id: 1485452251-1593-2-git-send-email-clg@kaod.org | ||
16 | [clg: - fixed compile breakage | ||
17 | - fixed io region size | ||
18 | - added watchdog_perform_action() on timer expiry | ||
19 | - wrote a commit log | ||
20 | - merged fixes from Andrew Jeffery to scale the reload value ] | ||
21 | Signed-off-by: Cédric Le Goater <clg@kaod.org> | ||
22 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
23 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 6 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
24 | --- | 7 | --- |
25 | hw/watchdog/Makefile.objs | 1 + | 8 | include/hw/arm/exynos4210.h | 9 +++++++-- |
26 | include/hw/watchdog/wdt_aspeed.h | 32 ++++++ | 9 | hw/arm/exynos4210.c | 28 ++++++++++++++++++++++++---- |
27 | hw/watchdog/wdt_aspeed.c | 225 +++++++++++++++++++++++++++++++++++++++ | 10 | hw/arm/exynos4_boards.c | 9 ++++++--- |
28 | 3 files changed, 258 insertions(+) | 11 | 3 files changed, 37 insertions(+), 9 deletions(-) |
29 | create mode 100644 include/hw/watchdog/wdt_aspeed.h | ||
30 | create mode 100644 hw/watchdog/wdt_aspeed.c | ||
31 | 12 | ||
32 | diff --git a/hw/watchdog/Makefile.objs b/hw/watchdog/Makefile.objs | 13 | diff --git a/include/hw/arm/exynos4210.h b/include/hw/arm/exynos4210.h |
33 | index XXXXXXX..XXXXXXX 100644 | 14 | index XXXXXXX..XXXXXXX 100644 |
34 | --- a/hw/watchdog/Makefile.objs | 15 | --- a/include/hw/arm/exynos4210.h |
35 | +++ b/hw/watchdog/Makefile.objs | 16 | +++ b/include/hw/arm/exynos4210.h |
36 | @@ -XXX,XX +XXX,XX @@ common-obj-y += watchdog.o | 17 | @@ -XXX,XX +XXX,XX @@ typedef struct Exynos4210Irq { |
37 | common-obj-$(CONFIG_WDT_IB6300ESB) += wdt_i6300esb.o | 18 | } Exynos4210Irq; |
38 | common-obj-$(CONFIG_WDT_IB700) += wdt_ib700.o | 19 | |
39 | common-obj-$(CONFIG_WDT_DIAG288) += wdt_diag288.o | 20 | typedef struct Exynos4210State { |
40 | +common-obj-$(CONFIG_ASPEED_SOC) += wdt_aspeed.o | ||
41 | diff --git a/include/hw/watchdog/wdt_aspeed.h b/include/hw/watchdog/wdt_aspeed.h | ||
42 | new file mode 100644 | ||
43 | index XXXXXXX..XXXXXXX | ||
44 | --- /dev/null | ||
45 | +++ b/include/hw/watchdog/wdt_aspeed.h | ||
46 | @@ -XXX,XX +XXX,XX @@ | ||
47 | +/* | ||
48 | + * ASPEED Watchdog Controller | ||
49 | + * | ||
50 | + * Copyright (C) 2016-2017 IBM Corp. | ||
51 | + * | ||
52 | + * This code is licensed under the GPL version 2 or later. See the | ||
53 | + * COPYING file in the top-level directory. | ||
54 | + */ | ||
55 | +#ifndef ASPEED_WDT_H | ||
56 | +#define ASPEED_WDT_H | ||
57 | + | ||
58 | +#include "hw/sysbus.h" | ||
59 | + | ||
60 | +#define TYPE_ASPEED_WDT "aspeed.wdt" | ||
61 | +#define ASPEED_WDT(obj) \ | ||
62 | + OBJECT_CHECK(AspeedWDTState, (obj), TYPE_ASPEED_WDT) | ||
63 | + | ||
64 | +#define ASPEED_WDT_REGS_MAX (0x20 / 4) | ||
65 | + | ||
66 | +typedef struct AspeedWDTState { | ||
67 | + /*< private >*/ | 21 | + /*< private >*/ |
68 | + SysBusDevice parent_obj; | 22 | + SysBusDevice parent_obj; |
69 | + QEMUTimer *timer; | 23 | + /*< public >*/ |
24 | ARMCPU *cpu[EXYNOS4210_NCPUS]; | ||
25 | Exynos4210Irq irqs; | ||
26 | qemu_irq *irq_table; | ||
27 | @@ -XXX,XX +XXX,XX @@ typedef struct Exynos4210State { | ||
28 | I2CBus *i2c_if[EXYNOS4210_I2C_NUMBER]; | ||
29 | } Exynos4210State; | ||
30 | |||
31 | +#define TYPE_EXYNOS4210_SOC "exynos4210" | ||
32 | +#define EXYNOS4210_SOC(obj) \ | ||
33 | + OBJECT_CHECK(Exynos4210State, obj, TYPE_EXYNOS4210_SOC) | ||
70 | + | 34 | + |
71 | + /*< public >*/ | 35 | void exynos4210_write_secondary(ARMCPU *cpu, |
72 | + MemoryRegion iomem; | 36 | const struct arm_boot_info *info); |
73 | + uint32_t regs[ASPEED_WDT_REGS_MAX]; | 37 | |
38 | -Exynos4210State *exynos4210_init(MemoryRegion *system_mem); | ||
39 | - | ||
40 | /* Initialize exynos4210 IRQ subsystem stub */ | ||
41 | qemu_irq *exynos4210_init_irq(Exynos4210Irq *env); | ||
42 | |||
43 | diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c | ||
44 | index XXXXXXX..XXXXXXX 100644 | ||
45 | --- a/hw/arm/exynos4210.c | ||
46 | +++ b/hw/arm/exynos4210.c | ||
47 | @@ -XXX,XX +XXX,XX @@ static void pl330_create(uint32_t base, qemu_irq irq, int nreq) | ||
48 | sysbus_connect_irq(busdev, 0, irq); | ||
49 | } | ||
50 | |||
51 | -Exynos4210State *exynos4210_init(MemoryRegion *system_mem) | ||
52 | +static void exynos4210_realize(DeviceState *socdev, Error **errp) | ||
53 | { | ||
54 | - Exynos4210State *s = g_new0(Exynos4210State, 1); | ||
55 | + Exynos4210State *s = EXYNOS4210_SOC(socdev); | ||
56 | + MemoryRegion *system_mem = get_system_memory(); | ||
57 | qemu_irq gate_irq[EXYNOS4210_NCPUS][EXYNOS4210_IRQ_GATE_NINPUTS]; | ||
58 | SysBusDevice *busdev; | ||
59 | DeviceState *dev; | ||
60 | @@ -XXX,XX +XXX,XX @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem) | ||
61 | qemu_irq_invert(s->irq_table[exynos4210_get_irq(36, 1)]), 32); | ||
62 | pl330_create(EXYNOS4210_PL330_BASE2_ADDR, | ||
63 | qemu_irq_invert(s->irq_table[exynos4210_get_irq(34, 1)]), 1); | ||
64 | - | ||
65 | - return s; | ||
66 | } | ||
74 | + | 67 | + |
75 | + uint32_t pclk_freq; | 68 | +static void exynos4210_class_init(ObjectClass *klass, void *data) |
76 | +} AspeedWDTState; | ||
77 | + | ||
78 | +#endif /* ASPEED_WDT_H */ | ||
79 | diff --git a/hw/watchdog/wdt_aspeed.c b/hw/watchdog/wdt_aspeed.c | ||
80 | new file mode 100644 | ||
81 | index XXXXXXX..XXXXXXX | ||
82 | --- /dev/null | ||
83 | +++ b/hw/watchdog/wdt_aspeed.c | ||
84 | @@ -XXX,XX +XXX,XX @@ | ||
85 | +/* | ||
86 | + * ASPEED Watchdog Controller | ||
87 | + * | ||
88 | + * Copyright (C) 2016-2017 IBM Corp. | ||
89 | + * | ||
90 | + * This code is licensed under the GPL version 2 or later. See the | ||
91 | + * COPYING file in the top-level directory. | ||
92 | + */ | ||
93 | + | ||
94 | +#include "qemu/osdep.h" | ||
95 | +#include "qemu/log.h" | ||
96 | +#include "sysemu/watchdog.h" | ||
97 | +#include "hw/sysbus.h" | ||
98 | +#include "qemu/timer.h" | ||
99 | +#include "hw/watchdog/wdt_aspeed.h" | ||
100 | + | ||
101 | +#define WDT_STATUS (0x00 / 4) | ||
102 | +#define WDT_RELOAD_VALUE (0x04 / 4) | ||
103 | +#define WDT_RESTART (0x08 / 4) | ||
104 | +#define WDT_CTRL (0x0C / 4) | ||
105 | +#define WDT_CTRL_RESET_MODE_SOC (0x00 << 5) | ||
106 | +#define WDT_CTRL_RESET_MODE_FULL_CHIP (0x01 << 5) | ||
107 | +#define WDT_CTRL_1MHZ_CLK BIT(4) | ||
108 | +#define WDT_CTRL_WDT_EXT BIT(3) | ||
109 | +#define WDT_CTRL_WDT_INTR BIT(2) | ||
110 | +#define WDT_CTRL_RESET_SYSTEM BIT(1) | ||
111 | +#define WDT_CTRL_ENABLE BIT(0) | ||
112 | + | ||
113 | +#define WDT_TIMEOUT_STATUS (0x10 / 4) | ||
114 | +#define WDT_TIMEOUT_CLEAR (0x14 / 4) | ||
115 | +#define WDT_RESET_WDITH (0x18 / 4) | ||
116 | + | ||
117 | +#define WDT_RESTART_MAGIC 0x4755 | ||
118 | + | ||
119 | +static bool aspeed_wdt_is_enabled(const AspeedWDTState *s) | ||
120 | +{ | ||
121 | + return s->regs[WDT_CTRL] & WDT_CTRL_ENABLE; | ||
122 | +} | ||
123 | + | ||
124 | +static uint64_t aspeed_wdt_read(void *opaque, hwaddr offset, unsigned size) | ||
125 | +{ | ||
126 | + AspeedWDTState *s = ASPEED_WDT(opaque); | ||
127 | + | ||
128 | + offset >>= 2; | ||
129 | + | ||
130 | + switch (offset) { | ||
131 | + case WDT_STATUS: | ||
132 | + return s->regs[WDT_STATUS]; | ||
133 | + case WDT_RELOAD_VALUE: | ||
134 | + return s->regs[WDT_RELOAD_VALUE]; | ||
135 | + case WDT_RESTART: | ||
136 | + qemu_log_mask(LOG_GUEST_ERROR, | ||
137 | + "%s: read from write-only reg at offset 0x%" | ||
138 | + HWADDR_PRIx "\n", __func__, offset); | ||
139 | + return 0; | ||
140 | + case WDT_CTRL: | ||
141 | + return s->regs[WDT_CTRL]; | ||
142 | + case WDT_TIMEOUT_STATUS: | ||
143 | + case WDT_TIMEOUT_CLEAR: | ||
144 | + case WDT_RESET_WDITH: | ||
145 | + qemu_log_mask(LOG_UNIMP, | ||
146 | + "%s: uninmplemented read at offset 0x%" HWADDR_PRIx "\n", | ||
147 | + __func__, offset); | ||
148 | + return 0; | ||
149 | + default: | ||
150 | + qemu_log_mask(LOG_GUEST_ERROR, | ||
151 | + "%s: Out-of-bounds read at offset 0x%" HWADDR_PRIx "\n", | ||
152 | + __func__, offset); | ||
153 | + return 0; | ||
154 | + } | ||
155 | + | ||
156 | +} | ||
157 | + | ||
158 | +static void aspeed_wdt_reload(AspeedWDTState *s, bool pclk) | ||
159 | +{ | ||
160 | + uint32_t reload; | ||
161 | + | ||
162 | + if (pclk) { | ||
163 | + reload = muldiv64(s->regs[WDT_RELOAD_VALUE], NANOSECONDS_PER_SECOND, | ||
164 | + s->pclk_freq); | ||
165 | + } else { | ||
166 | + reload = s->regs[WDT_RELOAD_VALUE] * 1000; | ||
167 | + } | ||
168 | + | ||
169 | + if (aspeed_wdt_is_enabled(s)) { | ||
170 | + timer_mod(s->timer, qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + reload); | ||
171 | + } | ||
172 | +} | ||
173 | + | ||
174 | +static void aspeed_wdt_write(void *opaque, hwaddr offset, uint64_t data, | ||
175 | + unsigned size) | ||
176 | +{ | ||
177 | + AspeedWDTState *s = ASPEED_WDT(opaque); | ||
178 | + bool enable = data & WDT_CTRL_ENABLE; | ||
179 | + | ||
180 | + offset >>= 2; | ||
181 | + | ||
182 | + switch (offset) { | ||
183 | + case WDT_STATUS: | ||
184 | + qemu_log_mask(LOG_GUEST_ERROR, | ||
185 | + "%s: write to read-only reg at offset 0x%" | ||
186 | + HWADDR_PRIx "\n", __func__, offset); | ||
187 | + break; | ||
188 | + case WDT_RELOAD_VALUE: | ||
189 | + s->regs[WDT_RELOAD_VALUE] = data; | ||
190 | + break; | ||
191 | + case WDT_RESTART: | ||
192 | + if ((data & 0xFFFF) == WDT_RESTART_MAGIC) { | ||
193 | + s->regs[WDT_STATUS] = s->regs[WDT_RELOAD_VALUE]; | ||
194 | + aspeed_wdt_reload(s, !(data & WDT_CTRL_1MHZ_CLK)); | ||
195 | + } | ||
196 | + break; | ||
197 | + case WDT_CTRL: | ||
198 | + if (enable && !aspeed_wdt_is_enabled(s)) { | ||
199 | + s->regs[WDT_CTRL] = data; | ||
200 | + aspeed_wdt_reload(s, !(data & WDT_CTRL_1MHZ_CLK)); | ||
201 | + } else if (!enable && aspeed_wdt_is_enabled(s)) { | ||
202 | + s->regs[WDT_CTRL] = data; | ||
203 | + timer_del(s->timer); | ||
204 | + } | ||
205 | + break; | ||
206 | + case WDT_TIMEOUT_STATUS: | ||
207 | + case WDT_TIMEOUT_CLEAR: | ||
208 | + case WDT_RESET_WDITH: | ||
209 | + qemu_log_mask(LOG_UNIMP, | ||
210 | + "%s: uninmplemented write at offset 0x%" HWADDR_PRIx "\n", | ||
211 | + __func__, offset); | ||
212 | + break; | ||
213 | + default: | ||
214 | + qemu_log_mask(LOG_GUEST_ERROR, | ||
215 | + "%s: Out-of-bounds write at offset 0x%" HWADDR_PRIx "\n", | ||
216 | + __func__, offset); | ||
217 | + } | ||
218 | + return; | ||
219 | +} | ||
220 | + | ||
221 | +static WatchdogTimerModel model = { | ||
222 | + .wdt_name = TYPE_ASPEED_WDT, | ||
223 | + .wdt_description = "Aspeed watchdog device", | ||
224 | +}; | ||
225 | + | ||
226 | +static const VMStateDescription vmstate_aspeed_wdt = { | ||
227 | + .name = "vmstate_aspeed_wdt", | ||
228 | + .version_id = 0, | ||
229 | + .minimum_version_id = 0, | ||
230 | + .fields = (VMStateField[]) { | ||
231 | + VMSTATE_TIMER_PTR(timer, AspeedWDTState), | ||
232 | + VMSTATE_UINT32_ARRAY(regs, AspeedWDTState, ASPEED_WDT_REGS_MAX), | ||
233 | + VMSTATE_END_OF_LIST() | ||
234 | + } | ||
235 | +}; | ||
236 | + | ||
237 | +static const MemoryRegionOps aspeed_wdt_ops = { | ||
238 | + .read = aspeed_wdt_read, | ||
239 | + .write = aspeed_wdt_write, | ||
240 | + .endianness = DEVICE_LITTLE_ENDIAN, | ||
241 | + .valid.min_access_size = 4, | ||
242 | + .valid.max_access_size = 4, | ||
243 | + .valid.unaligned = false, | ||
244 | +}; | ||
245 | + | ||
246 | +static void aspeed_wdt_reset(DeviceState *dev) | ||
247 | +{ | ||
248 | + AspeedWDTState *s = ASPEED_WDT(dev); | ||
249 | + | ||
250 | + s->regs[WDT_STATUS] = 0x3EF1480; | ||
251 | + s->regs[WDT_RELOAD_VALUE] = 0x03EF1480; | ||
252 | + s->regs[WDT_RESTART] = 0; | ||
253 | + s->regs[WDT_CTRL] = 0; | ||
254 | + | ||
255 | + timer_del(s->timer); | ||
256 | +} | ||
257 | + | ||
258 | +static void aspeed_wdt_timer_expired(void *dev) | ||
259 | +{ | ||
260 | + AspeedWDTState *s = ASPEED_WDT(dev); | ||
261 | + | ||
262 | + qemu_log_mask(CPU_LOG_RESET, "Watchdog timer expired.\n"); | ||
263 | + watchdog_perform_action(); | ||
264 | + timer_del(s->timer); | ||
265 | +} | ||
266 | + | ||
267 | +#define PCLK_HZ 24000000 | ||
268 | + | ||
269 | +static void aspeed_wdt_realize(DeviceState *dev, Error **errp) | ||
270 | +{ | ||
271 | + SysBusDevice *sbd = SYS_BUS_DEVICE(dev); | ||
272 | + AspeedWDTState *s = ASPEED_WDT(dev); | ||
273 | + | ||
274 | + s->timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, aspeed_wdt_timer_expired, dev); | ||
275 | + | ||
276 | + /* FIXME: This setting should be derived from the SCU hw strapping | ||
277 | + * register SCU70 | ||
278 | + */ | ||
279 | + s->pclk_freq = PCLK_HZ; | ||
280 | + | ||
281 | + memory_region_init_io(&s->iomem, OBJECT(s), &aspeed_wdt_ops, s, | ||
282 | + TYPE_ASPEED_WDT, ASPEED_WDT_REGS_MAX * 4); | ||
283 | + sysbus_init_mmio(sbd, &s->iomem); | ||
284 | +} | ||
285 | + | ||
286 | +static void aspeed_wdt_class_init(ObjectClass *klass, void *data) | ||
287 | +{ | 69 | +{ |
288 | + DeviceClass *dc = DEVICE_CLASS(klass); | 70 | + DeviceClass *dc = DEVICE_CLASS(klass); |
289 | + | 71 | + |
290 | + dc->realize = aspeed_wdt_realize; | 72 | + dc->realize = exynos4210_realize; |
291 | + dc->reset = aspeed_wdt_reset; | ||
292 | + set_bit(DEVICE_CATEGORY_MISC, dc->categories); | ||
293 | + dc->vmsd = &vmstate_aspeed_wdt; | ||
294 | +} | 73 | +} |
295 | + | 74 | + |
296 | +static const TypeInfo aspeed_wdt_info = { | 75 | +static const TypeInfo exynos4210_info = { |
76 | + .name = TYPE_EXYNOS4210_SOC, | ||
297 | + .parent = TYPE_SYS_BUS_DEVICE, | 77 | + .parent = TYPE_SYS_BUS_DEVICE, |
298 | + .name = TYPE_ASPEED_WDT, | 78 | + .instance_size = sizeof(Exynos4210State), |
299 | + .instance_size = sizeof(AspeedWDTState), | 79 | + .class_init = exynos4210_class_init, |
300 | + .class_init = aspeed_wdt_class_init, | ||
301 | +}; | 80 | +}; |
302 | + | 81 | + |
303 | +static void wdt_aspeed_register_types(void) | 82 | +static void exynos4210_register_types(void) |
304 | +{ | 83 | +{ |
305 | + watchdog_add_model(&model); | 84 | + type_register_static(&exynos4210_info); |
306 | + type_register_static(&aspeed_wdt_info); | ||
307 | +} | 85 | +} |
308 | + | 86 | + |
309 | +type_init(wdt_aspeed_register_types) | 87 | +type_init(exynos4210_register_types) |
88 | diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c | ||
89 | index XXXXXXX..XXXXXXX 100644 | ||
90 | --- a/hw/arm/exynos4_boards.c | ||
91 | +++ b/hw/arm/exynos4_boards.c | ||
92 | @@ -XXX,XX +XXX,XX @@ typedef enum Exynos4BoardType { | ||
93 | } Exynos4BoardType; | ||
94 | |||
95 | typedef struct Exynos4BoardState { | ||
96 | - Exynos4210State *soc; | ||
97 | + Exynos4210State soc; | ||
98 | MemoryRegion dram0_mem; | ||
99 | MemoryRegion dram1_mem; | ||
100 | } Exynos4BoardState; | ||
101 | @@ -XXX,XX +XXX,XX @@ exynos4_boards_init_common(MachineState *machine, | ||
102 | exynos4_boards_init_ram(s, get_system_memory(), | ||
103 | exynos4_board_ram_size[board_type]); | ||
104 | |||
105 | - s->soc = exynos4210_init(get_system_memory()); | ||
106 | + object_initialize(&s->soc, sizeof(s->soc), TYPE_EXYNOS4210_SOC); | ||
107 | + qdev_set_parent_bus(DEVICE(&s->soc), sysbus_get_default()); | ||
108 | + object_property_set_bool(OBJECT(&s->soc), true, "realized", | ||
109 | + &error_fatal); | ||
110 | |||
111 | return s; | ||
112 | } | ||
113 | @@ -XXX,XX +XXX,XX @@ static void smdkc210_init(MachineState *machine) | ||
114 | EXYNOS4_BOARD_SMDKC210); | ||
115 | |||
116 | lan9215_init(SMDK_LAN9118_BASE_ADDR, | ||
117 | - qemu_irq_invert(s->soc->irq_table[exynos4210_get_irq(37, 1)])); | ||
118 | + qemu_irq_invert(s->soc.irq_table[exynos4210_get_irq(37, 1)])); | ||
119 | arm_load_kernel(ARM_CPU(first_cpu), &exynos4_board_binfo); | ||
120 | } | ||
121 | |||
310 | -- | 122 | -- |
311 | 2.7.4 | 123 | 2.20.1 |
312 | 124 | ||
313 | 125 | diff view generated by jsdifflib |
Deleted patch | |||
---|---|---|---|
1 | From: Cédric Le Goater <clg@kaod.org> | ||
2 | 1 | ||
3 | This enables reboot of a guest from U-Boot and Linux. | ||
4 | |||
5 | Signed-off-by: Cédric Le Goater <clg@kaod.org> | ||
6 | Reviewed-by: Joel Stanley <joel@jms.id.au> | ||
7 | Message-id: 1485452251-1593-3-git-send-email-clg@kaod.org | ||
8 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | ||
9 | --- | ||
10 | include/hw/arm/aspeed_soc.h | 2 ++ | ||
11 | hw/arm/aspeed_soc.c | 13 +++++++++++++ | ||
12 | 2 files changed, 15 insertions(+) | ||
13 | |||
14 | diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h | ||
15 | index XXXXXXX..XXXXXXX 100644 | ||
16 | --- a/include/hw/arm/aspeed_soc.h | ||
17 | +++ b/include/hw/arm/aspeed_soc.h | ||
18 | @@ -XXX,XX +XXX,XX @@ | ||
19 | #include "hw/timer/aspeed_timer.h" | ||
20 | #include "hw/i2c/aspeed_i2c.h" | ||
21 | #include "hw/ssi/aspeed_smc.h" | ||
22 | +#include "hw/watchdog/wdt_aspeed.h" | ||
23 | |||
24 | #define ASPEED_SPIS_NUM 2 | ||
25 | |||
26 | @@ -XXX,XX +XXX,XX @@ typedef struct AspeedSoCState { | ||
27 | AspeedSMCState fmc; | ||
28 | AspeedSMCState spi[ASPEED_SPIS_NUM]; | ||
29 | AspeedSDMCState sdmc; | ||
30 | + AspeedWDTState wdt; | ||
31 | } AspeedSoCState; | ||
32 | |||
33 | #define TYPE_ASPEED_SOC "aspeed-soc" | ||
34 | diff --git a/hw/arm/aspeed_soc.c b/hw/arm/aspeed_soc.c | ||
35 | index XXXXXXX..XXXXXXX 100644 | ||
36 | --- a/hw/arm/aspeed_soc.c | ||
37 | +++ b/hw/arm/aspeed_soc.c | ||
38 | @@ -XXX,XX +XXX,XX @@ | ||
39 | #define ASPEED_SOC_SCU_BASE 0x1E6E2000 | ||
40 | #define ASPEED_SOC_SRAM_BASE 0x1E720000 | ||
41 | #define ASPEED_SOC_TIMER_BASE 0x1E782000 | ||
42 | +#define ASPEED_SOC_WDT_BASE 0x1E785000 | ||
43 | #define ASPEED_SOC_I2C_BASE 0x1E78A000 | ||
44 | |||
45 | static const int uart_irqs[] = { 9, 32, 33, 34, 10 }; | ||
46 | @@ -XXX,XX +XXX,XX @@ static void aspeed_soc_init(Object *obj) | ||
47 | sc->info->silicon_rev); | ||
48 | object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc), | ||
49 | "ram-size", &error_abort); | ||
50 | + | ||
51 | + object_initialize(&s->wdt, sizeof(s->wdt), TYPE_ASPEED_WDT); | ||
52 | + object_property_add_child(obj, "wdt", OBJECT(&s->wdt), NULL); | ||
53 | + qdev_set_parent_bus(DEVICE(&s->wdt), sysbus_get_default()); | ||
54 | } | ||
55 | |||
56 | static void aspeed_soc_realize(DeviceState *dev, Error **errp) | ||
57 | @@ -XXX,XX +XXX,XX @@ static void aspeed_soc_realize(DeviceState *dev, Error **errp) | ||
58 | return; | ||
59 | } | ||
60 | sysbus_mmio_map(SYS_BUS_DEVICE(&s->sdmc), 0, ASPEED_SOC_SDMC_BASE); | ||
61 | + | ||
62 | + /* Watch dog */ | ||
63 | + object_property_set_bool(OBJECT(&s->wdt), true, "realized", &err); | ||
64 | + if (err) { | ||
65 | + error_propagate(errp, err); | ||
66 | + return; | ||
67 | + } | ||
68 | + sysbus_mmio_map(SYS_BUS_DEVICE(&s->wdt), 0, ASPEED_SOC_WDT_BASE); | ||
69 | } | ||
70 | |||
71 | static void aspeed_soc_class_init(ObjectClass *oc, void *data) | ||
72 | -- | ||
73 | 2.7.4 | ||
74 | |||
75 | diff view generated by jsdifflib |