On Wed, 10 Apr 2019 at 02:12, Stefan Weil <s...@weilnetz.de> wrote: > > When QEMU for ARM is compiled with sanitizers > (--extra-cflags=-fsanitize=address,undefined), I see runtime errors for > qemu-arm which indicate misaligned memory writes: > > /qemu/linux-user/fd-trans.c:1389:18: runtime error: store to misaligned > address 0x42ff4b34 for type 'uint64_t', which requires 8 byte alignment > > The full sanitizer report is available from > https://qemu.weilnetz.de/doc/sanitizer.txt. Some of the reported runtime > errors can easily be fixed, for example by replacing "1 << 31" > (currently 188 matches) by "1U << 31" (currently only 70 matches).
The shift "errors" are not in fact errors -- we compile with -fwrapv to request a dialect of C where left shift of signed integers is well-defined and has 2s complement semantics. Some versions of the clang sanitizers don't correctly handle that flag as meaning that those shifts should no longer generate sanitizer warnings. If you're using a clang without that bug fix you can suppress the warnings by adding "-fno-sanitize=shift-base" to the --extra-cflags you pass to QEMU's configure. The misaligned accesses are definitely bugs. The linux-user code unfortunately has quite a lot of code in its data-structure conversion code that is pretty fast and loose about accessing fields in target-layout data structures. thanks -- PMM
