Re: [PATCH 0/3] hw/intc/loongarch_extioi: Fix undefined behaviour with bit array APIs

2024-11-18 Thread Philippe Mathieu-Daudé
On 8/11/24 14:55, Peter Maydell wrote: Peter Maydell (3): bitops.h: Define bit operations on 'uint32_t' arrays hw/intc/arm_gicv3: Use bitops.h uint32_t bit array functions hw/intc/loongarch_extioi: Use set_bit32() and clear_bit32() for s->isr Series: Reviewed-by: Philippe Mathieu-Daud

Re: [PATCH 0/3] hw/intc/loongarch_extioi: Fix undefined behaviour with bit array APIs

2024-11-18 Thread Peter Maydell
Any chance of a review on patches 1 and 2 here? (I guess I should have cc'd qemu-arm on this, since patch 2 is for GICv3.) thanks -- PMM On Fri, 8 Nov 2024 at 13:55, Peter Maydell wrote: > > The primary aim of this series is to fix some undefined behaviour in > loongarch_extioi which you can see

[PATCH 0/3] hw/intc/loongarch_extioi: Fix undefined behaviour with bit array APIs

2024-11-08 Thread Peter Maydell
The primary aim of this series is to fix some undefined behaviour in loongarch_extioi which you can see if you run the functional test loongarch64-virt with a QEMU built with the clang undefined-behaviour sanitizer: include/qemu/bitops.h:41:5: runtime error: store to misaligned address 0x5974