From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 0A01A5A49 for ; Fri, 18 Mar 2016 13:27:05 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga102.jf.intel.com with ESMTP; 18 Mar 2016 05:27:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,354,1455004800"; d="scan'208";a="671858188" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.44]) by FMSMGA003.fm.intel.com with SMTP; 18 Mar 2016 05:27:03 -0700 Received: by (sSMTP sendmail emulation); Fri, 18 Mar 2016 12:27:02 +0025 Date: Fri, 18 Mar 2016 12:27:02 +0000 From: Bruce Richardson To: Tetsuya Mukawa Cc: dev@dpdk.org, ann.zhuangyanying@huawei.com Message-ID: <20160318122702.GA12932@bricha3-MOBL3> References: <1457316434-19128-3-git-send-email-mukawa@igel.co.jp> <1458030701-11487-3-git-send-email-mukawa@igel.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1458030701-11487-3-git-send-email-mukawa@igel.co.jp> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD 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: Fri, 18 Mar 2016 12:27:06 -0000 On Tue, Mar 15, 2016 at 05:31:41PM +0900, Tetsuya Mukawa wrote: > The patch introduces a new PMD. This PMD is implemented as thin wrapper > of librte_vhost. It means librte_vhost is also needed to compile the PMD. > The vhost messages will be handled only when a port is started. So start > a port first, then invoke QEMU. > > The PMD has 2 parameters. > - iface: The parameter is used to specify a path to connect to a > virtio-net device. > - queues: The parameter is used to specify the number of the queues > virtio-net device has. > (Default: 1) > > Here is an example. > $ ./testpmd -c f -n 4 --vdev 'eth_vhost0,iface=/tmp/sock0,queues=1' -- -i > > To connect above testpmd, here is qemu command example. > > $ qemu-system-x86_64 \ > > -chardev socket,id=chr0,path=/tmp/sock0 \ > -netdev vhost-user,id=net0,chardev=chr0,vhostforce,queues=1 \ > -device virtio-net-pci,netdev=net0,mq=on > > Signed-off-by: Tetsuya Mukawa > Acked-by: Ferruh Yigit > Acked-by: Yuanhan Liu > Acked-by: Rich Lane > Tested-by: Rich Lane Hi Tetsuya, I hope to get this set merged for RC2 very soon. Can you provide an update for the nic overview.rst doc listing out the features of this new PMD. If you want, you can provide it as a separate patch, that I will merge into this one for you on apply to next-net. If you do decide to respin this patchset with the extra doc, please take into account the following patchwork issues also - otherwise I'll also fix them on apply: WARNING:STATIC_CONST_CHAR_ARRAY: static const char * array should probably be static const char * const #364: FILE: drivers/net/vhost/rte_eth_vhost.c:56: +static const char *valid_arguments[] = { WARNING:LINE_SPACING: Missing a blank line after declarations #399: FILE: drivers/net/vhost/rte_eth_vhost.c:91: + char *iface_name; + volatile uint16_t once; WARNING:TYPO_SPELLING: 'Unknow' may be misspelled - perhaps 'Unknown'? #684: FILE: drivers/net/vhost/rte_eth_vhost.c:376: + RTE_LOG(ERR, PMD, "Unknow numa node\n"); Regards, /Bruce