On 6 July 2015 at 20:17, John Snow <[email protected]> wrote:
> A little bit of accidental byteswapping that was
> unwarranted for NCQ on PPC64.
>
> Does not fix the issue reported by Paolo, but does
> fix the issue reported by Peter Maydell.

Checked on the ppc64 box I have access to and this does
fix the hang I was seeing.

Tested-by: Peter Maydell <[email protected]>

PS: you don't really need a cover letter for a
single patch.

I do still see some test failures in other tests,
for instance

  /x86_64/virtio/blk/pci/basic:                                        OK
  /x86_64/virtio/blk/pci/indirect:                                     **
ERROR:/home/pm215/qemu/tests/virtio-blk-test.c:354:pci_indirect:
assertion failed (features & QVIRTIO_F_RING_INDIRECT_DESC != 0):
(0x00000000 !=
 0x00000000)
FAIL
GTester: last random seed: R02S8c870ae988a53fb86ee3e101fac57706
(pid=31320)
  /x86_64/virtio/blk/pci/config:                                       OK
  /x86_64/virtio/blk/pci/msix:                                         OK
  /x86_64/virtio/blk/pci/idx:                                          **
ERROR:/home/pm215/qemu/tests/libqos/virtio.c:277:qvirtqueue_set_used_event:
assertion failed: (vq->event)
FAIL
GTester: last random seed: R02S69d9a4b783eb5fa265c334eb5619f007
(pid=31341)
  /x86_64/virtio/blk/pci/hotplug:                                      OK


-- PMM

Reply via email to