qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL 00/47] virtio, vhost, pc, pci: tests, documentati


From: Michael S. Tsirkin
Subject: Re: [Qemu-devel] [PULL 00/47] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups
Date: Fri, 11 Nov 2016 00:33:17 +0200

On Thu, Nov 10, 2016 at 03:29:21PM -0700, Alex Williamson wrote:
> 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


Thanks!
Stefan, pls defer merging while we investigate.

Cc a bunch of relevant people.

-- 
MST



reply via email to

[Prev in Thread] Current Thread [Next in Thread]