From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from tama50.ecl.ntt.co.jp (tama50.ecl.ntt.co.jp [129.60.39.147]) by dpdk.org (Postfix) with ESMTP id 3F1791C150 for ; Fri, 11 May 2018 05:45:32 +0200 (CEST) Received: from vc2.ecl.ntt.co.jp (vc2.ecl.ntt.co.jp [129.60.86.154]) by tama50.ecl.ntt.co.jp (8.13.8/8.13.8) with ESMTP id w4B3jVgk009100; Fri, 11 May 2018 12:45:31 +0900 Received: from vc2.ecl.ntt.co.jp (localhost [127.0.0.1]) by vc2.ecl.ntt.co.jp (Postfix) with ESMTP id 687D1638A78; Fri, 11 May 2018 12:45:31 +0900 (JST) Received: from jcms-pop21.ecl.ntt.co.jp (jcms-pop21.ecl.ntt.co.jp [129.60.87.134]) by vc2.ecl.ntt.co.jp (Postfix) with ESMTP id 5CF79638A40; Fri, 11 May 2018 12:45:31 +0900 (JST) Received: from [IPv6:::1] (watercress.nslab.ecl.ntt.co.jp [129.60.13.73]) by jcms-pop21.ecl.ntt.co.jp (Postfix) with ESMTPSA id 50E7C4006AF; Fri, 11 May 2018 12:45:31 +0900 (JST) References: <00b1168e-1a4e-1464-da3f-ce60a879290e@lab.ntt.co.jp> <201805090658.w496wDQX011485@imss04.silk.ntt-tx.co.jp> From: Yasufumi Ogawa Message-ID: <4f1064f5-ff2c-aa33-5497-c7b736e66818@lab.ntt.co.jp> Date: Fri, 11 May 2018 12:44:06 +0900 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <201805090658.w496wDQX011485@imss04.silk.ntt-tx.co.jp> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-CC-Mail-RelayStamp: 1 To: x-fn-spp@sl.ntt-tx.co.jp Cc: spp@dpdk.org, ferruh.yigit@intel.com, ogawa.yasufumi@lab.ntt.co.jp X-TM-AS-MML: disable Subject: Re: [spp] [PATCH v2 1/2] docs: fix document error under spp_vf directory X-BeenThere: spp@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Soft Patch Panel List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 May 2018 03:45:34 -0000 On 2018/05/09 15:58, x-fn-spp@sl.ntt-tx.co.jp wrote: > From: Hideyuki-Yamashita > > - Fix typo in build.rst > - Fix typo and error in usecase1.rst Hi Hideyuki, You fixed wrong command name 'unmount' in description, but still remains the codeblock. +Finally, you umount default hugepage. $ sudo unmount /dev/hugepages Could you give us a bit more detailed summary for 2nd topic, correcting usecase, to explain what was wrong. For example, "Fix incorrect configuration. Coremask should be '0x3ffd', not '0x00fe'. And there are many mistakes for assigning resource IDs while setting up spp_vf." Thanks, Yasufumi > > Signed-off-by: Hideyuki Yamashita > Signed-off-by: Naoki Takada > --- > docs/guides/spp_vf/gsg/build.rst | 4 +- > docs/guides/spp_vf/use_cases/usecase1.rst | 80 +++++++++++++++---------------- > 2 files changed, 41 insertions(+), 43 deletions(-) > > diff --git a/docs/guides/spp_vf/gsg/build.rst b/docs/guides/spp_vf/gsg/build.rst > index a37f86a..52611dc 100644 > --- a/docs/guides/spp_vf/gsg/build.rst > +++ b/docs/guides/spp_vf/gsg/build.rst > @@ -79,7 +79,7 @@ You need to run ``update-grub`` and reboot to activate grub config. > > .. code-block:: console > > - $ sudo upadte-grub > + $ sudo update-grub > $ sudo reboot > > > @@ -101,7 +101,7 @@ You can check hugepage settings as following. > hugetlbfs-kvm on /run/hugepages/kvm type hugetlbfs (rw,relatime,mode=775,gid=117) > hugetlb on /run/lxcfs/controllers/hugetlb type cgroup (rw,relatime,hugetlb,release_agent=/run/cgmanager/agents/cgm-release-agent.hugetlb,nsroot=/) > > -Finally, you unmount default hugepage. > +Finally, you umount default hugepage. > > .. code-block:: console > > diff --git a/docs/guides/spp_vf/use_cases/usecase1.rst b/docs/guides/spp_vf/use_cases/usecase1.rst > index 080aba1..b5256e5 100644 > --- a/docs/guides/spp_vf/use_cases/usecase1.rst > +++ b/docs/guides/spp_vf/use_cases/usecase1.rst > @@ -82,7 +82,7 @@ After ``spp_primary`` is launched, run secondary process ``spp_vf``. > .. code-block:: console > > $ sudo ./src/vf/x86_64-native-linuxapp-gcc/spp_vf \ > - -c 0x00fe -n 4 --proc-type=secondary \ > + -c 0x3ffd -n 4 --proc-type=secondary \ > -- \ > --client-id 1 \ > -s 127.0.0.1:6666 \ > @@ -156,14 +156,14 @@ merger has two rx ports. > spp > sec 1;port add ring:0 rx forwarder1 > spp > sec 1;port add vhost:0 tx forwarder1 > # forwarder2 > - spp > sec 1;port add vhost:0 rx forwarder2 > - spp > sec 1;port add ring:2 tx forwarder2 > + spp > sec 1;port add ring:1 rx forwarder2 > + spp > sec 1;port add vhost:2 tx forwarder2 > # forwarder3 > - spp > sec 1;port add ring:1 rx forwarder3 > - spp > sec 1;port add vhost:2 rx forwarder3 > + spp > sec 1;port add vhost:0 rx forwarder3 > + spp > sec 1;port add ring:2 tx forwarder3 > # forwarder4 > - spp > sec 1;port add vhost:2 tx forwarder4 > - spp > sec 1;port add ring:3 rx forwarder4 > + spp > sec 1;port add vhost:2 rx forwarder4 > + spp > sec 1;port add ring:3 tx forwarder4 > # merger1 > spp > sec 1;port add ring:2 rx merger1 > spp > sec 1;port add ring:3 rx merger1 > @@ -213,14 +213,14 @@ Add ports to each of components. > spp > sec 1;port add ring:4 rx forwarder5 > spp > sec 1;port add vhost:1 tx forwarder5 > # forwarder6 > - spp > sec 1;port add vhost:1 rx forwarder6 > - spp > sec 1;port add ring:6 tx forwarder6 > + spp > sec 1;port add ring:5 rx forwarder6 > + spp > sec 1;port add vhost:3 tx forwarder6 > # forwarder7 > - spp > sec 1;port add ring:5 rx forwarder7 > - spp > sec 1;port add vhost:3 rx forwarder7 > + spp > sec 1;port add vhost:1 rx forwarder7 > + spp > sec 1;port add ring:6 tx forwarder7 > # forwarder8 > - spp > sec 1;port add vhost:3 tx forwarder8 > - spp > sec 1;port add ring:7 rx forwarder8 > + spp > sec 1;port add vhost:3 rx forwarder8 > + spp > sec 1;port add ring:7 tx forwarder8 > # merger2 > spp > sec 1;port add ring:6 rx merger2 > spp > sec 1;port add ring:7 rx merger2 > @@ -287,8 +287,6 @@ Configurations also for ``spp-vm2`` as ``spp-vm1``. > > .. code-block:: console > > - $ ssh -oStrictHostKeyChecking=no sppuser@192.168.122.32 > - > # up interfaces > $ sudo ifconfig ens4 inet 192.168.140.31 netmask 255.255.255.0 up > $ sudo ifconfig ens5 inet 192.168.150.32 netmask 255.255.255.0 up > @@ -340,21 +338,21 @@ for the first SSH login path. > spp > sec 1;port del ring:0 tx classifier1 > spp > sec 1;port del ring:1 tx classifier1 > # forwarder1 > - spp > sec 1;port del ring:0 rx forward1 > - spp > sec 1;port del vhost:0 tx forward1 > + spp > sec 1;port del ring:0 rx forwarder1 > + spp > sec 1;port del vhost:0 tx forwarder1 > # forwarder2 > - spp > sec 1;port del vhost:0 rx forward2 > - spp > sec 1;port del ring:2 tx forward2 > + spp > sec 1;port del ring:1 rx forwarder2 > + spp > sec 1;port del vhost:2 tx forwarder2 > # forwarder3 > - spp > sec 1;port del ring:1 rx forward3 > - spp > sec 1;port del vhost:2 rx forward3 > + spp > sec 1;port del vhost:0 rx forwarder3 > + spp > sec 1;port del ring:2 tx forwarder3 > # forwarder4 > - spp > sec 1;port del vhost:2 tx forward4 > - spp > sec 1;port del ring:3 rx forward4 > + spp > sec 1;port del vhost:2 rx forwarder4 > + spp > sec 1;port del ring:3 tx forwarder4 > # merger1 > - spp > sec 1;port del ring:2 rx merge1 > - spp > sec 1;port del ring:3 rx merge1 > - spp > sec 1;port del phy:0 tx merge1 > + spp > sec 1;port del ring:2 rx merger1 > + spp > sec 1;port del ring:3 rx merger1 > + spp > sec 1;port del phy:0 tx merger1 > > Then, stop components. > > @@ -362,11 +360,11 @@ Then, stop components. > > # Stop component to spp_vf > spp > sec 1;component stop classifier1 > - spp > sec 1;component stop forward1 > - spp > sec 1;component stop forward2 > - spp > sec 1;component stop forward3 > - spp > sec 1;component stop forward4 > - spp > sec 1;component stop merge1 > + spp > sec 1;component stop forwarder1 > + spp > sec 1;component stop forwarder2 > + spp > sec 1;component stop forwarder3 > + spp > sec 1;component stop forwarder4 > + spp > sec 1;component stop merger1 > > Second, do termination for the second path. > Delete entries from ``classifier_table`` and ports from each of > @@ -388,11 +386,11 @@ components. > spp > sec 1;port del ring:4 rx forwarder5 > spp > sec 1;port del vhost:1 tx forwarder5 > # forwarder6 > - spp > sec 1;port del vhost:1 rx forwarder6 > - spp > sec 1;port del ring:6 tx forwarder6 > + spp > sec 1;port del ring:5 rx forwarder6 > + spp > sec 1;port del vhost:3 tx forwarder6 > # forwarder7 > - spp > sec 1;port del ring:5 rx forwarder7 > - spp > sec 1;port del vhost:3 rx forwarder7 > + spp > sec 1;port del vhost:1 rx forwarder7 > + spp > sec 1;port del ring:6 tx forwarder7 > # forwarder8 > spp > sec 1;port del vhost:3 tx forwarder8 > spp > sec 1;port del ring:7 rx forwarder8 > @@ -406,12 +404,12 @@ Then, stop components. > .. code-block:: console > > # Stop component to spp_vf > - spp > sec 1;component stop classifier2 8 classifier_mac > - spp > sec 1;component stop forward5 9 forward > - spp > sec 1;component stop forward6 10 forward > - spp > sec 1;component stop forward7 11 forward > - spp > sec 1;component stop forward8 12 forward > - spp > sec 1;component stop merge2 13 merge > + spp > sec 1;component stop classifier2 > + spp > sec 1;component stop forwarder5 > + spp > sec 1;component stop forwarder6 > + spp > sec 1;component stop forwarder7 > + spp > sec 1;component stop forwarder8 > + spp > sec 1;component stop merger2 > > Finally, run ``flush`` subcommand. > > -- Yasufumi Ogawa NTT Network Service Systems Labs