Not very much here, but several people have fallen over

the vector operation segfault bug, so let's get the fix

into master.

The following changes since commit d418238dca7b4e0b124135827ead3076233052b1:

Merge remote-tracking branch 'remotes/rth/tags/pull-rng-20190522' into staging (2019-05-23 12:57:17 +0100)

https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20190523

for you to fetch changes up to 98e4f4fdb8ea05d840f51f47125924c2bb9df2df:

hw/arm/exynos4210: QOM'ify the Exynos4210 SoC (2019-05-23 14:47:44 +0100)

* exynos4210: QOM'ify the Exynos4210 SoC

* exynos4210: Add DMA support for the Exynos4210

* arm_gicv3: Fix writes to ICC_CTLR_EL3

* arm_gicv3: Fix write of ICH_VMCR_EL2.{VBPR0, VBPR1}

* target/arm: Fix vector operation segfault

* target/arm: Minor improvements to BFXIL, EXTR

Alistair Francis (1):

target/arm: Fix vector operation segfault

Guenter Roeck (1):

hw/arm/exynos4210: Add DMA support for the Exynos4210

Peter Maydell (5):

arm: Move system_clock_scale to armv7m_systick.h

arm: Remove unnecessary includes of hw/arm/arm.h

arm: Rename hw/arm/arm.h to hw/arm/boot.h

hw/intc/arm_gicv3: Fix write of ICH_VMCR_EL2.{VBPR0, VBPR1}

hw/intc/arm_gicv3: Fix writes to ICC_CTLR_EL3

Philippe Mathieu-Daudé (3):

hw/arm/exynos4: Remove unuseful debug code

hw/arm/exynos4: Use the IEC binary prefix definitions

hw/arm/exynos4210: QOM'ify the Exynos4210 SoC

target/arm: Use extract2 for EXTR

target/arm: Simplify BFXIL expansion

include/hw/arm/allwinner-a10.h | 2 +-

include/hw/arm/aspeed_soc.h | 1 -

include/hw/arm/bcm2836.h | 1 -

include/hw/arm/{arm.h => boot.h} | 12 +++------

include/hw/arm/exynos4210.h | 9 +++++--

include/hw/arm/fsl-imx25.h | 2 +-

include/hw/arm/fsl-imx31.h | 2 +-

include/hw/arm/fsl-imx6.h | 2 +-

include/hw/arm/fsl-imx6ul.h | 2 +-

include/hw/arm/fsl-imx7.h | 2 +-

include/hw/arm/virt.h | 2 +-

include/hw/arm/xlnx-versal.h | 2 +-

include/hw/arm/xlnx-zynqmp.h | 2 +-

include/hw/timer/armv7m_systick.h | 22 ++++++++++++++++

hw/arm/armsse.c | 2 +-

hw/arm/armv7m.c | 2 +-

hw/arm/aspeed.c | 2 +-

hw/arm/boot.c | 2 +-

hw/arm/collie.c | 2 +-

hw/arm/exynos4210.c | 54 ++++++++++++++++++++++++++++++++++++---

hw/arm/exynos4_boards.c | 40 ++++++++---------------------

hw/arm/highbank.c | 2 +-

hw/arm/integratorcp.c | 2 +-

hw/arm/mainstone.c | 2 +-

hw/arm/microbit.c | 2 +-

hw/arm/mps2-tz.c | 2 +-

hw/arm/mps2.c | 2 +-

hw/arm/msf2-soc.c | 1 -

hw/arm/msf2-som.c | 2 +-

hw/arm/musca.c | 2 +-

hw/arm/musicpal.c | 2 +-

hw/arm/netduino2.c | 2 +-

hw/arm/nrf51_soc.c | 2 +-

hw/arm/nseries.c | 2 +-

hw/arm/omap1.c | 2 +-

hw/arm/omap2.c | 2 +-

hw/arm/omap_sx1.c | 2 +-

hw/arm/palm.c | 2 +-

hw/arm/raspi.c | 2 +-

hw/arm/realview.c | 2 +-

hw/arm/spitz.c | 2 +-

hw/arm/stellaris.c | 2 +-

hw/arm/stm32f205_soc.c | 2 +-

hw/arm/strongarm.c | 2 +-

hw/arm/tosa.c | 2 +-

hw/arm/versatilepb.c | 2 +-

hw/arm/vexpress.c | 2 +-

