Re: [Qemu-devel] [PULL 00/47] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups

[Alex Williamson]

On Thu, 10 Nov 2016 18:12:20 +0200
"Michael S. Tsirkin" <address@hidden> wrote:

> The following changes since commit 6bbcb76301a72dc80c8d29af13d40bb9a759c9c6:
> 
>   MAINTAINERS: Remove obsolete stable branches (2016-11-10 15:29:59 +0000)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
> 
> for you to fetch changes up to 8038753b86f4cb1e79d4225a799395c4dae96b17:
> 
>   docs: add PCIe devices placement guidelines (2016-11-10 18:08:06 +0200)
> 
> ----------------------------------------------------------------
> virtio, vhost, pc, pci: tests, documentation, fixes and cleanups
> 
> Lots of fixes all over the place.  I allowed some cleanups in even though they
> are not strictly bugfixes, they might prevent bugs and seem very safe.
> 
> Most importantly, this fixes a regression with vhost introduced
> by the last pull.

I think I'm hitting this previous regression, I see this in my libvirt
log:

kvm_mem_ioeventfd_add: error adding ioeventfd: File exists

And a backtrace as seen here
https://paste.fedoraproject.org/477562/88144131/

So I merge this tag to try to resolve it, now I get qemu segfaulting
bisected to:

commit a6d8372bc6764ee279b473d13ff4ecc8acb7a978
Author: Cao jin <address@hidden>
Date:   Sat Nov 5 10:07:21 2016 +0800

    hcd-xhci: check & correct param before using it

And if I revert that, vhost still fails with:

qemu-system-x86_64: /net/gimli/home/alwillia/Work/qemu.git/memory.c:2012: 
memory_region_del_eventfd: Assertion `i != mr->ioeventfd_nb' failed.

So I think this does not fix the previous regression and adds a new
one :-\

Thanks,
Alex