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

test/mempool: fix autotest retry

Browse Source 
Single producer / single consumer mempool handle is stored in static
variable and the mempool allocated if stored value is NULL.
If the mempool is freed, NULL should be restored to make sure that
the mempool is allocated once again next time when the test is run.

Fixes: 8ef772aee072 ("app/test: rework mempool test")
Cc: stable@dpdk.org

Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
This commit is contained in:
Andrew Rybchenko 2018-04-25 18:00:37 +01:00 committed by Thomas Monjalon
parent 896af46bd1
commit 2d645858a9
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
test/test/test_mempool.c
View File

@ -327,17 +327,17 @@ test_mempool_sp_sc(void)
}
if (rte_mempool_lookup("test_mempool_sp_sc") != mp_spsc) {
printf("Cannot lookup mempool from its name\n");
rte_mempool_free(mp_spsc);
RET_ERR();
ret = -1;
goto err;
}
lcore_next = rte_get_next_lcore(lcore_id, 0, 1);
if (lcore_next >= RTE_MAX_LCORE) {
rte_mempool_free(mp_spsc);
RET_ERR();
ret = -1;
goto err;
}
if (rte_eal_lcore_role(lcore_next) != ROLE_RTE) {
rte_mempool_free(mp_spsc);
RET_ERR();
ret = -1;
goto err;
}
rte_spinlock_init(&scsp_spinlock);
memset(scsp_obj_table, 0, sizeof(scsp_obj_table));
@ -348,7 +348,10 @@ test_mempool_sp_sc(void)
if (rte_eal_wait_lcore(lcore_next) < 0)
ret = -1;
err:
rte_mempool_free(mp_spsc);
mp_spsc = NULL;
return ret;
}