From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <yuanhan.liu@linux.intel.com>
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 by dpdk.org (Postfix) with ESMTP id A18755A51
 for <dev@dpdk.org>; Mon, 21 Sep 2015 04:21:03 +0200 (CEST)
Received: from orsmga001.jf.intel.com ([10.7.209.18])
 by fmsmga103.fm.intel.com with ESMTP; 20 Sep 2015 19:21:02 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.17,564,1437462000"; d="scan'208";a="773189594"
Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.66.60])
 by orsmga001.jf.intel.com with ESMTP; 20 Sep 2015 19:21:01 -0700
Date: Mon, 21 Sep 2015 10:22:52 +0800
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Message-ID: <20150921022252.GO2339@yliu-dev.sh.intel.com>
References: <1442589061-19225-1-git-send-email-yuanhan.liu@linux.intel.com>
 <1442589061-19225-6-git-send-email-yuanhan.liu@linux.intel.com>
 <20150920123021-mutt-send-email-mst@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20150920123021-mutt-send-email-mst@redhat.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v5 resend 05/12] vhost: add
 VHOST_USER_SET_VRING_ENABLE message
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 21 Sep 2015 02:21:04 -0000

On Sun, Sep 20, 2015 at 12:37:35PM +0300, Michael S. Tsirkin wrote:
> On Fri, Sep 18, 2015 at 11:10:54PM +0800, Yuanhan Liu wrote:
> > From: Changchun Ouyang <changchun.ouyang@intel.com>
> > 
> > This message is used to enable/disable a specific vring queue pair.
> > The first queue pair is enabled by default.
> > 
> > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
> > Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
> > ---
[snip...]
> >  void
> >  user_destroy_device(struct vhost_device_ctx ctx)
> >  {
> 
> It might be a good idea to flush any packets being processed
> on relevant cores at this point.

They are offloaded to the application (examples/vhost/vhost-switch in
this case).

user_destroy_device will invoke the application's "destroy_device()"
callback in the end, which, in our case, will set "remove" flag. The
core worker will then drain and free the RX queue and free TX queue
once the "remove" flag is set.

	--yliu