[dpdk-dev,v2] doc: add note for MAX QUEUES macro in vhost sample

Message ID 20180524120507.5089-1-xiao.w.wang@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Maxime Coquelin
Headers

Checks

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

Commit Message

Xiao Wang May 24, 2018, 12:05 p.m. UTC
  If the NIC has a queue number larger than 128, then we need to change
the ``MAX_QUEUES`` to a larger number to make sure we allocate a big
enough memory pool for device setup.

Signed-off-by: Xiao Wang <xiao.w.wang@intel.com>
Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
---
v2:
- Fix unexpected indentation warning.
---
 doc/guides/sample_app_ug/vhost.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)
  

Comments

Maxime Coquelin May 24, 2018, 12:33 p.m. UTC | #1
On 05/24/2018 02:05 PM, Xiao Wang wrote:
> If the NIC has a queue number larger than 128, then we need to change
> the ``MAX_QUEUES`` to a larger number to make sure we allocate a big
> enough memory pool for device setup.
> 
> Signed-off-by: Xiao Wang <xiao.w.wang@intel.com>
> Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
> ---
> v2:
> - Fix unexpected indentation warning.
> ---
>   doc/guides/sample_app_ug/vhost.rst | 11 +++++++++++
>   1 file changed, 11 insertions(+)
> 

Applied to dpdk-next-virtio/master.

Thanks,
Maxime
  

Patch

diff --git a/doc/guides/sample_app_ug/vhost.rst b/doc/guides/sample_app_ug/vhost.rst
index 5e7f24c4e..9112e7897 100644
--- a/doc/guides/sample_app_ug/vhost.rst
+++ b/doc/guides/sample_app_ug/vhost.rst
@@ -181,3 +181,14 @@  Common Issues
 * Failed to build DPDK in VM
 
   Make sure "-cpu host" QEMU option is given.
+
+* Device start fails if NIC's max queues > the default number of 128
+
+  mbuf pool size is dependent on the MAX_QUEUES configuration, if NIC's
+  max queue number is larger than 128, device start will fail due to
+  insufficient mbuf.
+
+  Change the default number to make it work as below, just set the number
+  according to the NIC's property. ::
+
+      make EXTRA_CFLAGS="-DMAX_QUEUES=320"