From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2AF52C531CF for ; Thu, 23 Jul 2026 12:57:01 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3AA7E40B8F; Thu, 23 Jul 2026 14:56:49 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id D53E940A72 for ; Thu, 23 Jul 2026 14:56:47 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1784811407; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=DHCBBgbv2rlKziPT+rwRQxCCXbdOU/s+ntlgfXlGjU8=; b=PFmVFnGJhWMap3JDCnAijuPvbJytzN/SifeQAakLIKUyZRXl8Z3VjPH1pibXYR7zkfHSPM qL+xO19EKG3EX1Jxvc4d7J1ZOQBw4kKYDUGtl4RqnHFvZB05JYVT9HgqxaHEPVl/IOYlHt FDUBzuEEIwzF+9orJkgLxFSmquubcqg= Received: from mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-77-oJaaDP8XOM-9QAwXjnSmzw-1; Thu, 23 Jul 2026 08:56:42 -0400 X-MC-Unique: oJaaDP8XOM-9QAwXjnSmzw-1 X-Mimecast-MFC-AGG-ID: oJaaDP8XOM-9QAwXjnSmzw_1784811400 Received: from mx-prod-int-08.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-08.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.111]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 4D21319540F0; Thu, 23 Jul 2026 12:56:40 +0000 (UTC) Received: from dmarchan.lan (unknown [10.44.48.202]) by mx-prod-int-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 3301A18002A0; Thu, 23 Jul 2026 12:56:36 +0000 (UTC) From: David Marchand To: dev@dpdk.org Cc: bruce.richardson@intel.com, Anatoly Burakov , Andrew Rybchenko , =?UTF-8?q?Morten=20Br=C3=B8rup?= , Reshma Pattan , Stephen Hemminger Subject: [PATCH v2 7/9] test: limit internal API usage Date: Thu, 23 Jul 2026 14:55:56 +0200 Message-ID: <20260723125559.3087579-8-david.marchand@redhat.com> In-Reply-To: <20260723125559.3087579-1-david.marchand@redhat.com> References: <20260717093006.229370-1-david.marchand@redhat.com> <20260723125559.3087579-1-david.marchand@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.111 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: FCJ2xdqM9HZdlVOWNhRAaWys7eVGrkiefaBSTCr1x5w_1784811400 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit content-type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Internal APIs may be used for validating the internals of some component, but in general, it should be discouraged. Let's not allow globally internal API in the unit tests. Signed-off-by: David Marchand --- app/test/meson.build | 3 --- app/test/process.c | 2 ++ app/test/test_devargs.c | 2 ++ app/test/test_external_mem.c | 2 ++ app/test/test_malloc.c | 2 ++ app/test/test_mempool.c | 2 ++ app/test/test_pdump.c | 3 +++ app/test/virtual_pmd.c | 2 ++ 8 files changed, 15 insertions(+), 3 deletions(-) diff --git a/app/test/meson.build b/app/test/meson.build index d8efa93050..e3dfa1d24c 100644 --- a/app/test/meson.build +++ b/app/test/meson.build @@ -286,9 +286,6 @@ foreach arg: extra_flags endif endforeach -# Enable using internal APIs in unit tests -cflags += '-DALLOW_INTERNAL_API' - # create a symlink in the app/test directory for the binary, for backward compatibility if not is_windows custom_target('test_symlink', diff --git a/app/test/process.c b/app/test/process.c index 2b6d7ae84b..92c8af702b 100644 --- a/app/test/process.c +++ b/app/test/process.c @@ -2,6 +2,8 @@ * Copyright(c) 2010-2014 Intel Corporation */ +#define ALLOW_INTERNAL_API + #include /* errno */ #include /* PATH_MAX */ #ifndef RTE_EXEC_ENV_WINDOWS diff --git a/app/test/test_devargs.c b/app/test/test_devargs.c index 0300a2c753..571f57500c 100644 --- a/app/test/test_devargs.c +++ b/app/test/test_devargs.c @@ -2,6 +2,8 @@ * Copyright (c) 2021 NVIDIA Corporation & Affiliates */ +#define ALLOW_INTERNAL_API + #include #include #include diff --git a/app/test/test_external_mem.c b/app/test/test_external_mem.c index 53300983ed..2c8d0d8e00 100644 --- a/app/test/test_external_mem.c +++ b/app/test/test_external_mem.c @@ -2,6 +2,8 @@ * Copyright(c) 2018 Intel Corporation */ +#define ALLOW_INTERNAL_API + #include "test.h" #include diff --git a/app/test/test_malloc.c b/app/test/test_malloc.c index da868c8091..151d1a3f2c 100644 --- a/app/test/test_malloc.c +++ b/app/test/test_malloc.c @@ -2,6 +2,8 @@ * Copyright(c) 2010-2019 Intel Corporation */ +#define ALLOW_INTERNAL_API + #include "test.h" #include diff --git a/app/test/test_mempool.c b/app/test/test_mempool.c index e54249ce61..f1e8adeef9 100644 --- a/app/test/test_mempool.c +++ b/app/test/test_mempool.c @@ -2,6 +2,8 @@ * Copyright(c) 2010-2014 Intel Corporation */ +#define ALLOW_INTERNAL_API + #include #include #include diff --git a/app/test/test_pdump.c b/app/test/test_pdump.c index 64f1bbf21c..f1c9bd9d7e 100644 --- a/app/test/test_pdump.c +++ b/app/test/test_pdump.c @@ -1,6 +1,9 @@ /* SPDX-License-Identifier: BSD-3-Clause * Copyright(c) 2018 Intel Corporation */ + +#define ALLOW_INTERNAL_API + #include #include #include diff --git a/app/test/virtual_pmd.c b/app/test/virtual_pmd.c index aa37e4073c..74bb3516ab 100644 --- a/app/test/virtual_pmd.c +++ b/app/test/virtual_pmd.c @@ -2,6 +2,8 @@ * Copyright(c) 2010-2014 Intel Corporation */ +#define ALLOW_INTERNAL_API + #include #include #include -- 2.54.0