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 9919B1B97B for ; Fri, 11 May 2018 06:24:59 +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 w4B4Ovv3010334; Fri, 11 May 2018 13:24:57 +0900 Received: from vc2.ecl.ntt.co.jp (localhost [127.0.0.1]) by vc2.ecl.ntt.co.jp (Postfix) with ESMTP id 7FA39638B94; Fri, 11 May 2018 13:24:57 +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 7446C638253; Fri, 11 May 2018 13:24:57 +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 6E24A4006AF; Fri, 11 May 2018 13:24:57 +0900 (JST) References: <00b1168e-1a4e-1464-da3f-ce60a879290e@lab.ntt.co.jp> <201805090658.w496wJwt011531@imss04.silk.ntt-tx.co.jp> From: Yasufumi Ogawa Message-ID: <285a674f-bd36-4856-5d61-9cf705511be3@lab.ntt.co.jp> Date: Fri, 11 May 2018 13:23:35 +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.w496wJwt011531@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 2/2] docs: improve document 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 04:25:00 -0000 On 2018/05/09 15:58, x-fn-spp@sl.ntt-tx.co.jp wrote: > From: Hideyuki-Yamashita > > - Add detail about where to edit using virsh edit command in build.rst > - Change command in Trouble Shooting Guide in build.rst > - Additional note when registering MAC address to classfier in usecase1.rst I think it is ambiguous for "assumes that VM is already installed". Do you mean "installed" that the image file is already setup (in which SPP is installed) and ready to boot the VM, right? I am not sure about "${VMNAME}" and "[VM_NAME]". Are there the same name or different? Although I am not so well understanding for virsh, I wander the name of '--name ${VMNAME}' and 'virsh edit [VM_NAME]' should be the same, and can be different for '${VMNAME}.img'. If so, could you revise it for using correct variable names? Thanks > > Signed-off-by: Hideyuki Yamashita > Signed-off-by: Naoki Takada > --- > docs/guides/spp_vf/gsg/build.rst | 49 ++++++++++++++++++++++++++----- > docs/guides/spp_vf/use_cases/usecase1.rst | 9 ++++++ > 2 files changed, 50 insertions(+), 8 deletions(-) > > diff --git a/docs/guides/spp_vf/gsg/build.rst b/docs/guides/spp_vf/gsg/build.rst > index 52611dc..1a8e114 100644 > --- a/docs/guides/spp_vf/gsg/build.rst > +++ b/docs/guides/spp_vf/gsg/build.rst > @@ -30,7 +30,7 @@ > > .. _spp_vf_gsg_build: > > -Build > +Setup > ===== > > Environment > @@ -165,6 +165,32 @@ Then, bind it with PCI_Number. > > virsh setup > ----------- > +.. note:: > + > + This document assumes that VM is already installed. If not please follow the instruction below and install the VM. > + > +.. code-block:: console > + > + virt-install \ > + --name ${VMNAME} \ > + --ram 4096 \ > + --disk path=/var/lib/libvirt/images/${VMNAME}.img,size=30 \ > + --vcpus 4 \ > + --os-type linux \ > + --os-variant ubuntu16.04 \ > + --network network=default \ > + --graphics none \ > + --console pty,target_type=serial \ > + --location 'http://archive.ubuntu.com/ubuntu/dists/xenial/main/installer-amd64/' \ > + --extra-args 'console=ttyS0,115200n8 serial' > + > +You may need type the following commands through ssh to activate console. > + > +.. code-block:: console > + > + $sudo systemctl enable serial-getty@ttyS0.service > + $sudo systemctl start serial-getty@ttyS0.service > + > > Edit VM configuration with virsh. > > @@ -172,6 +198,13 @@ Edit VM configuration with virsh. > > $ virsh edit [VM_NAME] > > +You need to add ``xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'`` into the domain tag because of adding ```` tag. > +In addition, you need to add the tag enclosed by ```` and ````, ```` and ```` because SPP uses vhost-user as interface with VM. > +Note that number used in those tags should be the same value (e.g. chr0,sock0,vhost-net0) and these values should correspond to "add vhost N" (in this example 0). > +MAC address used in ```` can be specified when registering MAC address to classifier using Secondary command. > + > + The following is an example of modified xml file: > + > .. code-block:: xml > > > @@ -266,13 +299,13 @@ In this case, you should try it. > > .. code-block:: console > > - $ sudo ln -s /etc/apparmor.d/usr.lib.libvirt.virt-aa-helper /etc/apparmor.d/disable/usr.lib.libvirt.virt-aa-helper > - $ sudo ln -s /etc/apparmor.d/usr.sbin.libvirtd /etc/apparmor.d/disable/usr.sbin.libvirtd > - $ sudo apparmor_parser -R /etc/apparmor.d/usr.lib.libvirt.virt-aa-helper > - $ sudo apparmor_parser -R /etc/apparmor.d/usr.sbin.libvirtd > - $ sudo service apparmor reload > - $ sudo service apparmor restart > - $ sudo service libvirt-bin restart > + #Edit /etc/libvirt/qemu.conf and set security_driver to none: > + $sudo vi /etc/libvirt/qemu.conf > + ... > + security_driver = "none" > + ... > + #Restart libvirtd: > + $sudo systemctl restart libvirtd.service > > Or, you remove appamor. > > diff --git a/docs/guides/spp_vf/use_cases/usecase1.rst b/docs/guides/spp_vf/use_cases/usecase1.rst > index b5256e5..6a4f62f 100644 > --- a/docs/guides/spp_vf/use_cases/usecase1.rst > +++ b/docs/guides/spp_vf/use_cases/usecase1.rst > @@ -186,6 +186,10 @@ In this usecase, you need to register two MAC addresses for merger1. > spp > sec 1;classifier_table add mac 52:54:00:12:34:56 ring:0 > spp > sec 1;classifier_table add mac 52:54:00:12:34:58 ring:1 > > +.. note:: > + > + Please verify that MAC address of target VM is specified in > + [MACADDRESS] parameter. > > Configuration for the second login path is almost similar to the first > path. > @@ -234,6 +238,11 @@ Register entries to classifier_table for classifier2. > spp > sec 1;classifier_table add mac 52:54:00:12:34:57 ring:4 > spp > sec 1;classifier_table add mac 52:54:00:12:34:59 ring:5 > > +.. note:: > + > + Please verify that MAC address of target VM is specified in > + [MACADDRESS] parameter. > + > Finally, activate all of settings by doign `flush` subcommand. > > .. code-block:: console > -- Yasufumi Ogawa NTT Network Service Systems Labs