hw/arm/virt.c | 2 +-

hw/arm/xilinx_zynq.c | 2 +-

hw/arm/xlnx-versal.c | 2 +-

hw/arm/z2.c | 2 +-

hw/intc/arm_gicv3_cpuif.c | 6 ++---

hw/intc/armv7m_nvic.c | 1 -

target/arm/arm-semi.c | 1 -

target/arm/cpu.c | 1 -

target/arm/cpu64.c | 1 -

target/arm/kvm.c | 1 -

target/arm/kvm32.c | 1 -

target/arm/kvm64.c | 1 -

target/arm/translate-a64.c | 44 ++++++++++++++++---------------

target/arm/translate.c | 4 +--

61 files changed, 164 insertions(+), 123 deletions(-)

rename include/hw/arm/{arm.h => boot.h} (96%)

The mask implied by the extract is redundant with the one

implied by the deposit. Also, fix spelling of BFXIL.

Message-id: 20190514011129.11330-3-richard.henderson@linaro.org

target/arm/translate-a64.c | 6 +++---

1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c

index XXXXXXX..XXXXXXX 100644

--- a/target/arm/translate-a64.c

+++ b/target/arm/translate-a64.c

@@ -XXX,XX +XXX,XX @@ static void disas_bitfield(DisasContext *s, uint32_t insn)

tcg_gen_extract_i64(tcg_rd, tcg_tmp, ri, len);

return;

}

- /* opc == 1, BXFIL fall through to deposit */

- tcg_gen_extract_i64(tcg_tmp, tcg_tmp, ri, len);

+ /* opc == 1, BFXIL fall through to deposit */

+ tcg_gen_shri_i64(tcg_tmp, tcg_tmp, ri);

