Hi, > -----Original Message----- > From: Ma, WenwuX <[email protected]> > Sent: Wednesday, November 3, 2021 10:17 PM > To: [email protected] > Cc: [email protected]; Xia, Chenbo <[email protected]>; Yigit, > Ferruh <[email protected]>; Jiang, Cheng1 <[email protected]>; Hu, > Jiayu <[email protected]>; Yang, YvonneX <[email protected]>; Ma, WenwuX > <[email protected]>; [email protected] > Subject: [PATCH v4] examples/vhost: fix failure to launch dpdk-vhost > > When the default max queue value of vhost is 128, dpdk-vhost will fail to > launch with a 40G i40e port due to the mbuf allocation of the port queue > fails. > Therefore, we modify the default value of max queue from 128 to 512, > it will ensure that there are enough mbufs. > > Fixes: 4796ad63ba1f ("examples/vhost: import userspace vhost application") > Cc: [email protected]
When I was tracking the bad commit, I notice this change is on purpose to save hugepages: commit f17eb179046962fdc29c3609b1c9bd46bfefa4f5 Author: Bernard Iremonger <[email protected]> Date: Thu Dec 10 15:49:17 2015 +0000 examples/vhost: reduce number of hugepages needed Add #ifndef MAX_QUEUES to change MAX_QUEUES at compile time if needed. Change MAX_QUEUES from 512 to 128 to reduce the number of hugepages required by the vhost-switch program. To change MAX_QUEUES add '-D MAX_QUEUES=512' to the EXTRA_CFLAGS variable, before building the application. Signed-off-by: Bernard Iremonger <[email protected]> Acked-by: Konstantin Ananyev <[email protected]> So as there's a way to solve the problem (although not perfect), I would suggest using the compiler flag to do tests with the specific NIC. If no one has objection, let's drop the patch. Thanks, Chenbo > > Signed-off-by: Wenwu Ma <[email protected]> > Reviewed-by: Maxime Coquelin <[email protected]> > --- > examples/vhost/main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/examples/vhost/main.c b/examples/vhost/main.c > index bc3d71c898..36969a4de5 100644 > --- a/examples/vhost/main.c > +++ b/examples/vhost/main.c > @@ -29,7 +29,7 @@ > #include "main.h" > > #ifndef MAX_QUEUES > -#define MAX_QUEUES 128 > +#define MAX_QUEUES 512 > #endif > > /* the maximum number of external ports supported */ > -- > 2.25.1
