[dpdk-dev,v3] doc: fix parameter of virtio_user for container

Message ID 1488959153-10341-1-git-send-email-wang.yong19@zte.com.cn (mailing list archive)
State Accepted, archived
Delegated to: Yuanhan Liu
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Yong Wang March 8, 2017, 7:45 a.m. UTC
  Update the "Virtio_user for Container Networking" doc, add the
"--file-prefix" option to testpmd in host and container to avoid
hugepage config file conflict.

Fixes: 50665deebda0 ("doc: add guide to use virtio-user for container networking")

Signed-off-by: Yong Wang <wang.yong19@zte.com.cn>
---
v3:
* Modify the commit message.

v2:
* Modify the fixline from two lines to one line.
* Remove the unnecessary "_" in "--file-prefix" option.

Acked-by: John McNamara <john.mcnamara@intel.com>

Acked-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
---
 doc/guides/howto/virtio_user_for_container_networking.rst | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
  

Comments

Yuanhan Liu March 22, 2017, 9:01 a.m. UTC | #1
On Wed, Mar 08, 2017 at 02:45:53AM -0500, Yong Wang wrote:
> Update the "Virtio_user for Container Networking" doc, add the
> "--file-prefix" option to testpmd in host and container to avoid
> hugepage config file conflict.
> 
> Fixes: 50665deebda0 ("doc: add guide to use virtio-user for container networking")
> 
> Signed-off-by: Yong Wang <wang.yong19@zte.com.cn>

I'm just aware that this patch dispatched to me at patchwork, so
Applied to dpdk-next-virtio.

> ---
> v3:
> * Modify the commit message.
> 
> v2:
> * Modify the fixline from two lines to one line.
> * Remove the unnecessary "_" in "--file-prefix" option.
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>
> 
> Acked-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>

Note that you should move the Acked-by before '---': they are
part of the commit log, while the changes log are not.

	--yliu
> ---
>  doc/guides/howto/virtio_user_for_container_networking.rst | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/doc/guides/howto/virtio_user_for_container_networking.rst b/doc/guides/howto/virtio_user_for_container_networking.rst
> index dbb9319..f71d071 100644
> --- a/doc/guides/howto/virtio_user_for_container_networking.rst
> +++ b/doc/guides/howto/virtio_user_for_container_networking.rst
> @@ -111,7 +111,8 @@ some minor changes.
>      .. code-block:: console
>  
>          $(testpmd) -l 0-1 -n 4 --socket-mem 1024,1024 \
> -            --vdev 'eth_vhost0,iface=/tmp/sock0' --no-pci -- -i
> +            --vdev 'eth_vhost0,iface=/tmp/sock0' \
> +            --file-prefix=host --no-pci -- -i
>  
>  #. Start a container instance with a virtio-user port.
>  
> @@ -121,7 +122,8 @@ some minor changes.
>              -v /dev/hugepages:/dev/hugepages \
>              dpdk-app-testpmd testpmd -l 6-7 -n 4 -m 1024 --no-pci \
>              --vdev=virtio_user0,path=/var/run/usvhost \
> -	    -- -i --txqflags=0xf00 --disable-hw-vlan
> +            --file-prefix=container \
> +            -- -i --txqflags=0xf00 --disable-hw-vlan
>  
>  Note: If we run all above setup on the host, it's a shm-based IPC.
>  
> -- 
> 1.8.3.1
>
  

Patch

diff --git a/doc/guides/howto/virtio_user_for_container_networking.rst b/doc/guides/howto/virtio_user_for_container_networking.rst
index dbb9319..f71d071 100644
--- a/doc/guides/howto/virtio_user_for_container_networking.rst
+++ b/doc/guides/howto/virtio_user_for_container_networking.rst
@@ -111,7 +111,8 @@  some minor changes.
     .. code-block:: console
 
         $(testpmd) -l 0-1 -n 4 --socket-mem 1024,1024 \
-            --vdev 'eth_vhost0,iface=/tmp/sock0' --no-pci -- -i
+            --vdev 'eth_vhost0,iface=/tmp/sock0' \
+            --file-prefix=host --no-pci -- -i
 
 #. Start a container instance with a virtio-user port.
 
@@ -121,7 +122,8 @@  some minor changes.
             -v /dev/hugepages:/dev/hugepages \
             dpdk-app-testpmd testpmd -l 6-7 -n 4 -m 1024 --no-pci \
             --vdev=virtio_user0,path=/var/run/usvhost \
-	    -- -i --txqflags=0xf00 --disable-hw-vlan
+            --file-prefix=container \
+            -- -i --txqflags=0xf00 --disable-hw-vlan
 
 Note: If we run all above setup on the host, it's a shm-based IPC.