On 01/08/2018 03:10 PM, Paolo Bonzini wrote: > On 08/01/2018 19:02, Philippe Mathieu-Daudé wrote: >> /* Data in mem is always in the byte order of the controller (le). >> * It must be dword aligned to allow direct access to 32 bit values. */ >> - uint8_t mem[PCI_MEM_SIZE] QEMU_ALIGNED(8); >> + uint8_t mem[PCI_MEM_SIZE] QEMU_ALIGNED(4); > > It's actually not needed at all, since accesses go through ld*_le_p and > st*_le_p (commit 4d9be25, "hw/net/eepro100.c: Don't use cpu_to_*w() and > *_to_cpup()", 2016-06-16).
Cool, I'll update this. Thank you, Phil.
signature.asc
Description: OpenPGP digital signature