pos = 0;

} else {

/* Handle the ri > si case with a deposit

@@ -XXX,XX +XXX,XX @@ static void disas_bitfield(DisasContext *s, uint32_t insn)

len = ri;

}

- if (opc == 1) { /* BFM, BXFIL */

+ if (opc == 1) { /* BFM, BFXIL */

tcg_gen_deposit_i64(tcg_rd, tcg_rd, tcg_tmp, pos, len);

} else {

/* SBFM or UBFM: We start with zero, and we haven't modified

2.20.1

This is, after all, how we implement extract2 in tcg/aarch64.

Message-id: 20190514011129.11330-2-richard.henderson@linaro.org

target/arm/translate-a64.c | 38 ++++++++++++++++++++------------------

1 file changed, 20 insertions(+), 18 deletions(-)

diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c

--- a/target/arm/translate-a64.c

+++ b/target/arm/translate-a64.c

@@ -XXX,XX +XXX,XX @@ static void disas_extract(DisasContext *s, uint32_t insn)

} else {

tcg_gen_ext32u_i64(tcg_rd, cpu_reg(s, rm));

}

- } else if (rm == rn) { /* ROR */

- tcg_rm = cpu_reg(s, rm);

- if (sf) {

- tcg_gen_rotri_i64(tcg_rd, tcg_rm, imm);

- } else {

- TCGv_i32 tmp = tcg_temp_new_i32();

- tcg_gen_extrl_i64_i32(tmp, tcg_rm);

- tcg_gen_rotri_i32(tmp, tmp, imm);

- tcg_gen_extu_i32_i64(tcg_rd, tmp);

- tcg_temp_free_i32(tmp);

- }

} else {

- tcg_rm = read_cpu_reg(s, rm, sf);

- tcg_rn = read_cpu_reg(s, rn, sf);

- tcg_gen_shri_i64(tcg_rm, tcg_rm, imm);

- tcg_gen_shli_i64(tcg_rn, tcg_rn, bitsize - imm);

- tcg_gen_or_i64(tcg_rd, tcg_rm, tcg_rn);

- if (!sf) {

- tcg_gen_ext32u_i64(tcg_rd, tcg_rd);

+ tcg_rm = cpu_reg(s, rm);

+ tcg_rn = cpu_reg(s, rn);

+

+ if (sf) {

+ /* Specialization to ROR happens in EXTRACT2. */

+ tcg_gen_extract2_i64(tcg_rd, tcg_rm, tcg_rn, imm);

+ } else {

+ TCGv_i32 t0 = tcg_temp_new_i32();

+

+ tcg_gen_extrl_i64_i32(t0, tcg_rm);

+ if (rm == rn) {

+ tcg_gen_rotri_i32(t0, t0, imm);

+ } else {

+ TCGv_i32 t1 = tcg_temp_new_i32();

+ tcg_gen_extrl_i64_i32(t1, tcg_rn);

+ tcg_gen_extract2_i32(t0, t0, t1, imm);

+ tcg_temp_free_i32(t1);

+ }

+ tcg_gen_extu_i32_i64(tcg_rd, t0);

+ tcg_temp_free_i32(t0);

}

}

}

2.20.1

From: Philippe Mathieu-Daudé <philmd@redhat.com>

It eases code review, unit is explicit.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Message-id: 20190520214342.13709-3-philmd@redhat.com

hw/arm/exynos4_boards.c | 5 +++--

diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c

--- a/hw/arm/exynos4_boards.c

+++ b/hw/arm/exynos4_boards.c

@@ -XXX,XX +XXX,XX @@

*/

#include "qemu/osdep.h"

+#include "qemu/units.h"

#include "qapi/error.h"

#include "qemu/error-report.h"

#include "qemu-common.h"

@@ -XXX,XX +XXX,XX @@ static int exynos4_board_smp_bootreg_addr[EXYNOS4_NUM_OF_BOARDS] = {

};

static unsigned long exynos4_board_ram_size[EXYNOS4_NUM_OF_BOARDS] = {

- [EXYNOS4_BOARD_NURI] = 0x40000000,

- [EXYNOS4_BOARD_SMDKC210] = 0x40000000,

+ [EXYNOS4_BOARD_NURI] = 1 * GiB,

+ [EXYNOS4_BOARD_SMDKC210] = 1 * GiB,

};

static struct arm_boot_info exynos4_board_binfo = {

2.20.1

The system_clock_scale global is used only by the armv7m systick

device; move the extern declaration to the armv7m_systick.h header,

and expand the comment to explain what it is and that it should

ideally be replaced with a different approach.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190516163857.6430-2-peter.maydell@linaro.org

include/hw/arm/arm.h | 4 ----

include/hw/timer/armv7m_systick.h | 22 ++++++++++++++++++++++

2 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/include/hw/arm/arm.h b/include/hw/arm/arm.h

--- a/include/hw/arm/arm.h

+++ b/include/hw/arm/arm.h

@@ -XXX,XX +XXX,XX @@ void arm_write_secure_board_setup_dummy_smc(ARMCPU *cpu,

const struct arm_boot_info *info,

hwaddr mvbar_addr);

-/* Multiplication factor to convert from system clock ticks to qemu timer

- ticks. */

-extern int system_clock_scale;

-

#endif /* HW_ARM_H */

diff --git a/include/hw/timer/armv7m_systick.h b/include/hw/timer/armv7m_systick.h

index XXXXXXX..XXXXXXX 100644

--- a/include/hw/timer/armv7m_systick.h

+++ b/include/hw/timer/armv7m_systick.h

@@ -XXX,XX +XXX,XX @@ typedef struct SysTickState {

qemu_irq irq;

} SysTickState;

+ * Multiplication factor to convert from system clock ticks to qemu timer

+ * ticks. This should be set (by board code, usually) to a value

+ * equal to NANOSECONDS_PER_SECOND / frq, where frq is the clock frequency

+ * in Hz of the CPU.

+ *

+ * This value is used by the systick device when it is running in

+ * its "use the CPU clock" mode (ie when SYST_CSR.CLKSOURCE == 1) to

+ * set how fast the timer should tick.

+ *

+ * TODO: we should refactor this so that rather than using a global

+ * we use a device property or something similar. This is complicated

+ * because (a) the property would need to be plumbed through from the

+ * board code down through various layers to the systick device

+ * and (b) the property needs to be modifiable after realize, because

+ * the stellaris board uses this to implement the behaviour where the

+ * guest can reprogram the PLL registers to downclock the CPU, and the

+ * systick device needs to react accordingly. Possibly this should

+ * be deferred until we have a good API for modelling clock trees.

+extern int system_clock_scale;

2.20.1

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Message-id: 20190520214342.13709-5-philmd@redhat.com

include/hw/arm/exynos4210.h | 9 +++++++--

hw/arm/exynos4210.c | 28 ++++++++++++++++++++++++----

hw/arm/exynos4_boards.c | 9 ++++++---

3 files changed, 37 insertions(+), 9 deletions(-)

diff --git a/include/hw/arm/exynos4210.h b/include/hw/arm/exynos4210.h

--- a/include/hw/arm/exynos4210.h

+++ b/include/hw/arm/exynos4210.h

@@ -XXX,XX +XXX,XX @@ typedef struct Exynos4210Irq {

} Exynos4210Irq;

typedef struct Exynos4210State {

+ /*< private >*/

+ SysBusDevice parent_obj;

+ /*< public >*/

ARMCPU *cpu[EXYNOS4210_NCPUS];

Exynos4210Irq irqs;

qemu_irq *irq_table;

@@ -XXX,XX +XXX,XX @@ typedef struct Exynos4210State {

I2CBus *i2c_if[EXYNOS4210_I2C_NUMBER];

} Exynos4210State;

+#define TYPE_EXYNOS4210_SOC "exynos4210"

+#define EXYNOS4210_SOC(obj) \

+ OBJECT_CHECK(Exynos4210State, obj, TYPE_EXYNOS4210_SOC)

+

void exynos4210_write_secondary(ARMCPU *cpu,

const struct arm_boot_info *info);

-Exynos4210State *exynos4210_init(MemoryRegion *system_mem);

-

/* Initialize exynos4210 IRQ subsystem stub */

qemu_irq *exynos4210_init_irq(Exynos4210Irq *env);

diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/exynos4210.c

+++ b/hw/arm/exynos4210.c

@@ -XXX,XX +XXX,XX @@ static void pl330_create(uint32_t base, qemu_irq irq, int nreq)

sysbus_connect_irq(busdev, 0, irq);

}

-Exynos4210State *exynos4210_init(MemoryRegion *system_mem)

+static void exynos4210_realize(DeviceState *socdev, Error **errp)

{

- Exynos4210State *s = g_new0(Exynos4210State, 1);

+ Exynos4210State *s = EXYNOS4210_SOC(socdev);

+ MemoryRegion *system_mem = get_system_memory();

qemu_irq gate_irq[EXYNOS4210_NCPUS][EXYNOS4210_IRQ_GATE_NINPUTS];

SysBusDevice *busdev;

DeviceState *dev;

@@ -XXX,XX +XXX,XX @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem)

qemu_irq_invert(s->irq_table[exynos4210_get_irq(36, 1)]), 32);

pl330_create(EXYNOS4210_PL330_BASE2_ADDR,

qemu_irq_invert(s->irq_table[exynos4210_get_irq(34, 1)]), 1);

-

- return s;

}

