Message ID | 1428510667-6438-1-git-send-email-iryzhov@nfware.com (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id BB6FB5A5D; Wed, 8 Apr 2015 18:31:21 +0200 (CEST) Received: from mail-la0-f47.google.com (mail-la0-f47.google.com [209.85.215.47]) by dpdk.org (Postfix) with ESMTP id 017E85A59 for <dev@dpdk.org>; Wed, 8 Apr 2015 18:31:20 +0200 (CEST) Received: by labbd9 with SMTP id bd9so58743024lab.2 for <dev@dpdk.org>; Wed, 08 Apr 2015 09:31:19 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-type:content-transfer-encoding; bh=0KtdCkWNWDNti+S7ZQkRzqJPsmMR7Cxi47ZG7eOI+O4=; b=iWhrV+jEVuu6vtRhPkYB2YeQ7hfqovU5qND3OfRF3aDk8m+11EbBX3ZH1BC3MLVpZa 5u7+KziXweKIvp6G6Jd72p3YZ2vPKW73nn3RxkE6fVnHked5DSa1dMUraLtW0CbSxr3m NuEryKBERY8YYgqT2kQOZ9DOZxtT0Jwj2wKYBGgaOngJzEIXoQNPIWD/YThCFIksgI/5 eEE/vcv+j+24aG5Pd5yAE3kAFJRXwgFo5ONBGqF9rbIJex3GmcS7t7ybF5nwgRQtIb4z Av/Z29HPVdRPBawImnmWnlRjdhfR0LIB8KVqlXR1sue0lYzlbw6URIiphji39X6Vjy8O HXsw== X-Gm-Message-State: ALoCoQkNMvPYunjjVp/n2PcpMfYK1r3MI/9GS3a6+/6caSL5ZBtcoPv3tY7QQr0qnb40AJHj7PpT X-Received: by 10.112.168.102 with SMTP id zv6mr1144497lbb.45.1428510679681; Wed, 08 Apr 2015 09:31:19 -0700 (PDT) Received: from localhost.localdomain (vpn.arccn.ru. [95.182.74.2]) by mx.google.com with ESMTPSA id si3sm2599650lbb.32.2015.04.08.09.31.18 (version=TLSv1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 08 Apr 2015 09:31:18 -0700 (PDT) From: Igor Ryzhov <iryzhov@nfware.com> To: dev@dpdk.org Date: Wed, 8 Apr 2015 19:31:07 +0300 Message-Id: <1428510667-6438-1-git-send-email-iryzhov@nfware.com> X-Mailer: git-send-email 1.9.5 (Apple Git-50.3) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Igor Ryzhov <iryzhov@nfware.com> Subject: [dpdk-dev] [PATCH] doc: fix vhost guide 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> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Igor Ryzhov
April 8, 2015, 4:31 p.m. UTC
Guide says that a configure parameter to choose between vhost cuse and vhost user will be introduced in the future, but it’s already added by commit 28a1ccca41bf.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
---
doc/guides/sample_app_ug/vhost.rst | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
Comments
> -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Igor Ryzhov > Sent: Wednesday, April 8, 2015 5:31 PM > To: dev@dpdk.org > Cc: Igor Ryzhov > Subject: [dpdk-dev] [PATCH] doc: fix vhost guide > > Guide says that a configure parameter to choose between vhost cuse and > vhost user will be introduced in the future, but it’s already added by commit > 28a1ccca41bf. Good point Igor- thanks for the spot. Siobhan > > Signed-off-by: Igor Ryzhov <iryzhov@nfware.com> > --- > doc/guides/sample_app_ug/vhost.rst | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/doc/guides/sample_app_ug/vhost.rst > b/doc/guides/sample_app_ug/vhost.rst > index 8a7eb3b..df8cd8c 100644 > --- a/doc/guides/sample_app_ug/vhost.rst > +++ b/doc/guides/sample_app_ug/vhost.rst > @@ -309,13 +309,12 @@ Compiling the Sample Code > > CONFIG_RTE_LIBRTE_VHOST=n > > - vhost user is turned on by default in the lib/librte_vhost/Makefile. > - To enable vhost cuse, uncomment vhost cuse and comment vhost user > manually. In future, a configure will be created for switch between two > implementations. > + vhost user is turned on by default in the configure file > config/common_linuxapp. > + To enable vhost cuse, disable vhost user. > > .. code-block:: console > > - SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_cuse/vhost-net-cdev.c > vhost_cuse/virtio-net-cdev.c vhost_cuse/eventfd_copy.c > - #SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_user/vhost-net-user.c > vhost_user/virtio-net-user.c vhost_user/fd_man.c > + CONFIG_RTE_LIBRTE_VHOST_USER=y > > After vhost is enabled and the implementation is selected, build the vhost > library. > > -- > 1.9.5 (Apple Git-50.3)
Hi Igor, Good catch, comments as below. > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Igor Ryzhov > Sent: Thursday, April 9, 2015 12:31 AM > To: dev@dpdk.org > Cc: Igor Ryzhov > Subject: [dpdk-dev] [PATCH] doc: fix vhost guide > > Guide says that a configure parameter to choose between vhost cuse and > vhost user will be introduced in the future, but it’s already added by commit > 28a1ccca41bf. > > Signed-off-by: Igor Ryzhov <iryzhov@nfware.com> > --- > doc/guides/sample_app_ug/vhost.rst | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/doc/guides/sample_app_ug/vhost.rst > b/doc/guides/sample_app_ug/vhost.rst > index 8a7eb3b..df8cd8c 100644 > --- a/doc/guides/sample_app_ug/vhost.rst > +++ b/doc/guides/sample_app_ug/vhost.rst > @@ -309,13 +309,12 @@ Compiling the Sample Code > > CONFIG_RTE_LIBRTE_VHOST=n > > - vhost user is turned on by default in the lib/librte_vhost/Makefile. > - To enable vhost cuse, uncomment vhost cuse and comment vhost user > manually. In future, a configure will be created for switch between two > implementations. > + vhost user is turned on by default in the configure file > config/common_linuxapp. > + To enable vhost cuse, disable vhost user. > > .. code-block:: console > > - SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_cuse/vhost-net-cdev.c > vhost_cuse/virtio-net-cdev.c vhost_cuse/eventfd_copy.c > - #SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_user/vhost-net-user.c > vhost_user/virtio-net-user.c vhost_user/fd_man.c > + CONFIG_RTE_LIBRTE_VHOST_USER=y If it wants to guide user how to enable vhost cuse, then I think It makes sense to change it into: CONFIG_RTE_LIBRTE_VHOST_USER=n Thanks Changchun
Hello, Changchun. Previous paragraph says «To enable vhost, turn on vhost library in the configure file config/common_linuxapp», but string in a code-block is «CONFIG_RTE_LIBRTE_VHOST=n». I thought that idea is to use the default string from the config file that user have to change, not already changed string. So I used the same style. Regards, Igor > 13 апр. 2015 г., в 7:52, Ouyang, Changchun <changchun.ouyang@intel.com> написал(а): > > Hi Igor, > > Good catch, comments as below. > >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Igor Ryzhov >> Sent: Thursday, April 9, 2015 12:31 AM >> To: dev@dpdk.org >> Cc: Igor Ryzhov >> Subject: [dpdk-dev] [PATCH] doc: fix vhost guide >> >> Guide says that a configure parameter to choose between vhost cuse and >> vhost user will be introduced in the future, but it’s already added by commit >> 28a1ccca41bf. >> >> Signed-off-by: Igor Ryzhov <iryzhov@nfware.com> >> --- >> doc/guides/sample_app_ug/vhost.rst | 7 +++---- >> 1 file changed, 3 insertions(+), 4 deletions(-) >> >> diff --git a/doc/guides/sample_app_ug/vhost.rst >> b/doc/guides/sample_app_ug/vhost.rst >> index 8a7eb3b..df8cd8c 100644 >> --- a/doc/guides/sample_app_ug/vhost.rst >> +++ b/doc/guides/sample_app_ug/vhost.rst >> @@ -309,13 +309,12 @@ Compiling the Sample Code >> >> CONFIG_RTE_LIBRTE_VHOST=n >> >> - vhost user is turned on by default in the lib/librte_vhost/Makefile. >> - To enable vhost cuse, uncomment vhost cuse and comment vhost user >> manually. In future, a configure will be created for switch between two >> implementations. >> + vhost user is turned on by default in the configure file >> config/common_linuxapp. >> + To enable vhost cuse, disable vhost user. >> >> .. code-block:: console >> >> - SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_cuse/vhost-net-cdev.c >> vhost_cuse/virtio-net-cdev.c vhost_cuse/eventfd_copy.c >> - #SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_user/vhost-net-user.c >> vhost_user/virtio-net-user.c vhost_user/fd_man.c >> + CONFIG_RTE_LIBRTE_VHOST_USER=y > > If it wants to guide user how to enable vhost cuse, then I think > It makes sense to change it into: CONFIG_RTE_LIBRTE_VHOST_USER=n > > Thanks > Changchun
Sorry, I used wrong email address to reply from. This one is correct. On Mon, Apr 13, 2015 at 10:11 AM, Igor Ryzhov <iryzhov@arccn.ru> wrote: > Hello, Changchun. > > Previous paragraph says «To enable vhost, turn on vhost library in the > configure file config/common_linuxapp», but string in a code-block is > «CONFIG_RTE_LIBRTE_VHOST=n». I thought that idea is to use the default > string from the config file that user have to change, not already changed > string. So I used the same style. > > Regards, > Igor > > 13 апр. 2015 г., в 7:52, Ouyang, Changchun <changchun.ouyang@intel.com> > написал(а): > > Hi Igor, > > Good catch, comments as below. > > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org <dev-bounces@dpdk.org>] On Behalf > Of Igor Ryzhov > Sent: Thursday, April 9, 2015 12:31 AM > To: dev@dpdk.org > Cc: Igor Ryzhov > Subject: [dpdk-dev] [PATCH] doc: fix vhost guide > > Guide says that a configure parameter to choose between vhost cuse and > vhost user will be introduced in the future, but it’s already added by > commit > 28a1ccca41bf. > > Signed-off-by: Igor Ryzhov <iryzhov@nfware.com> > --- > doc/guides/sample_app_ug/vhost.rst | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/doc/guides/sample_app_ug/vhost.rst > b/doc/guides/sample_app_ug/vhost.rst > index 8a7eb3b..df8cd8c 100644 > --- a/doc/guides/sample_app_ug/vhost.rst > +++ b/doc/guides/sample_app_ug/vhost.rst > @@ -309,13 +309,12 @@ Compiling the Sample Code > > CONFIG_RTE_LIBRTE_VHOST=n > > - vhost user is turned on by default in the lib/librte_vhost/Makefile. > - To enable vhost cuse, uncomment vhost cuse and comment vhost user > manually. In future, a configure will be created for switch between two > implementations. > + vhost user is turned on by default in the configure file > config/common_linuxapp. > + To enable vhost cuse, disable vhost user. > > .. code-block:: console > > - SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_cuse/vhost-net-cdev.c > vhost_cuse/virtio-net-cdev.c vhost_cuse/eventfd_copy.c > - #SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_user/vhost-net-user.c > vhost_user/virtio-net-user.c vhost_user/fd_man.c > + CONFIG_RTE_LIBRTE_VHOST_USER=y > > > If it wants to guide user how to enable vhost cuse, then I think > It makes sense to change it into: CONFIG_RTE_LIBRTE_VHOST_USER=n > > Thanks > Changchun > > >
From: Igor Ryzhov [mailto:iryzhov@nfware.com] Sent: Monday, April 13, 2015 3:14 PM To: Ouyang, Changchun Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] doc: fix vhost guide Sorry, I used wrong email address to reply from. This one is correct. On Mon, Apr 13, 2015 at 10:11 AM, Igor Ryzhov <iryzhov@arccn.ru<mailto:iryzhov@arccn.ru>> wrote: Hello, Changchun. Previous paragraph says «To enable vhost, turn on vhost library in the configure file config/common_linuxapp», but string in a code-block is «CONFIG_RTE_LIBRTE_VHOST=n». I thought that idea is to use the default string from the config file that user have to change, not already changed string. So I used the same style. Regards, Igor 13 апр. 2015 г., в 7:52, Ouyang, Changchun <changchun.ouyang@intel.com<mailto:changchun.ouyang@intel.com>> написал(а): Hi Igor, Good catch, comments as below. -----Original Message----- From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Igor Ryzhov Sent: Thursday, April 9, 2015 12:31 AM To: dev@dpdk.org<mailto:dev@dpdk.org> Cc: Igor Ryzhov Subject: [dpdk-dev] [PATCH] doc: fix vhost guide Guide says that a configure parameter to choose between vhost cuse and vhost user will be introduced in the future, but it’s already added by commit 28a1ccca41bf. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com<mailto:iryzhov@nfware.com>> Acked-by: Changchun Ouyang <changchun.ouyang@intel.com>
> Guide says that a configure parameter to choose between vhost cuse and > vhost user will be introduced in the future, but it’s already added by commit > 28a1ccca41bf. > > Signed-off-by: Igor Ryzhov <iryzhov@nfware.com<mailto:iryzhov@nfware.com>> > > Acked-by: Changchun Ouyang <changchun.ouyang@intel.com> Applied, thanks
diff --git a/doc/guides/sample_app_ug/vhost.rst b/doc/guides/sample_app_ug/vhost.rst index 8a7eb3b..df8cd8c 100644 --- a/doc/guides/sample_app_ug/vhost.rst +++ b/doc/guides/sample_app_ug/vhost.rst @@ -309,13 +309,12 @@ Compiling the Sample Code CONFIG_RTE_LIBRTE_VHOST=n - vhost user is turned on by default in the lib/librte_vhost/Makefile. - To enable vhost cuse, uncomment vhost cuse and comment vhost user manually. In future, a configure will be created for switch between two implementations. + vhost user is turned on by default in the configure file config/common_linuxapp. + To enable vhost cuse, disable vhost user. .. code-block:: console - SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_cuse/vhost-net-cdev.c vhost_cuse/virtio-net-cdev.c vhost_cuse/eventfd_copy.c - #SRCS-$(CONFIG_RTE_LIBRTE_VHOST) += vhost_user/vhost-net-user.c vhost_user/virtio-net-user.c vhost_user/fd_man.c + CONFIG_RTE_LIBRTE_VHOST_USER=y After vhost is enabled and the implementation is selected, build the vhost library.