d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

test/memzone: fix wrong test

Browse Source 
When reserving memzones in autotest, it makes no sense to expect a
failed memzone reserve when we specify both size flags - instead,
we should expect a memzone reserved with one of the two sizes.

Fixes: af75078fec ("first public release")
Cc: stable@dpdk.org

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
This commit is contained in:
Anatoly Burakov 2017-12-21 18:19:44 +00:00 committed by Thomas Monjalon
parent 97d144a8a2
commit 82da6af3ca
1 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
test/test/test_memzone.c
View File

@ -260,10 +260,19 @@ test_memzone_reserve_flags(void)
if (hugepage_2MB_avail && hugepage_1GB_avail) {
mz = rte_memzone_reserve("flag_zone_2M_HINT", size, SOCKET_ID_ANY,
RTE_MEMZONE_2MB|RTE_MEMZONE_1GB);
if (mz != NULL) {
if (mz == NULL) {
printf("BOTH SIZES SET\n");
return -1;
}
if (mz->hugepage_sz != RTE_PGSIZE_1G &&
mz->hugepage_sz != RTE_PGSIZE_2M) {
printf("Wrong size when both sizes set\n");
return -1;
}
if (rte_memzone_free(mz)) {
printf("Fail memzone free\n");
return -1;
}
}
}
/*
@ -395,10 +404,19 @@ test_memzone_reserve_flags(void)
mz = rte_memzone_reserve("flag_zone_16M_HINT", size,
SOCKET_ID_ANY,
RTE_MEMZONE_16MB|RTE_MEMZONE_16GB);
if (mz != NULL) {
if (mz == NULL) {
printf("BOTH SIZES SET\n");
return -1;
}
if (mz->hugepage_sz != RTE_PGSIZE_16G &&
mz->hugepage_sz != RTE_PGSIZE_16M) {
printf("Wrong size when both sizes set\n");
return -1;
}
if (rte_memzone_free(mz)) {
printf("Fail memzone free\n");
return -1;
}
}
}
return 0;