DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] app/test: decrease memory requirements for hash
@ 2016-09-12 11:38 Olivier Matz
  2016-09-23 16:42 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Olivier Matz @ 2016-09-12 11:38 UTC (permalink / raw)
  To: dev, pablo.de.lara.guarch, bruce.richardson

In hash autotest, the size of tables that should be succesfully created
is 32K entries (256KB), except for the table called "different_name",
which is 1M entries (8MB). When memory is too fragmented (with 2M
hugepages), the test can fail.

To avoid allocation failures due to memory fragmentation, decrease the
size of the table to 32K.

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
 app/test/test_hash.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/test/test_hash.c b/app/test/test_hash.c
index 94300e1..2c87efe 100644
--- a/app/test/test_hash.c
+++ b/app/test/test_hash.c
@@ -783,7 +783,7 @@ fbk_hash_unit_test(void)
 	 */
 	struct rte_fbk_hash_params different_name = {
 		.name = "different_name",			/* different name */
-		.entries = RTE_FBK_HASH_ENTRIES_MAX,
+		.entries = LOCAL_FBK_HASH_ENTRIES_MAX,
 		.entries_per_bucket = 4,
 		.socket_id = 0,
 	};
-- 
2.8.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] app/test: decrease memory requirements for hash
  2016-09-12 11:38 [dpdk-dev] [PATCH] app/test: decrease memory requirements for hash Olivier Matz
@ 2016-09-23 16:42 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2016-09-23 16:42 UTC (permalink / raw)
  To: Olivier Matz; +Cc: dev, pablo.de.lara.guarch, bruce.richardson

2016-09-12 13:38, Olivier Matz:
> In hash autotest, the size of tables that should be succesfully created
> is 32K entries (256KB), except for the table called "different_name",
> which is 1M entries (8MB). When memory is too fragmented (with 2M
> hugepages), the test can fail.
> 
> To avoid allocation failures due to memory fragmentation, decrease the
> size of the table to 32K.
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-09-23 16:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-12 11:38 [dpdk-dev] [PATCH] app/test: decrease memory requirements for hash Olivier Matz
2016-09-23 16:42 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).