On Mon, 10 Dec 2018 12:04:36 +0000 Peter Maydell <peter.mayd...@linaro.org> wrote:
> Taking the address of a field in a packed struct is a bad idea, because > it might not be actually aligned enough for that pointer type (and > thus cause a crash on dereference on some host architectures). Newer > versions of clang warn about this. Avoid the bug by not using the > "modify in place" byte swapping functions. > > Patch produced with scripts/coccinelle/inplace-byteswaps.cocci > (with a couple of long lines manually wrapped). > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> Thanks, applied.