devtools: fix required memory for null test
diff mbox series

Message ID 20191120092835.25325-1-thomas@monjalon.net
State Rejected, archived
Delegated to: David Marchand
Headers show
Series
  • devtools: fix required memory for null test
Related show

Checks

Context Check Description
ci/Intel-compilation fail Compilation issues
ci/travis-robot success Travis build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-compilation success Compile Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/checkpatch success coding style OK

Commit Message

Thomas Monjalon Nov. 20, 2019, 9:28 a.m. UTC
The testpmd fails in memory allocation since some ethdev structs
have been extended.
Increasing memory allocation from 150 to 300 MB makes it working again.

Fixes: 436b3a6b6e62 ("ethdev: reserve space in main structs for extension")

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 devtools/test-null.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Marchand Nov. 22, 2019, 12:15 p.m. UTC | #1
On Wed, Nov 20, 2019 at 10:29 AM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> The testpmd fails in memory allocation since some ethdev structs
> have been extended.
> Increasing memory allocation from 150 to 300 MB makes it working again.
>
> Fixes: 436b3a6b6e62 ("ethdev: reserve space in main structs for extension")

Fixed with https://patchwork.dpdk.org/patch/63226/
Marking as rejected.

Patch
diff mbox series

diff --git a/devtools/test-null.sh b/devtools/test-null.sh
index 9f9a459f76..f2396a4ca0 100755
--- a/devtools/test-null.sh
+++ b/devtools/test-null.sh
@@ -25,6 +25,6 @@  else
 fi
 
 (sleep 1 && echo stop) |
-$testpmd -c $coremask --no-huge -m 150 \
+$testpmd -c $coremask --no-huge -m 300 \
 	$libs --vdev net_null1 --vdev net_null2 $eal_options -- \
 	--no-mlockall --total-num-mbufs=2048 $testpmd_options -ia