On Mon, Jan 08, 2018 at 09:14:41AM +0000, Peter Maydell wrote:
On 21 December 2017 at 23:43, Michael S. Tsirkin <address@hidden>
wrote:
you also need to drop "hw/pci: remove obsolete PCIDevice->init()"
(applied in your tree as 18951fce55d5aa58cd1629b4cb704ee51bab4420)
else the not-yet-QOM'ified NVME won't work anymore.
Done that too, thanks a lot
Hi. I'm afraid this pullreq (merge of 880b1ff) has new runtime
errors from the clang sanitizer:
/home/petmay01/linaro/qemu-for-merges/hw/net/net_tx_pkt.c:201:27:
runtime error: member access within misaligned address 0x55b4ffee8246
for type 'struct ip_header', which requires 4 byte alignment
0x55b4ffee8246: note: pointer points here
01 00 00 00 45 00 01 a9 01 00 00 00 40 11 78 45 00 00 00 00 ff ff
ff ff 00 00 00 00 00 00 00 00
^
/home/petmay01/linaro/qemu-for-merges/hw/net/net_tx_pkt.c:201:27:
runtime error: load of misaligned address 0x55b4ffee8246 for type
'uint8_t' (aka 'unsigned char'), which requires 4 byte alignment
0x55b4ffee8246: note: pointer points here
01 00 00 00 45 00 01 a9 01 00 00 00 40 11 78 45 00 00 00 00 ff ff
ff ff 00 00 00 00 00 00 00 00
^
/home/petmay01/linaro/qemu-for-merges/hw/net/net_tx_pkt.c:208:65:
runtime error: member access within misaligned address 0x55b4ffee8246
for type 'struct ip_header', which requires 4 byte alignment
0x55b4ffee8246: note: pointer points here
01 00 00 00 45 00 01 a9 01 00 00 00 40 11 78 45 00 00 00 00 ff ff
ff ff 00 00 00 00 00 00 00 00
^
/home/petmay01/linaro/qemu-for-merges/hw/net/net_tx_pkt.c:210:13:
runtime error: member access within misaligned address 0x55b4ffee8246
for type 'struct ip_header', which requires 4 byte alignment
0x55b4ffee8246: note: pointer points here
01 00 00 00 45 00 01 a9 01 00 00 00 40 11 78 45 00 00 00 00 ff ff
ff ff 00 00 00 00 00 00 00 00
^
/home/petmay01/linaro/qemu-for-merges/hw/net/net_tx_pkt.c:210:13:
runtime error: load of misaligned address 0x55b4ffee8246 for type
'uint8_t' (aka 'unsigned char'), which requires 4 byte alignment
0x55b4ffee8246: note: pointer points here
01 00 00 00 45 00 01 a9 01 00 00 00 40 11 78 45 00 00 00 00 ff ff
ff ff 00 00 00 00 00 00 00 00
^
Sorry for the late report, I didn't quite have time
to process the pull before Christmas holidays.
thanks
-- PMM
So I bisected and the reason is this commit:
commit 18b20bb43a2f37f0c8ae23a3e9b3d9a4a05b6bd4
Author: David Gibson <address@hidden>
Date: Tue Dec 19 15:45:22 2017 +1100
tests/pxe-test: Add some extra tests
Previously virtio-net was only tested for ppc64 in "slow"
mode. That
doesn't make much sense since virtio-net is used much more
often in
practice than the spapr-vlan device which was tested always.
So, move
virtio-net to always be tested on ppc64.
We had no tests at all for the q35 machine, which doesn't
seem wise
given its increasing prominence. Add a couple of tests for it,
including testing the newer e1000e adapter.
Signed-off-by: David Gibson <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
:040000 040000 5a982bfea24b9ac3c651b84425a39b3c85f4871e
771af3fdfb2778c6d6ed6b1098d1e79c181d6fb0 M tests
Pls either fix or drop e1000e test