Re: [PATCH v2] doc: add note for builtin-net-driver option
From: Tiwei Bie <hidden>
Date: 2018-07-31 07:29:18
On Thu, Jul 26, 2018 at 10:27:38PM +0800, Marvin Liu wrote:
quoted hunk ↗ jump to hunk
Option builtin-net-driver is used to demonstrate how to implement vhost-user driver based on vhost lib APIs. Unfortunately, very simple vhost driver in vhost sample is incompatible with QEMU as protocol feature not supported. Signed-off-by: Marvin Liu <redacted> --- v2: * Updated descriptions. ---diff --git a/doc/guides/sample_app_ug/vhost.rst b/doc/guides/sample_app_ug/vhost.rst index 9112e7897..396d83211 100644 --- a/doc/guides/sample_app_ug/vhost.rst +++ b/doc/guides/sample_app_ug/vhost.rst@@ -158,6 +158,11 @@ when disabling VLAN strip. Such feature, which heavily depends on hardware, should be removed from this example to reduce confusion. Now, VLAN strip is enabled and cannot be disabled. +**--builtin-net-driver** +Very simple enqueue and dequeue function will be used when this option is +given. It can demonstrate how to construct vhost-user driver with vhost lib +APIs. It is disabled by default.
Maybe something like: A very simple vhost-user net driver which demonstrates how to use the generic vhost APIs will be used when this option is given. It is disabled by default.
quoted hunk ↗ jump to hunk
+ Common Issues -------------@@ -192,3 +197,8 @@ Common Issues according to the NIC's property. :: make EXTRA_CFLAGS="-DMAX_QUEUES=320" + +* Option "builtin-net-driver" is incompatible with QEMU.
Better to remove above '.'
+ + QEMU vhost net device start will fail if protocol feature is not + negotiated. DPDK virtio-user can be the replacement of QEMU. -- 2.17.0