+

+static void exynos4210_class_init(ObjectClass *klass, void *data)

+{

+ DeviceClass *dc = DEVICE_CLASS(klass);

+

+ dc->realize = exynos4210_realize;

+}

+

+static const TypeInfo exynos4210_info = {

+ .name = TYPE_EXYNOS4210_SOC,

+ .parent = TYPE_SYS_BUS_DEVICE,

+ .instance_size = sizeof(Exynos4210State),

+ .class_init = exynos4210_class_init,

+};

+

+static void exynos4210_register_types(void)

+{

+ type_register_static(&exynos4210_info);

+}

+

+type_init(exynos4210_register_types)

diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/exynos4_boards.c

+++ b/hw/arm/exynos4_boards.c

@@ -XXX,XX +XXX,XX @@ typedef enum Exynos4BoardType {

} Exynos4BoardType;

typedef struct Exynos4BoardState {

- Exynos4210State *soc;

+ Exynos4210State soc;

MemoryRegion dram0_mem;

MemoryRegion dram1_mem;

} Exynos4BoardState;

@@ -XXX,XX +XXX,XX @@ exynos4_boards_init_common(MachineState *machine,

exynos4_boards_init_ram(s, get_system_memory(),

exynos4_board_ram_size[board_type]);

- s->soc = exynos4210_init(get_system_memory());

+ object_initialize(&s->soc, sizeof(s->soc), TYPE_EXYNOS4210_SOC);

+ qdev_set_parent_bus(DEVICE(&s->soc), sysbus_get_default());

+ object_property_set_bool(OBJECT(&s->soc), true, "realized",

+ &error_fatal);

return s;

}

@@ -XXX,XX +XXX,XX @@ static void smdkc210_init(MachineState *machine)

