From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 2838D5591 for ; Thu, 18 Aug 2016 04:36:03 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP; 17 Aug 2016 19:36:00 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,537,1464678000"; d="scan'208";a="750217649" Received: from shwdeisgchi083.ccr.corp.intel.com (HELO [10.239.67.193]) ([10.239.67.193]) by FMSMGA003.fm.intel.com with ESMTP; 17 Aug 2016 19:35:58 -0700 To: Maxime Coquelin , Yuanhan Liu , Pankaj Chauhan References: <20160816025614.GM30752@yliu-dev.sh.intel.com> Cc: dev@dpdk.org, hemant.agrawal@nxp.com, shreyansh.jain@nxp.com From: "Tan, Jianfeng" Message-ID: Date: Thu, 18 Aug 2016 10:35:57 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] vhost [query] : support for multiple ports and non VMDQ devices in vhost switch X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Aug 2016 02:36:03 -0000 Hi Maxime, On 8/17/2016 7:18 PM, Maxime Coquelin wrote: > Hi Jianfeng, > > On 08/17/2016 04:33 AM, Tan, Jianfeng wrote: >> Hi, >> >> Please review below proposal of Pankaj and myself after an offline >> discussion. (Pankaj, please correct me if I'm going somewhere wrong). >> >> a. Remove HW dependent option, --strip-vlan, because different kinds of >> NICs behave differently. It's a bug fix. >> b. Abstract switching logic into a framework, so that we can develop >> different kinds of switching logics. In this phase, we will have two >> switching logics: (1) a simple software-based mac learning switching; >> (2) VMDQ based switching. Any other advanced switching logics can be >> proposed based on this framework. >> c. Merge tep_termination example vxlan as a switching logic of the >> framework. > > I was also thinking of making physical port optional and add MAC > learning, > so this is all good for me. To make it clear, we are not proposing to eliminate physical port, instead, we just eliminate the binding of VMDQ and virtio ports, superseding it with a MAC learning switching. > > Let me know if I can help in implementation, I'll be happy to > contribute. Thank you for participating. Currently, I'm working on item a (will be a quick and simple fix). Pankaj is working on item b (which would be a huge change). Item c is depending on item b. So let's wait RFC patch from Pankaj and see what we can help. > >> To be decided: >> d. Support multiple physical ports. >> e. Keep the current way to use vhost lib directly or use vhost pmd >> instead. > Do you see advantages of using vhost lib directly vs. pmd? > Wouldn't using vhost pmd make achieving zero-copy harder? > (I'm not sure, I didn't investigate the topic much for now). Yes, by using vhost lib, we can add back the removed feature zero-copy. But my understanding is zero-copy (nic-to-vm or vm-to-nic) or delayed copy (vm-to-vm) would be great and common features, which should be integrated into vhost lib and enabled in vhost pmd, so that all applications can benefit from it. And in fact, Yuanhan is working on the delayed copy now. An exception is rx-side-zero-copy, I don't know if it's common enough to be integrated in a vhost lib, because it'll require hardware queue binding. Besides, vhost pmd would be easier to use than vhost lib (personal opinion). Secondly, vhost pmd would be more clear in logic, 1:1:1 mapping among vhost port, unix socket path, and virtio port. Thirdly, by using vhost pmd, we can treat vhost ports the way of physical ports, otherwise, we use different API to receive/transmit packets. > > Also, if we use pmd directly, then it would no more be a vhost switch > only, as it could potentially be used with physical NICs also. You mean we are building a switch instead of vhost switch? Yes, a switch can switch packets between virtio-virtio and virtio-physical nic. Thanks, Jianfeng > > Any thoughts? > > Thanks, > Maxime