replace panic calls with log and return value. -- v2: - update doxigen to include new error value - reformat error message to include literal string in a single line
Signed-off-by: Arnon Warshavsky <ar...@qwilt.com> --- lib/librte_eal/common/eal_common_memzone.c | 3 ++- lib/librte_eal/common/include/rte_memzone.h | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/librte_eal/common/eal_common_memzone.c b/lib/librte_eal/common/eal_common_memzone.c index 1ab3ade..fe058cc 100644 --- a/lib/librte_eal/common/eal_common_memzone.c +++ b/lib/librte_eal/common/eal_common_memzone.c @@ -314,8 +314,9 @@ if (addr == NULL) ret = -EINVAL; else if (mcfg->memzone_cnt == 0) { - rte_panic("%s(): memzone address not NULL but memzone_cnt is 0!\n", + RTE_LOG(CRIT, EAL, "%s(): memzone address not NULL but memzone_cnt is 0!\n", __func__); + return -1; } else { memset(&mcfg->memzone[idx], 0, sizeof(mcfg->memzone[idx])); mcfg->memzone_cnt--; diff --git a/lib/librte_eal/common/include/rte_memzone.h b/lib/librte_eal/common/include/rte_memzone.h index 2bfb273..e2b4c7d 100644 --- a/lib/librte_eal/common/include/rte_memzone.h +++ b/lib/librte_eal/common/include/rte_memzone.h @@ -234,6 +234,7 @@ const struct rte_memzone *rte_memzone_reserve_bounded(const char *name, * A pointer to the memzone * @return * -EINVAL - invalid parameter. + * -1 - internal state error * 0 - success */ int rte_memzone_free(const struct rte_memzone *mz); -- 1.8.3.1