From 76480e38856e70345b473efbf28a1ef453e33a4b Mon Sep 17 00:00:00 2001 From: Anatoly Burakov Date: Thu, 31 May 2018 17:11:47 +0100 Subject: [PATCH] mem: mark pages as freeable on exit When rte_eal_cleanup() is called, it is expected that DPDK will be able to release all of its memory back to the system. However, if pages are marked as unfreeable, the pages will not be released back. Fix this to mark all pages as freeable on calling rte_eal_cleanup(), but only do it for primary process, as secondaries can come and go. Signed-off-by: Anatoly Burakov --- lib/librte_eal/linuxapp/eal/eal.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c index 8655b86915..987b57f877 100644 --- a/lib/librte_eal/linuxapp/eal/eal.c +++ b/lib/librte_eal/linuxapp/eal/eal.c @@ -1044,9 +1044,26 @@ rte_eal_init(int argc, char **argv) return fctret; } +static int +mark_freeable(const struct rte_memseg_list *msl, const struct rte_memseg *ms, + void *arg __rte_unused) +{ + /* ms is const, so find this memseg */ + struct rte_memseg *found = rte_mem_virt2memseg(ms->addr, msl); + + found->flags &= ~RTE_MEMSEG_FLAG_DO_NOT_FREE; + + return 0; +} + int __rte_experimental rte_eal_cleanup(void) { + /* if we're in a primary process, we need to mark hugepages as freeable + * so that finalization can release them back to the system. + */ + if (rte_eal_process_type() == RTE_PROC_PRIMARY) + rte_memseg_walk(mark_freeable, NULL); rte_service_finalize(); return 0; } -- 2.20.1