From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37236) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c4xuR-00086e-Pu for qemu-devel@nongnu.org; Thu, 10 Nov 2016 17:33:28 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c4xuN-0003Az-Kq for qemu-devel@nongnu.org; Thu, 10 Nov 2016 17:33:27 -0500 Received: from mx1.redhat.com ([209.132.183.28]:35414) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c4xuN-00039z-Ep for qemu-devel@nongnu.org; Thu, 10 Nov 2016 17:33:23 -0500 Date: Fri, 11 Nov 2016 00:33:17 +0200 From: "Michael S. Tsirkin" Message-ID: <20161111003049-mutt-send-email-mst@kernel.org> References: <1478794177-29377-1-git-send-email-mst@redhat.com> <20161110152921.58253387@t450s.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161110152921.58253387@t450s.home> Subject: Re: [Qemu-devel] [PULL 00/47] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex Williamson Cc: qemu-devel@nongnu.org, Peter Maydell , Stefan Hajnoczi , Cao jin , Paolo Bonzini , Felipe Franciosi 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" 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 > 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