From: Gaetan Rivet <gaetan.rivet@6wind.com>
Date: Mon, 1 Jun 2015 12:11:00 +0000 (+0200)
Subject: app/test: fix default numa memory assignment
X-Git-Tag: spdx-start~9105
X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=a3894e2fbc30da56dcb76e03a41ede2869590b50;p=dpdk.git

app/test: fix default numa memory assignment

Each test requires a certain minimal amount of memory.
Spreading memory on all sockets means that the test will get less memory than
what it wanted on multi sockets system.
So replace all_sockets() with per_sockets().

Also doubled memory on group_5 as current requirement is not enough.

Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>
Signed-off-by: David Marchand <david.marchand@6wind.com>
---

diff --git a/app/test/autotest_data.py b/app/test/autotest_data.py
index 02921ff2c5..0c3802bed2 100644
--- a/app/test/autotest_data.py
+++ b/app/test/autotest_data.py
@@ -43,11 +43,10 @@ def num_sockets():
 		return 1
 	return result
 
-# spread given number for all sockets
-# e.g. 32 becomes 16,16 or 8,8,8,8 etc.
-def all_sockets(num):
-	mem_per_socket = num / num_sockets()
-	return ",".join([str(mem_per_socket)] * num_sockets())
+# Assign given number to each socket
+# e.g. 32 becomes 32,32 or 32,32,32,32
+def per_sockets(num):
+    return ",".join([str(num)] * num_sockets())
 
 # groups of tests that can be run in parallel
 # the grouping has been found largely empirically
@@ -55,7 +54,7 @@ parallel_test_group_list = [
 
 {
 	"Prefix":	"group_1",
-	"Memory" :	all_sockets(8),
+	"Memory" :	per_sockets(8),
 	"Tests" :
 	[
 		{
@@ -165,7 +164,7 @@ parallel_test_group_list = [
 },
 {
 	"Prefix":	"group_3",
-	"Memory" :	all_sockets(1024),
+	"Memory" :	per_sockets(1024),
 	"Tests" :
 	[
 		{
@@ -208,7 +207,7 @@ parallel_test_group_list = [
 },
 {
 	"Prefix":	"group_4",
-	"Memory" :	all_sockets(128),
+	"Memory" :	per_sockets(128),
 	"Tests" :
 	[
 		{
@@ -251,7 +250,7 @@ parallel_test_group_list = [
 },
 {
 	"Prefix":	"group_5",
-	"Memory" :	"16",
+	"Memory" :	"32",
 	"Tests" :
 	[
 		{
@@ -288,7 +287,7 @@ parallel_test_group_list = [
 },
 {
 	"Prefix":	"group_6",
-	"Memory" :	all_sockets(620),
+	"Memory" :	per_sockets(620),
 	"Tests" :
 	[
 		{
@@ -368,7 +367,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix":	"mempool_perf",
-	"Memory" :	all_sockets(256),
+	"Memory" :	per_sockets(256),
 	"Tests" :
 	[
 		{
@@ -387,7 +386,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix":	"memcpy_perf",
-	"Memory" :	all_sockets(512),
+	"Memory" :	per_sockets(512),
 	"Tests" :
 	[
 		{
@@ -400,7 +399,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix":	"hash_perf",
-	"Memory" :	all_sockets(512),
+	"Memory" :	per_sockets(512),
 	"Tests" :
 	[
 		{
@@ -413,7 +412,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix" :      "power",
-	"Memory" :      all_sockets(512),
+	"Memory" :      per_sockets(512),
 	"Tests" :
 	[
 		{
@@ -426,7 +425,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix" :      "power_acpi_cpufreq",
-	"Memory" :      all_sockets(512),
+	"Memory" :      per_sockets(512),
 	"Tests" :
 	[
 		{
@@ -465,7 +464,7 @@ non_parallel_test_group_list = [
 },
 {
 	"Prefix":	"timer_perf",
-	"Memory" :	all_sockets(512),
+	"Memory" :	per_sockets(512),
 	"Tests" :
 	[
 		{
@@ -482,7 +481,7 @@ non_parallel_test_group_list = [
 #
 {
 	"Prefix":	"ring_perf",
-	"Memory" :	all_sockets(512),
+	"Memory" :	per_sockets(512),
 	"Tests" :
 	[
 		{