diff options
author | Haiyang Tan <haiyangtan@tencent.com> | 2018-10-09 19:09:45 -0700 |
---|---|---|
committer | Damjan Marion <dmarion@me.com> | 2018-10-10 21:32:46 +0000 |
commit | 642829de7958d5cd9c1c88f9845385882a7c211d (patch) | |
tree | 31f7b391e76fa6ce4973df95dee528653d9bf9f7 /src/vppinfra | |
parent | cd80692dfccb31a1200a2c8701151705765e51bc (diff) |
vppinfra: introduce clib_mem_vm_ext_free() to avoid fd leaks
Change-Id: I8691a10493d159a97574550c111f07722960a7cd
Signed-off-by: Haiyang Tan <haiyangtan@tencent.com>
Diffstat (limited to 'src/vppinfra')
-rw-r--r-- | src/vppinfra/linux/mem.c | 11 | ||||
-rw-r--r-- | src/vppinfra/mem.h | 1 |
2 files changed, 12 insertions, 0 deletions
diff --git a/src/vppinfra/linux/mem.c b/src/vppinfra/linux/mem.c index bceb3b217b7..2d968fae0ac 100644 --- a/src/vppinfra/linux/mem.c +++ b/src/vppinfra/linux/mem.c @@ -252,6 +252,17 @@ done: return err; } +void +clib_mem_vm_ext_free (clib_mem_vm_alloc_t * a) +{ + if (a != 0) + { + clib_mem_vm_free (a->addr, 1 << a->log2_page_size); + if (a->fd != -1) + close (a->fd); + } +} + u64 * clib_mem_vm_get_paddr (void *mem, int log2_page_size, int n_pages) { diff --git a/src/vppinfra/mem.h b/src/vppinfra/mem.h index 0702aabb90a..fa7de204fbb 100644 --- a/src/vppinfra/mem.h +++ b/src/vppinfra/mem.h @@ -387,6 +387,7 @@ typedef struct } clib_mem_vm_alloc_t; clib_error_t *clib_mem_vm_ext_alloc (clib_mem_vm_alloc_t * a); +void clib_mem_vm_ext_free (clib_mem_vm_alloc_t * a); u64 clib_mem_vm_get_page_size (int fd); int clib_mem_vm_get_log2_page_size (int fd); u64 *clib_mem_vm_get_paddr (void *mem, int log2_page_size, int n_pages); |