From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
by inbox.dpdk.org (Postfix) with ESMTP id F34F343F09;
Thu, 25 Apr 2024 20:28:16 +0200 (CEST)
Received: from mails.dpdk.org (localhost [127.0.0.1])
by mails.dpdk.org (Postfix) with ESMTP id 09AE143962;
Thu, 25 Apr 2024 20:27:57 +0200 (CEST)
Received: from mail-pf1-f173.google.com (mail-pf1-f173.google.com
[209.85.210.173])
by mails.dpdk.org (Postfix) with ESMTP id 5D32B4394C
for <dev@dpdk.org>; Thu, 25 Apr 2024 20:27:52 +0200 (CEST)
Received: by mail-pf1-f173.google.com with SMTP id
d2e1a72fcca58-6f07de6ab93so1272751b3a.2
for <dev@dpdk.org>; Thu, 25 Apr 2024 11:27:52 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1714069671;
x=1714674471; darn=dpdk.org;
h=content-transfer-encoding:mime-version:references:in-reply-to
:message-id:date:subject:cc:to:from:from:to:cc:subject:date
:message-id:reply-to;
bh=eToGpaZd1km8aJMGcaBIZz+3ftXcaJlJuBtSwUPoz8U=;
b=AeMjrlYI+mrKfk0oLJc/nv1Tn0JhPesIRurfFORh9z7Td6MD2P2Hh1U8+fDLqDreAP
vuu1fLROGj56/N6q8IEOKt6Dh6wrDE3FbXdaHYVVXM3bYzILpKjY7xoXWC+LcNOdLdFa
CM4NdKf0L+x+6qyhFNDudTqUQONG7yEmobIqL1eDwER3+L5K8UuN95JBXr8XhGQNEizR
5zzNfK4plq66DK6Vmgh+3sTflRPvhTpaFZVDVB3upM/qGjI1K2cfG6f1t0WWgil6W9xt
yMrNJindV2ABC3F43rlacgui5iPrFeGJvVb21/7EFkVK3KopFHXmF1HQmC0t9cueLb0U
trxA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=1e100.net; s=20230601; t=1714069671; x=1714674471;
h=content-transfer-encoding:mime-version:references:in-reply-to
:message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc
:subject:date:message-id:reply-to;
bh=eToGpaZd1km8aJMGcaBIZz+3ftXcaJlJuBtSwUPoz8U=;
b=L8I0vfQ/kLg7Ub2drDWCm3Ym5N9YxjnZu3wXHhIs73z30ZmgVJ5Tloux+gqrS/a0vO
RrljIbA2kFvIk/Hu4XOZf9NLtVeGnC9r1E/gakfBjQoTma25foZfgycYopy2E8O2gWqH
fz6ZyT5U+TGnp4qDEPOOIWVPh6fgMYxaek4Ra6ST2BKHLaOE3eAM1K72csE+K976Z4G/
5QXUhouCyjKcWoVEfwO849MhEsm1OqICcUHo/zBP4WRSC1hqjtc8ieP8bgGv1biQGaPw
gopHjjyykXGiD+9x0tcgKrDR5+8PMdYFCfeEHY99avxL84bA8MC4QWIXeKlEsT03Ybnr
GNEQ==
X-Gm-Message-State: AOJu0Yw0xRqTb014TSuHkZSnLwID3IOGxLnndx+5MZQR5v1jj2A/3gXz
/Z6iM4AToIDBDGlSgsNA8Q+WkETgqFIfP38LMy9jXoWbccNnvZ13QTHGLpE9BkxYNFLurW3Qu+p
O
X-Google-Smtp-Source: AGHT+IGAI4Udb8TU58DHFIDzZycafT2UzwkH7RQAUrKflKoGxfOF/Bbycu+09ErvPlI1sRjP2RuBgw==
X-Received: by 2002:a05:6a21:144c:b0:1a7:1b6e:4d4 with SMTP id
oc12-20020a056a21144c00b001a71b6e04d4mr460494pzb.23.1714069671495;
Thu, 25 Apr 2024 11:27:51 -0700 (PDT)
Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226])
by smtp.gmail.com with ESMTPSA id
d7-20020a056a00244700b006f0ba1c8ddesm4067006pfj.184.2024.04.25.11.27.50
(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
Thu, 25 Apr 2024 11:27:51 -0700 (PDT)
From: Stephen Hemminger <stephen@networkplumber.org>
To: dev@dpdk.org
Cc: Stephen Hemminger <stephen@networkplumber.org>
Subject: [RFC 4/4] eal/malloc: remove type argument from internal malloc
routines
Date: Thu, 25 Apr 2024 11:24:01 -0700
Message-ID: <20240425182738.4771-5-stephen@networkplumber.org>
X-Mailer: git-send-email 2.43.0
In-Reply-To: <20240425182738.4771-1-stephen@networkplumber.org>
References: <20240425182738.4771-1-stephen@networkplumber.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
The type argument is carried through malloc heap routines but
never used.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
lib/eal/common/eal_common_memzone.c | 6 ++---
lib/eal/common/malloc_heap.c | 39 ++++++++++++----------------
lib/eal/common/malloc_heap.h | 7 +++--
lib/eal/common/rte_malloc.c | 16 +++++-------
lib/eal/include/eal_trace_internal.h | 4 +--
5 files changed, 29 insertions(+), 43 deletions(-)
diff --git a/lib/eal/common/eal_common_memzone.c b/lib/eal/common/eal_common_memzone.c
index 32e6b78f87..2d9b6aa3e3 100644
--- a/lib/eal/common/eal_common_memzone.c
+++ b/lib/eal/common/eal_common_memzone.c
@@ -191,14 +191,12 @@ memzone_reserve_aligned_thread_unsafe(const char *name, size_t len,
if (len == 0 && bound == 0) {
/* no size constraints were placed, so use malloc elem len */
requested_len = 0;
- mz_addr = malloc_heap_alloc_biggest(NULL, socket_id, flags,
- align, contig);
+ mz_addr = malloc_heap_alloc_biggest(socket_id, flags, align, contig);
} else {
if (len == 0)
requested_len = bound;
/* allocate memory on heap */
- mz_addr = malloc_heap_alloc(NULL, requested_len, socket_id,
- flags, align, bound, contig);
+ mz_addr = malloc_heap_alloc(requested_len, socket_id, flags, align, bound, contig);
}
if (mz_addr == NULL) {
rte_errno = ENOMEM;
diff --git a/lib/eal/common/malloc_heap.c b/lib/eal/common/malloc_heap.c
index 5ff27548ff..058aaf4209 100644
--- a/lib/eal/common/malloc_heap.c
+++ b/lib/eal/common/malloc_heap.c
@@ -229,8 +229,8 @@ find_biggest_element(struct malloc_heap *heap, size_t *size,
* the new element after releasing the lock.
*/
static void *
-heap_alloc(struct malloc_heap *heap, const char *type __rte_unused, size_t size,
- unsigned int flags, size_t align, size_t bound, bool contig)
+heap_alloc(struct malloc_heap *heap, size_t size, unsigned int flags,
+ size_t align, size_t bound, bool contig)
{
struct malloc_elem *elem;
size_t user_size = size;
@@ -255,8 +255,7 @@ heap_alloc(struct malloc_heap *heap, const char *type __rte_unused, size_t size,
}
static void *
-heap_alloc_biggest(struct malloc_heap *heap, const char *type __rte_unused,
- unsigned int flags, size_t align, bool contig)
+heap_alloc_biggest(struct malloc_heap *heap, unsigned int flags, size_t align, bool contig)
{
struct malloc_elem *elem;
size_t size;
@@ -640,8 +639,7 @@ alloc_more_mem_on_socket(struct malloc_heap *heap, size_t size, int socket,
/* this will try lower page sizes first */
static void *
-malloc_heap_alloc_on_heap_id(const char *type, size_t size,
- unsigned int heap_id, unsigned int flags, size_t align,
+malloc_heap_alloc_on_heap_id(size_t size, unsigned int heap_id, unsigned int flags, size_t align,
size_t bound, bool contig)
{
struct rte_mem_config *mcfg = rte_eal_get_configuration()->mem_config;
@@ -658,7 +656,7 @@ malloc_heap_alloc_on_heap_id(const char *type, size_t size,
/* for legacy mode, try once and with all flags */
if (internal_conf->legacy_mem) {
- ret = heap_alloc(heap, type, size, flags, align, bound, contig);
+ ret = heap_alloc(heap, size, flags, align, bound, contig);
goto alloc_unlock;
}
@@ -679,7 +677,7 @@ malloc_heap_alloc_on_heap_id(const char *type, size_t size,
if (socket_id < 0)
size_flags |= RTE_MEMZONE_SIZE_HINT_ONLY;
- ret = heap_alloc(heap, type, size, size_flags, align, bound, contig);
+ ret = heap_alloc(heap, size, size_flags, align, bound, contig);
if (ret != NULL)
goto alloc_unlock;
@@ -689,7 +687,7 @@ malloc_heap_alloc_on_heap_id(const char *type, size_t size,
if (!alloc_more_mem_on_socket(heap, size, socket_id, flags, align,
bound, contig)) {
- ret = heap_alloc(heap, type, size, flags, align, bound, contig);
+ ret = heap_alloc(heap, size, flags, align, bound, contig);
/* this should have succeeded */
if (ret == NULL)
@@ -730,8 +728,8 @@ malloc_get_numa_socket(void)
}
void *
-malloc_heap_alloc(const char *type, size_t size, int socket_arg,
- unsigned int flags, size_t align, size_t bound, bool contig)
+malloc_heap_alloc(size_t size, int socket_arg, unsigned int flags,
+ size_t align, size_t bound, bool contig)
{
int socket, heap_id, i;
void *ret;
@@ -754,8 +752,7 @@ malloc_heap_alloc(const char *type, size_t size, int socket_arg,
if (heap_id < 0)
return NULL;
- ret = malloc_heap_alloc_on_heap_id(type, size, heap_id, flags, align,
- bound, contig);
+ ret = malloc_heap_alloc_on_heap_id(size, heap_id, flags, align, bound, contig);
if (ret != NULL || socket_arg != SOCKET_ID_ANY)
return ret;
@@ -765,8 +762,7 @@ malloc_heap_alloc(const char *type, size_t size, int socket_arg,
for (i = 0; i < (int) rte_socket_count(); i++) {
if (i == heap_id)
continue;
- ret = malloc_heap_alloc_on_heap_id(type, size, i, flags, align,
- bound, contig);
+ ret = malloc_heap_alloc_on_heap_id(size, i, flags, align, bound, contig);
if (ret != NULL)
return ret;
}
@@ -774,7 +770,7 @@ malloc_heap_alloc(const char *type, size_t size, int socket_arg,
}
static void *
-heap_alloc_biggest_on_heap_id(const char *type, unsigned int heap_id,
+heap_alloc_biggest_on_heap_id(unsigned int heap_id,
unsigned int flags, size_t align, bool contig)
{
struct rte_mem_config *mcfg = rte_eal_get_configuration()->mem_config;
@@ -785,7 +781,7 @@ heap_alloc_biggest_on_heap_id(const char *type, unsigned int heap_id,
align = align == 0 ? 1 : align;
- ret = heap_alloc_biggest(heap, type, flags, align, contig);
+ ret = heap_alloc_biggest(heap, flags, align, contig);
rte_spinlock_unlock(&(heap->lock));
@@ -793,8 +789,7 @@ heap_alloc_biggest_on_heap_id(const char *type, unsigned int heap_id,
}
void *
-malloc_heap_alloc_biggest(const char *type, int socket_arg, unsigned int flags,
- size_t align, bool contig)
+malloc_heap_alloc_biggest(int socket_arg, unsigned int flags, size_t align, bool contig)
{
int socket, i, cur_socket, heap_id;
void *ret;
@@ -817,8 +812,7 @@ malloc_heap_alloc_biggest(const char *type, int socket_arg, unsigned int flags,
if (heap_id < 0)
return NULL;
- ret = heap_alloc_biggest_on_heap_id(type, heap_id, flags, align,
- contig);
+ ret = heap_alloc_biggest_on_heap_id(heap_id, flags, align, contig);
if (ret != NULL || socket_arg != SOCKET_ID_ANY)
return ret;
@@ -827,8 +821,7 @@ malloc_heap_alloc_biggest(const char *type, int socket_arg, unsigned int flags,
cur_socket = rte_socket_id_by_idx(i);
if (cur_socket == socket)
continue;
- ret = heap_alloc_biggest_on_heap_id(type, i, flags, align,
- contig);
+ ret = heap_alloc_biggest_on_heap_id(i, flags, align, contig);
if (ret != NULL)
return ret;
}
diff --git a/lib/eal/common/malloc_heap.h b/lib/eal/common/malloc_heap.h
index 0c49588005..dfc56d4ae3 100644
--- a/lib/eal/common/malloc_heap.h
+++ b/lib/eal/common/malloc_heap.h
@@ -34,12 +34,11 @@ struct __rte_cache_aligned malloc_heap {
};
void *
-malloc_heap_alloc(const char *type, size_t size, int socket, unsigned int flags,
- size_t align, size_t bound, bool contig);
+malloc_heap_alloc(size_t size, int socket, unsigned int flags, size_t align,
+ size_t bound, bool contig);
void *
-malloc_heap_alloc_biggest(const char *type, int socket, unsigned int flags,
- size_t align, bool contig);
+malloc_heap_alloc_biggest(int socket, unsigned int flags, size_t align, bool contig);
int
malloc_heap_create(struct malloc_heap *heap, const char *heap_name);
diff --git a/lib/eal/common/rte_malloc.c b/lib/eal/common/rte_malloc.c
index 6d3c301a23..d1fcb2eff6 100644
--- a/lib/eal/common/rte_malloc.c
+++ b/lib/eal/common/rte_malloc.c
@@ -51,8 +51,7 @@ eal_free_no_trace(void *addr)
}
static void *
-malloc_socket(const char *type, size_t size, unsigned int align,
- int socket_arg, const bool trace_ena)
+malloc_socket(size_t size, unsigned int align, int socket_arg, const bool trace_ena)
{
void *ptr;
@@ -69,11 +68,11 @@ malloc_socket(const char *type, size_t size, unsigned int align,
!rte_eal_has_hugepages())
socket_arg = SOCKET_ID_ANY;
- ptr = malloc_heap_alloc(type, size, socket_arg, 0,
+ ptr = malloc_heap_alloc(size, socket_arg, 0,
align == 0 ? 1 : align, 0, false);
if (trace_ena)
- rte_eal_trace_mem_malloc(type, size, align, socket_arg, ptr);
+ rte_eal_trace_mem_malloc(size, align, socket_arg, ptr);
return ptr;
}
@@ -81,16 +80,15 @@ malloc_socket(const char *type, size_t size, unsigned int align,
* Allocate memory on specified heap.
*/
void *
-rte_malloc_socket(const char *type, size_t size, unsigned int align,
- int socket_arg)
+rte_malloc_socket(const char *type __rte_unused, size_t size, unsigned int align, int socket_arg)
{
- return malloc_socket(type, size, align, socket_arg, true);
+ return malloc_socket(size, align, socket_arg, true);
}
void *
-eal_malloc_no_trace(const char *type, size_t size, unsigned int align)
+eal_malloc_no_trace(const char *type __rte_unused, size_t size, unsigned int align)
{
- return malloc_socket(type, size, align, SOCKET_ID_ANY, false);
+ return malloc_socket(size, align, SOCKET_ID_ANY, false);
}
/*
diff --git a/lib/eal/include/eal_trace_internal.h b/lib/eal/include/eal_trace_internal.h
index 09c354717f..fa104c975f 100644
--- a/lib/eal/include/eal_trace_internal.h
+++ b/lib/eal/include/eal_trace_internal.h
@@ -103,9 +103,7 @@ RTE_TRACE_POINT(
RTE_TRACE_POINT(
rte_eal_trace_mem_malloc,
- RTE_TRACE_POINT_ARGS(const char *type, size_t size, unsigned int align,
- int socket, void *ptr),
- rte_trace_point_emit_string(type);
+ RTE_TRACE_POINT_ARGS(size_t size, unsigned int align, int socket, void *ptr),
rte_trace_point_emit_size_t(size);
rte_trace_point_emit_u32(align);
rte_trace_point_emit_int(socket);
--
2.43.0