EXYNOS4_BOARD_SMDKC210);

lan9215_init(SMDK_LAN9118_BASE_ADDR,

- qemu_irq_invert(s->soc->irq_table[exynos4210_get_irq(37, 1)]));

+ qemu_irq_invert(s->soc.irq_table[exynos4210_get_irq(37, 1)]));

arm_load_kernel(ARM_CPU(first_cpu), &exynos4_board_binfo);

}

2.20.1

The ICC_CTLR_EL3 register includes some bits which are aliases

of bits in the ICC_CTLR_EL1(S) and (NS) registers. QEMU chooses

to keep those bits in the cs->icc_ctlr_el1[] struct fields.

Unfortunately a missing '~' in the code to update the bits

in those fields meant that writing to ICC_CTLR_EL3 would corrupt

the ICC_CLTR_EL1 register values.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190520162809.2677-5-peter.maydell@linaro.org

hw/intc/arm_gicv3_cpuif.c | 4 ++--

diff --git a/hw/intc/arm_gicv3_cpuif.c b/hw/intc/arm_gicv3_cpuif.c

--- a/hw/intc/arm_gicv3_cpuif.c

+++ b/hw/intc/arm_gicv3_cpuif.c

@@ -XXX,XX +XXX,XX @@ static void icc_ctlr_el3_write(CPUARMState *env, const ARMCPRegInfo *ri,

trace_gicv3_icc_ctlr_el3_write(gicv3_redist_affid(cs), value);

/* *_EL1NS and *_EL1S bits are aliases into the ICC_CTLR_EL1 bits. */

- cs->icc_ctlr_el1[GICV3_NS] &= (ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE);

+ cs->icc_ctlr_el1[GICV3_NS] &= ~(ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE);

if (value & ICC_CTLR_EL3_EOIMODE_EL1NS) {

cs->icc_ctlr_el1[GICV3_NS] |= ICC_CTLR_EL1_EOIMODE;

}

@@ -XXX,XX +XXX,XX @@ static void icc_ctlr_el3_write(CPUARMState *env, const ARMCPRegInfo *ri,

cs->icc_ctlr_el1[GICV3_NS] |= ICC_CTLR_EL1_CBPR;

}

- cs->icc_ctlr_el1[GICV3_S] &= (ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE);

+ cs->icc_ctlr_el1[GICV3_S] &= ~(ICC_CTLR_EL1_CBPR | ICC_CTLR_EL1_EOIMODE);

if (value & ICC_CTLR_EL3_EOIMODE_EL1S) {

cs->icc_ctlr_el1[GICV3_S] |= ICC_CTLR_EL1_EOIMODE;

}

2.20.1

The hw/arm/arm.h header now only includes declarations relating

to boot.c code, so it is only needed by Arm board or SoC code.

Remove some unnecessary inclusions of it from target/arm files

and from hw/intc/armv7m_nvic.c.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190516163857.6430-3-peter.maydell@linaro.org

hw/intc/armv7m_nvic.c | 1 -

target/arm/arm-semi.c | 1 -

target/arm/cpu.c | 1 -

target/arm/cpu64.c | 1 -

target/arm/kvm.c | 1 -

target/arm/kvm32.c | 1 -

target/arm/kvm64.c | 1 -

7 files changed, 7 deletions(-)

diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c

--- a/hw/intc/armv7m_nvic.c

+++ b/hw/intc/armv7m_nvic.c

#include "cpu.h"

#include "hw/sysbus.h"

#include "qemu/timer.h"

-#include "hw/arm/arm.h"

#include "hw/intc/armv7m_nvic.h"

#include "target/arm/cpu.h"

#include "exec/exec-all.h"

diff --git a/target/arm/arm-semi.c b/target/arm/arm-semi.c

--- a/target/arm/arm-semi.c

+++ b/target/arm/arm-semi.c

#else

#include "qemu-common.h"

#include "exec/gdbstub.h"

-#include "hw/arm/arm.h"

#include "qemu/cutils.h"

#endif

diff --git a/target/arm/cpu.c b/target/arm/cpu.c

--- a/target/arm/cpu.c

+++ b/target/arm/cpu.c

#if !defined(CONFIG_USER_ONLY)

#include "hw/loader.h"

#endif

-#include "hw/arm/arm.h"

#include "sysemu/sysemu.h"

#include "sysemu/hw_accel.h"

#include "kvm_arm.h"

diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c

--- a/target/arm/cpu64.c

+++ b/target/arm/cpu64.c

#if !defined(CONFIG_USER_ONLY)

#include "hw/loader.h"

#endif

-#include "hw/arm/arm.h"

#include "sysemu/sysemu.h"

#include "sysemu/kvm.h"

#include "kvm_arm.h"

diff --git a/target/arm/kvm.c b/target/arm/kvm.c

--- a/target/arm/kvm.c

+++ b/target/arm/kvm.c

#include "cpu.h"

#include "trace.h"

#include "internals.h"

-#include "hw/arm/arm.h"

#include "hw/pci/pci.h"

#include "exec/memattrs.h"

#include "exec/address-spaces.h"

diff --git a/target/arm/kvm32.c b/target/arm/kvm32.c

index XXXXXXX..XXXXXXX 100644

--- a/target/arm/kvm32.c

+++ b/target/arm/kvm32.c

@@ -XXX,XX +XXX,XX @@

#include "sysemu/kvm.h"

#include "kvm_arm.h"

#include "internals.h"

-#include "hw/arm/arm.h"

#include "qemu/log.h"

static inline void set_feature(uint64_t *features, int feature)

diff --git a/target/arm/kvm64.c b/target/arm/kvm64.c

index XXXXXXX..XXXXXXX 100644

--- a/target/arm/kvm64.c

+++ b/target/arm/kvm64.c

@@ -XXX,XX +XXX,XX @@

#include "sysemu/kvm.h"

#include "kvm_arm.h"

#include "internals.h"

-#include "hw/arm/arm.h"

static bool have_guest_debug;

2.20.1

In ich_vmcr_write() we enforce "writes of BPR fields to less than

their minimum sets them to the minimum" by doing a "read vbpr and

write it back" operation. A typo here meant that we weren't handling

writes to these fields correctly, because we were reading from VBPR0

but writing to VBPR1.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190520162809.2677-4-peter.maydell@linaro.org

hw/intc/arm_gicv3_cpuif.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/intc/arm_gicv3_cpuif.c b/hw/intc/arm_gicv3_cpuif.c

--- a/hw/intc/arm_gicv3_cpuif.c

+++ b/hw/intc/arm_gicv3_cpuif.c

@@ -XXX,XX +XXX,XX @@ static void ich_vmcr_write(CPUARMState *env, const ARMCPRegInfo *ri,

/* Enforce "writing BPRs to less than minimum sets them to the minimum"

* by reading and writing back the fields.

- write_vbpr(cs, GICV3_G1, read_vbpr(cs, GICV3_G0));

+ write_vbpr(cs, GICV3_G0, read_vbpr(cs, GICV3_G0));

write_vbpr(cs, GICV3_G1, read_vbpr(cs, GICV3_G1));

gicv3_cpuif_virt_update(cs);

2.20.1

From: Guenter Roeck <linux@roeck-us.net>

QEMU already supports pl330. Instantiate it for Exynos4210.

Relevant part of Linux arch/arm/boot/dts/exynos4.dtsi:

/ {

soc: soc {

amba {

pdma0: pdma@12680000 {

compatible = "arm,pl330", "arm,primecell";

reg = <0x12680000 0x1000>;

interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;

clocks = <&clock CLK_PDMA0>;

clock-names = "apb_pclk";

#dma-cells = <1>;

#dma-channels = <8>;

#dma-requests = <32>;

};

pdma1: pdma@12690000 {

compatible = "arm,pl330", "arm,primecell";

reg = <0x12690000 0x1000>;

interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>;

clocks = <&clock CLK_PDMA1>;

clock-names = "apb_pclk";

#dma-cells = <1>;

#dma-channels = <8>;

#dma-requests = <32>;

};

mdma1: mdma@12850000 {

compatible = "arm,pl330", "arm,primecell";

reg = <0x12850000 0x1000>;

interrupts = <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>;

clocks = <&clock CLK_MDMA>;

clock-names = "apb_pclk";

#dma-cells = <1>;

#dma-channels = <8>;

#dma-requests = <1>;

};

};

};

};

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190520214342.13709-4-philmd@redhat.com

[PMD: Do not set default qdev properties, create the controllers in the SoC

rather than the board (Peter Maydell), add dtsi in commit message]

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

hw/arm/exynos4210.c | 26 ++++++++++++++++++++++++++

1 file changed, 26 insertions(+)

diff --git a/hw/arm/exynos4210.c b/hw/arm/exynos4210.c

--- a/hw/arm/exynos4210.c

+++ b/hw/arm/exynos4210.c

/* EHCI */

#define EXYNOS4210_EHCI_BASE_ADDR 0x12580000

+/* DMA */

+#define EXYNOS4210_PL330_BASE0_ADDR 0x12680000

+#define EXYNOS4210_PL330_BASE1_ADDR 0x12690000

+#define EXYNOS4210_PL330_BASE2_ADDR 0x12850000

+

static uint8_t chipid_and_omr[] = { 0x11, 0x02, 0x21, 0x43,

0x09, 0x00, 0x00, 0x00 };

@@ -XXX,XX +XXX,XX @@ static uint64_t exynos4210_calc_affinity(int cpu)

return (0x9 << ARM_AFF1_SHIFT) | cpu;

}

+static void pl330_create(uint32_t base, qemu_irq irq, int nreq)

+{

+ SysBusDevice *busdev;

+ DeviceState *dev;

+

+ dev = qdev_create(NULL, "pl330");

+ qdev_prop_set_uint8(dev, "num_periph_req", nreq);

+ qdev_init_nofail(dev);

+ busdev = SYS_BUS_DEVICE(dev);

+ sysbus_mmio_map(busdev, 0, base);

+ sysbus_connect_irq(busdev, 0, irq);

+}

+

Exynos4210State *exynos4210_init(MemoryRegion *system_mem)

{

Exynos4210State *s = g_new0(Exynos4210State, 1);

@@ -XXX,XX +XXX,XX @@ Exynos4210State *exynos4210_init(MemoryRegion *system_mem)

sysbus_create_simple(TYPE_EXYNOS4210_EHCI, EXYNOS4210_EHCI_BASE_ADDR,

s->irq_table[exynos4210_get_irq(28, 3)]);

+ /*** DMA controllers ***/

+ pl330_create(EXYNOS4210_PL330_BASE0_ADDR,

+ qemu_irq_invert(s->irq_table[exynos4210_get_irq(35, 1)]), 32);

+ pl330_create(EXYNOS4210_PL330_BASE1_ADDR,

+ qemu_irq_invert(s->irq_table[exynos4210_get_irq(36, 1)]), 32);

+ pl330_create(EXYNOS4210_PL330_BASE2_ADDR,

+ qemu_irq_invert(s->irq_table[exynos4210_get_irq(34, 1)]), 1);

+

return s;

}

2.20.1

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Message-id: 20190520214342.13709-2-philmd@redhat.com

hw/arm/exynos4_boards.c | 24 ------------------------

1 file changed, 24 deletions(-)

diff --git a/hw/arm/exynos4_boards.c b/hw/arm/exynos4_boards.c

--- a/hw/arm/exynos4_boards.c

+++ b/hw/arm/exynos4_boards.c

#include "hw/net/lan9118.h"

#include "hw/boards.h"

-#undef DEBUG

-

-//#define DEBUG

-

-#ifdef DEBUG

- #undef PRINT_DEBUG

- #define PRINT_DEBUG(fmt, args...) \

- do { \

- fprintf(stderr, " [%s:%d] "fmt, __func__, __LINE__, ##args); \

- } while (0)

-#else

- #define PRINT_DEBUG(fmt, args...) do {} while (0)

-#endif

-

#define SMDK_LAN9118_BASE_ADDR 0x05000000

typedef enum Exynos4BoardType {

@@ -XXX,XX +XXX,XX @@ exynos4_boards_init_common(MachineState *machine,

exynos4_board_binfo.gic_cpu_if_addr =

EXYNOS4210_SMP_PRIVATE_BASE_ADDR + 0x100;

- PRINT_DEBUG("\n ram_size: %luMiB [0x%08lx]\n"

- " kernel_filename: %s\n"

- " kernel_cmdline: %s\n"

- " initrd_filename: %s\n",

- exynos4_board_ram_size[board_type] / 1048576,

- exynos4_board_ram_size[board_type],

- machine->kernel_filename,

- machine->kernel_cmdline,

- machine->initrd_filename);

-

exynos4_boards_init_ram(s, get_system_memory(),

exynos4_board_ram_size[board_type]);

2.20.1

From: Alistair Francis <alistair.francis@wdc.com>

Commit 89e68b575 "target/arm: Use vector operations for saturation"

causes this abort() when booting QEMU ARM with a Cortex-A15:

0 0x00007ffff4c2382f in raise () at /usr/lib/libc.so.6

1 0x00007ffff4c0e672 in abort () at /usr/lib/libc.so.6

2 0x00005555559c1839 in disas_neon_data_insn (insn=<optimized out>, s=<optimized out>) at ./target/arm/translate.c:6673

3 0x00005555559c1839 in disas_neon_data_insn (s=<optimized out>, insn=<optimized out>) at ./target/arm/translate.c:6386

4 0x00005555559cd8a4 in disas_arm_insn (insn=4081107068, s=0x7fffe59a9510) at ./target/arm/translate.c:9289

5 0x00005555559cd8a4 in arm_tr_translate_insn (dcbase=0x7fffe59a9510, cpu=<optimized out>) at ./target/arm/translate.c:13612

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

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

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

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

10 0x00005555558ce467 in cpu_exec (cpu=cpu@entry=0x55555686a2f0) at ./accel/tcg/cpu-exec.c:728

11 0x000055555588b0cf in tcg_cpu_exec (cpu=0x55555686a2f0) at ./cpus.c:1431

12 0x000055555588d223 in qemu_tcg_cpu_thread_fn (arg=0x55555686a2f0) at ./cpus.c:1735

13 0x000055555588d223 in qemu_tcg_cpu_thread_fn (arg=arg@entry=0x55555686a2f0) at ./cpus.c:1709

14 0x0000555555d2629a in qemu_thread_start (args=<optimized out>) at ./util/qemu-thread-posix.c:502

15 0x00007ffff4db8a92 in start_thread () at /usr/lib/libpthread.

This patch ensures that we don't hit the abort() in the second switch

case in disas_neon_data_insn() as we will return from the first case.

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

Tested-by: Alex Bennée <alex.bennee@linaro.org>

Message-id: ad91b397f360b2fc7f4087e476f7df5b04d42ddb.1558021877.git.alistair.francis@wdc.com

target/arm/translate.c | 4 ++--

1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/target/arm/translate.c b/target/arm/translate.c

--- a/target/arm/translate.c

+++ b/target/arm/translate.c

@@ -XXX,XX +XXX,XX @@ static int disas_neon_data_insn(DisasContext *s, uint32_t insn)

tcg_gen_gvec_4(rd_ofs, offsetof(CPUARMState, vfp.qc),

rn_ofs, rm_ofs, vec_size, vec_size,

(u ? uqadd_op : sqadd_op) + size);

- break;

+ return 0;

case NEON_3R_VQSUB:

tcg_gen_gvec_4(rd_ofs, offsetof(CPUARMState, vfp.qc),

rn_ofs, rm_ofs, vec_size, vec_size,

(u ? uqsub_op : sqsub_op) + size);

- break;

+ return 0;

case NEON_3R_VMUL: /* VMUL */

if (u) {

2.20.1

The header file hw/arm/arm.h now includes only declarations

relating to hw/arm/boot.c functionality. Rename it accordingly,

and adjust its header comment.

The bulk of this commit was created via

perl -pi -e 's|hw/arm/arm.h|hw/arm/boot.h|' hw/arm/*.c include/hw/arm/*.h

In a few cases we can just delete the #include:

hw/arm/msf2-soc.c, include/hw/arm/aspeed_soc.h and

include/hw/arm/bcm2836.h did not require it.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20190516163857.6430-4-peter.maydell@linaro.org

include/hw/arm/allwinner-a10.h | 2 +-

include/hw/arm/aspeed_soc.h | 1 -

include/hw/arm/bcm2836.h | 1 -

include/hw/arm/{arm.h => boot.h} | 8 ++++----

include/hw/arm/fsl-imx25.h | 2 +-

include/hw/arm/fsl-imx31.h | 2 +-

include/hw/arm/fsl-imx6.h | 2 +-

include/hw/arm/fsl-imx6ul.h | 2 +-

include/hw/arm/fsl-imx7.h | 2 +-

include/hw/arm/virt.h | 2 +-

include/hw/arm/xlnx-versal.h | 2 +-

include/hw/arm/xlnx-zynqmp.h | 2 +-

hw/arm/armsse.c | 2 +-