From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 27CB546B81; Tue, 15 Jul 2025 18:18:42 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 08BBC406FF; Tue, 15 Jul 2025 18:17:26 +0200 (CEST) Received: from mail-qv1-f44.google.com (mail-qv1-f44.google.com [209.85.219.44]) by mails.dpdk.org (Postfix) with ESMTP id 6830F40A6F for ; Tue, 15 Jul 2025 18:17:23 +0200 (CEST) Received: by mail-qv1-f44.google.com with SMTP id 6a1803df08f44-6fb0eb0f0fbso478536d6.1 for ; Tue, 15 Jul 2025 09:17:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1752596243; x=1753201043; 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=2lAqmLj4pdQBJjtlxgWlHUnsk3u1djZ1R01G18ns7eg=; b=kQ7AJctwyrFDdjdMkYGXd5dYPYYnI09mEmqPvdsoyZIicbVaqjO1jKKi0cNq8i2Kkc WGqogjfg7gHnbv56/tyWno7RnIdbeHuz5mpQOfOHzm1jDC1p7fmKZ+ohPMUJt7fzp6DQ tuS6u2CUiEkh48Z4tYZkfWLccBh+K5kbcQbkIfdd0BltzkN7PLm9emALwgRAh4aAWoy+ eOSUTsjtjNxX9CsO7p9B+kEvrsZiN684kiDbFU208T+B/8iqzW/womf7KVKwbgydoXcg 4CUZnAMBdKIeMLUAX22G0bAcB1tDyOLxTTDnfbM55u6gvuo7Pug1da+F36l03Uz7G1Pq p+tw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1752596243; x=1753201043; 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=2lAqmLj4pdQBJjtlxgWlHUnsk3u1djZ1R01G18ns7eg=; b=ZFf2l2tGQYM2NcUh8x6NgCRyXocswOPrAxX7r3Exy+BhmqO4vOqN1RDCq0j8YaV5GD qUQKvyuv618rHafZ0D+jM3LK18q2SuEvrlNSgWTiGFLBkaO17faqLsafe+0pt56MaI5w GpjYWuwaxVjXDx8O62eX0Ypg3WTkDxAvUDW/OlXwFaiKCLISNO48u0iUtnS6jxXZElGe SadRQSBrMDcgWpceN+DHOlT8dq7YJak6sRMLSuORWoM/nV6wWSTYJyLI8K1H2MlAyTRz s1mZTzAzZ2Em5FNFVbrtGCuW0h1eQJ5mY5g34IwQaIr863WmvvwUAdmBF6E3xlDBJRPr 1zAg== X-Gm-Message-State: AOJu0YyX0UYk8ZMmuZ7vAdSos8YwWfqIZFwkAIN0rxDi5VIDE191hZc/ t+hNa2QvIhiXu+06uVEnJj/ugWcuhSvouWbYN5AJx7Lfi5wRaYHC4XZxVpP/Fi08aQ7Tis0SWqt rAAzp X-Gm-Gg: ASbGnct29QRtmtpaWZVrcCkvjnKvJqZYdpCIGpLFcxNIr1eKJUHzRT6Qvek9h2vr3xO txW4t/ulLIS00sfSFM2y5kGSuXP56uQY1EbAhbQrb/Yx+h/1vr2wUGhknJhTyO44MGdLCeoGZVl p30utffzfzXP9aUVbkZijOSzbOH9H5FygMNlGI/foQJRbmk7FY3uR8gnrm8+AwZrnGryd5Wn2xj Wo3GPws20T1iDqHSq8HpiApS9Jh0on/uUT5lDGw6zFeFspPfyfwsElVeEtDrCt0kTgNZzgviHOv YcDsXjKiSzxDSL9JnSAoEoSfT94a+MGmiytNXSnb0hsuLbGJbGux7MHdaFTzFnyLl6qYkFNNSYc PHWW5llk3RkjHUWTe60MdA8oSjF58QICsLQx1RSoNMEMdgES4amk8rE+KypAKLwtHpS5jo533 X-Google-Smtp-Source: AGHT+IH8elFJfvgfHlUePzxfUNspXMbdCAoIqeq0hXEKfCb5vN3+dTmUzTtgDV7tUklUyH38jeUvJQ== X-Received: by 2002:a05:6214:568c:b0:6fa:b468:caf4 with SMTP id 6a1803df08f44-704e1a8f543mr56540206d6.0.1752596242330; Tue, 15 Jul 2025 09:17:22 -0700 (PDT) Received: from hermes.lan (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-704979a8857sm59985106d6.9.2025.07.15.09.17.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 15 Jul 2025 09:17:22 -0700 (PDT) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Reshma Pattan , Aman Singh Subject: [PATCH v4 13/13] pdump: mark API and application as deprecated Date: Tue, 15 Jul 2025 09:15:51 -0700 Message-ID: <20250715161700.103209-14-stephen@networkplumber.org> X-Mailer: git-send-email 2.47.2 In-Reply-To: <20250715161700.103209-1-stephen@networkplumber.org> References: <20250411234927.114568-1-stephen@networkplumber.org> <20250715161700.103209-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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org When the pdump application is run print a warning. Add release note about deprecation and mark the API's as deprecated. Signed-off-by: Stephen Hemminger pdump build deprecated --- app/pdump/main.c | 3 +++ app/pdump/meson.build | 1 + app/test-pmd/meson.build | 1 + app/test/meson.build | 6 +++++- doc/guides/rel_notes/deprecation.rst | 3 +++ lib/pdump/rte_pdump.h | 12 +++++++++--- 6 files changed, 22 insertions(+), 4 deletions(-) diff --git a/app/pdump/main.c b/app/pdump/main.c index fa85859703..a4e0f4e5ae 100644 --- a/app/pdump/main.c +++ b/app/pdump/main.c @@ -986,6 +986,9 @@ main(int argc, char **argv) char mp_flag[] = "--proc-type=secondary"; char *argp[argc + 2]; + + fprintf(stderr, "%s: is deprecated use dpdk-dumpcap instead\n", argv[0]); + /* catch ctrl-c so we can cleanup on exit */ sigemptyset(&action.sa_mask); sigaction(SIGTERM, &action, NULL); diff --git a/app/pdump/meson.build b/app/pdump/meson.build index a4c7dd44e7..1fe034fcb8 100644 --- a/app/pdump/meson.build +++ b/app/pdump/meson.build @@ -11,3 +11,4 @@ sources = files('main.c') deps += ['ethdev', 'kvargs', 'pdump'] cflags += no_wvla_cflag +cflags += '-Wno-deprecated-declarations' diff --git a/app/test-pmd/meson.build b/app/test-pmd/meson.build index e67665028f..3ab8280ce7 100644 --- a/app/test-pmd/meson.build +++ b/app/test-pmd/meson.build @@ -60,6 +60,7 @@ if dpdk_conf.has('RTE_LIB_METRICS') endif if dpdk_conf.has('RTE_LIB_PDUMP') deps += 'pdump' + cflags += '-Wno-deprecated-declarations' endif if dpdk_conf.has('RTE_NET_BNXT') deps += 'net_bnxt' diff --git a/app/test/meson.build b/app/test/meson.build index 07bc00148d..45ece2179c 100644 --- a/app/test/meson.build +++ b/app/test/meson.build @@ -138,7 +138,6 @@ source_file_deps = { 'test_net_ip6.c': ['net'], 'test_pcapng.c': ['ethdev', 'net', 'pcapng', 'bus_vdev'], 'test_pdcp.c': ['eventdev', 'pdcp', 'net', 'timer', 'security'], - 'test_pdump.c': ['pdump'] + sample_packet_forward_deps, 'test_per_lcore.c': [], 'test_pflock.c': [], 'test_pie.c': ['sched'], @@ -217,6 +216,11 @@ source_file_ext_deps = { 'test_pcapng.c': ['pcap'], } +if dpdk_conf.has('RTE_LIB_PDUMP') + source_file_deps += { 'test_pdump.c': ['pdump'] + sample_packet_forward_deps, } + cflags += '-Wno-deprecated-declarations' +endif + def_lib = get_option('default_library') foreach f, f_deps : source_file_deps has_deps = true diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index e2d4125308..03a6b2712e 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -152,3 +152,6 @@ Deprecation Notices * bus/vmbus: Starting DPDK 25.11, all the vmbus API defined in ``drivers/bus/vmbus/rte_bus_vmbus.h`` will become internal to DPDK. Those API functions are used internally by DPDK core and netvsc PMD. + +* pdump: The pdump API and application have been deprecated and will + be removed in a later release. diff --git a/lib/pdump/rte_pdump.h b/lib/pdump/rte_pdump.h index 1e32d46097..bfea7c560d 100644 --- a/lib/pdump/rte_pdump.h +++ b/lib/pdump/rte_pdump.h @@ -39,6 +39,7 @@ enum { * @return * 0 on success, -1 on error */ +__rte_deprecated int rte_pdump_init(void); @@ -50,6 +51,7 @@ rte_pdump_init(void); * @return * 0 on success, -1 on error */ +__rte_deprecated int rte_pdump_uninit(void); @@ -75,7 +77,7 @@ rte_pdump_uninit(void); * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ - +__rte_deprecated int rte_pdump_enable(uint16_t port, uint16_t queue, uint32_t flags, struct rte_ring *ring, @@ -106,6 +108,7 @@ rte_pdump_enable(uint16_t port, uint16_t queue, uint32_t flags, * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ +__rte_deprecated int rte_pdump_enable_bpf(uint16_t port_id, uint16_t queue, uint32_t flags, uint32_t snaplen, @@ -129,7 +132,7 @@ rte_pdump_enable_bpf(uint16_t port_id, uint16_t queue, * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ - +__rte_deprecated int rte_pdump_disable(uint16_t port, uint16_t queue, uint32_t flags); @@ -156,7 +159,7 @@ rte_pdump_disable(uint16_t port, uint16_t queue, uint32_t flags); * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ - +__rte_deprecated int rte_pdump_enable_by_deviceid(char *device_id, uint16_t queue, uint32_t flags, @@ -189,6 +192,7 @@ rte_pdump_enable_by_deviceid(char *device_id, uint16_t queue, * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ +__rte_deprecated int rte_pdump_enable_bpf_by_deviceid(const char *device_id, uint16_t queue, uint32_t flags, uint32_t snaplen, @@ -215,6 +219,7 @@ rte_pdump_enable_bpf_by_deviceid(const char *device_id, uint16_t queue, * @return * 0 on success, -1 on error, rte_errno is set accordingly. */ +__rte_deprecated int rte_pdump_disable_by_deviceid(char *device_id, uint16_t queue, uint32_t flags); @@ -243,6 +248,7 @@ struct rte_pdump_stats { * @return * Zero if successful. -1 on error and rte_errno is set. */ +__rte_deprecated int rte_pdump_stats(uint16_t port_id, struct rte_pdump_stats *stats); -- 2.47.2