From 5348806e43bdaf5e5b12a1b9b9f5c816c9d1139a Mon Sep 17 00:00:00 2001 From: Stephen Hemminger Date: Wed, 26 Jun 2019 15:32:12 -0700 Subject: [PATCH] malloc: deprecate unused function to set limit The function rte_malloc_set_limit was defined but never implemented. Mark it as deprecated for now, and remove in next release. There is no point in keeping dead code. "You Aren't Going to Need It" Signed-off-by: Stephen Hemminger Acked-by: Anatoly Burakov --- app/test/test_malloc.c | 21 --------------------- doc/guides/rel_notes/release_19_08.rst | 3 +++ lib/librte_eal/common/include/rte_malloc.h | 1 + 3 files changed, 4 insertions(+), 21 deletions(-) diff --git a/app/test/test_malloc.c b/app/test/test_malloc.c index f0e608cdaf..7243e83894 100644 --- a/app/test/test_malloc.c +++ b/app/test/test_malloc.c @@ -372,18 +372,6 @@ test_multi_alloc_statistics(void) return 0; } -static int -test_rte_malloc_type_limits(void) -{ - /* The type-limits functionality is not yet implemented, - * so always return 0 no matter what the retval. - */ - const char *typename = "limit_test"; - rte_malloc_set_limit(typename, 64 * 1024); - rte_malloc_dump_stats(stdout, typename); - return 0; -} - static int test_realloc(void) { @@ -952,15 +940,6 @@ test_malloc(void) } else printf("test_random_alloc_free() passed\n"); - /*----------------------------*/ - ret = test_rte_malloc_type_limits(); - if (ret < 0){ - printf("test_rte_malloc_type_limits() failed\n"); - return ret; - } - /* TODO: uncomment following line once type limits are valid */ - /*else printf("test_rte_malloc_type_limits() passed\n");*/ - /*----------------------------*/ ret = test_rte_malloc_validate(); if (ret < 0){ diff --git a/doc/guides/rel_notes/release_19_08.rst b/doc/guides/rel_notes/release_19_08.rst index 8e0b13e051..60c2581361 100644 --- a/doc/guides/rel_notes/release_19_08.rst +++ b/doc/guides/rel_notes/release_19_08.rst @@ -162,6 +162,9 @@ API Changes * The network structures, definitions and functions have been prefixed by ``rte_`` to resolve conflicts with libc headers. +* malloc: The function ``rte_malloc_set_limit`` was never implemented + is deprecated and will be removed in a future release. + ABI Changes ----------- diff --git a/lib/librte_eal/common/include/rte_malloc.h b/lib/librte_eal/common/include/rte_malloc.h index 3d4fad40eb..3593fb45c7 100644 --- a/lib/librte_eal/common/include/rte_malloc.h +++ b/lib/librte_eal/common/include/rte_malloc.h @@ -536,6 +536,7 @@ rte_malloc_dump_heaps(FILE *f); * - 0: Success. * - (-1): Error. */ +__rte_deprecated int rte_malloc_set_limit(const char *type, size_t max); -- 2.20.1