From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tetsuya Mukawa Subject: Re: [PATCH v11 2/2] vhost: Add VHOST PMD Date: Tue, 15 Mar 2016 14:35:51 +0900 Message-ID: <56E79F37.3050204@igel.co.jp> References: <1457065062-4423-3-git-send-email-mukawa@igel.co.jp> <1457316434-19128-3-git-send-email-mukawa@igel.co.jp> <20160314120203.GA25348@bricha3-MOBL3> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, ann.zhuangyanying@huawei.com To: Bruce Richardson Return-path: Received: from mail-pf0-f177.google.com (mail-pf0-f177.google.com [209.85.192.177]) by dpdk.org (Postfix) with ESMTP id 3D5102C69 for ; Tue, 15 Mar 2016 06:35:54 +0100 (CET) Received: by mail-pf0-f177.google.com with SMTP id 124so13174918pfg.0 for ; Mon, 14 Mar 2016 22:35:54 -0700 (PDT) In-Reply-To: <20160314120203.GA25348@bricha3-MOBL3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2016/03/14 21:02, Bruce Richardson wrote: > On Mon, Mar 07, 2016 at 11:07:14AM +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 >> --- >> MAINTAINERS | 5 + >> config/common_base | 6 + >> config/common_linuxapp | 1 + >> doc/guides/nics/index.rst | 1 + > This adds a new entry for vhost PMD into the index, but there is no vhost.rst > file present in this patchset. Did you forget to add it? Yes, it seems so. The file is only on my environment. I will add it. Thanks, Tetsuya > >> doc/guides/rel_notes/release_16_04.rst | 4 + >> drivers/net/Makefile | 4 + >> drivers/net/vhost/Makefile | 62 ++ >> drivers/net/vhost/rte_eth_vhost.c | 916 ++++++++++++++++++++++++++++ >> drivers/net/vhost/rte_eth_vhost.h | 109 ++++ >> drivers/net/vhost/rte_pmd_vhost_version.map | 10 + >> mk/rte.app.mk | 6 + >> 11 files changed, 1124 insertions(+) >> create mode 100644 drivers/net/vhost/Makefile >> create mode 100644 drivers/net/vhost/rte_eth_vhost.c >> create mode 100644 drivers/net/vhost/rte_eth_vhost.h >> create mode 100644 drivers/net/vhost/rte_pmd_vhost_version.map > > > /Bruce >