hw/intc/arm_gicv3_its: Provide read accessor for translation_ops
authorPeter Maydell <peter.maydell@linaro.org>
Sat, 22 Jan 2022 18:24:40 +0000 (18:24 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Fri, 28 Jan 2022 14:29:47 +0000 (14:29 +0000)
commit7e062b98a2541ca9a632160aadbe8574c8bdce24
tree2c5922ba768e162dd3079f198bafca5ead0bf066
parent1611956bce06b0721ea949e24c089ef22967672a
hw/intc/arm_gicv3_its: Provide read accessor for translation_ops

The MemoryRegionOps gicv3_its_translation_ops currently provides only
a .write_with_attrs function, because the only register in this
region is the write-only GITS_TRANSLATER.  However, if you don't
provide a read function and the guest tries reading from this memory
region, QEMU will crash because
memory_region_read_with_attrs_accessor() calls a NULL pointer.

Add a read function which always returns 0, to cover both bogus
attempts to read GITS_TRANSLATER and also reads from the rest of the
region, which is documented to be reserved, RES0.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20220122182444.724087-11-peter.maydell@linaro.org
hw/intc/arm_gicv3_its.c