From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:59126)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1c76dD-0004wV-Kq
	for qemu-devel@nongnu.org; Wed, 16 Nov 2016 15:16:32 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1c76dA-00087D-GW
	for qemu-devel@nongnu.org; Wed, 16 Nov 2016 15:16:31 -0500
Received: from mx1.redhat.com ([209.132.183.28]:51110)
	by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)
	(Exim 4.71) (envelope-from <mst@redhat.com>) id 1c76dA-000874-B4
	for qemu-devel@nongnu.org; Wed, 16 Nov 2016 15:16:28 -0500
Date: Wed, 16 Nov 2016 22:16:26 +0200
From: "Michael S. Tsirkin" <mst@redhat.com>
Message-ID: <20161116221548-mutt-send-email-mst@kernel.org>
References: <20161115134629.23161-1-pbonzini@redhat.com>
	<15408f40-5f13-e98d-2988-8c068295a888@de.ibm.com>
	<981b7cda-ea4a-f082-6529-5f84c0c3f120@linux.vnet.ibm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <981b7cda-ea4a-f082-6529-5f84c0c3f120@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH for-2.8 0/3] virtio fixes
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Farhan Ali <alifm@linux.vnet.ibm.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>, qemu-devel@nongnu.org, Christian Borntraeger <borntraeger@de.ibm.com>, alex.williamson@redhat.com

On Wed, Nov 16, 2016 at 03:03:13PM -0500, Farhan Ali wrote:
> Hi Paolo,
> 
> I was able to test your patches in our s390 environment. I don't see the
> qemu crashes anymore which I noticed before.
> 
> Testing a guest running high stress I/O workload, without iothreads does
> show a delay in guest response time.

Compared to which version?

> But running
> 
> the same test with iothreads seems to solve the issue.
> 
> Tested-by : Farhan Ali <alifm@linux.vnet.ibm.com>

Could you also test just patches 1 and 2 pls?

> 
> Thank you
> 
> Farhan
> 
> 
> On 11/16/2016 02:50 PM, Christian Borntraeger wrote:
> > On 11/15/2016 02:46 PM, Paolo Bonzini wrote:
> > > Patch 1 fixes vhost, patches 2-3 fix Windows hibernation.
> > > 
> > > Paolo
> > > 
> > > Paolo Bonzini (3):
> > >    virtio: introduce grab/release_ioeventfd to fix vhost
> > >    virtio: access ISR atomically
> > >    virtio: set ISR on dataplane notifications
> > > 
> > >   hw/block/dataplane/virtio-blk.c |  4 +--
> > >   hw/scsi/virtio-scsi-dataplane.c |  7 ------
> > >   hw/scsi/virtio-scsi.c           |  2 +-
> > >   hw/virtio/trace-events          |  2 +-
> > >   hw/virtio/vhost.c               | 11 +++------
> > >   hw/virtio/virtio-bus.c          | 54 ++++++++++++++++++++++++++++++++---------
> > >   hw/virtio/virtio-mmio.c         |  6 ++---
> > >   hw/virtio/virtio-pci.c          |  9 +++----
> > >   hw/virtio/virtio.c              | 46 ++++++++++++++++++++++++++++-------
> > >   include/hw/virtio/virtio-bus.h  | 14 +++++++++++
> > >   include/hw/virtio/virtio-scsi.h |  1 -
> > >   include/hw/virtio/virtio.h      |  4 ++-
> > >   12 files changed, 110 insertions(+), 50 deletions(-)
> > > 
> > Farhan,
> > 
> > it was this mail thread.