From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCHv2] virtio-net: stop vhost backend on vmstop Date: Thu, 10 Jun 2010 17:19:55 +0300 Message-ID: <20100610141955.GA1814@redhat.com> References: <20100602180152.GA21861@redhat.com> <20100604170316.GC25107@amt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org To: Marcelo Tosatti Return-path: Received: from mx1.redhat.com ([209.132.183.28]:64619 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754040Ab0FJOYf (ORCPT ); Thu, 10 Jun 2010 10:24:35 -0400 Received: from int-mx08.intmail.prod.int.phx2.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o5AEOYCI007304 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 10 Jun 2010 10:24:34 -0400 Content-Disposition: inline In-Reply-To: <20100604170316.GC25107@amt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Jun 04, 2010 at 02:03:16PM -0300, Marcelo Tosatti wrote: > On Wed, Jun 02, 2010 at 09:01:52PM +0300, Michael S. Tsirkin wrote: > > vhost net currently keeps running after vmstop, > > which causes trouble as qemy does not check > > for dirty pages anymore. > > The fix is to simply keep vm and vhost running/stopped > > status in sync. > > > > Tested-by: David L Stevens > > Signed-off-by: Michael S. Tsirkin > > --- > > > > Changes from v1: > > Simplify code as suggested by Amit. > > > > hw/virtio-net.c | 11 +++++------ > > 1 files changed, 5 insertions(+), 6 deletions(-) > > Applied, thanks. Heh, wrong list, this is actually an upstream patch. I sent Anthony a merge request with it, hope this won't create problems. > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html