malloc: fix invalid argument handling
Checks
Commit Message
When adding memory to an external heap, do not go to unlock failure
handler because the memory hotplug lock hasn't been taken out yet.
Fixes: 7d75c31014f7 ("malloc: allow adding memory to named heaps")
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
lib/librte_eal/common/rte_malloc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Comments
05/11/2018 18:26, Anatoly Burakov:
> When adding memory to an external heap, do not go to unlock failure
> handler because the memory hotplug lock hasn't been taken out yet.
>
> Fixes: 7d75c31014f7 ("malloc: allow adding memory to named heaps")
>
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Applied, thanks
@@ -349,8 +349,7 @@ rte_malloc_heap_memory_add(const char *heap_name, void *va_addr, size_t len,
strnlen(heap_name, RTE_HEAP_NAME_MAX_LEN) ==
RTE_HEAP_NAME_MAX_LEN) {
rte_errno = EINVAL;
- ret = -1;
- goto unlock;
+ return -1;
}
rte_rwlock_write_lock(&mcfg->memory_hotplug_lock);