From: Ariel Otilibili <otilibil@eurecom.fr>
To: dev@dpdk.org
Cc: stable@dpdk.org, Thomas Monjalon <thomas@monjalon.net>,
David Marchand <david.marchand@redhat.com>,
Ariel Otilibili <otilibil@eurecom.fr>,
Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>,
Harman Kalra <hkalra@marvell.com>, Long Li <longli@microsoft.com>,
Wei Hu <weh@microsoft.com>, Chas Williams <chas3@att.com>,
"Min Hu (Connor)" <humin29@huawei.com>,
Dariusz Sosnowski <dsosnowski@nvidia.com>,
Viacheslav Ovsiienko <viacheslavo@nvidia.com>,
Bing Zhao <bingz@nvidia.com>, Ori Kam <orika@nvidia.com>,
Suanming Mou <suanmingm@nvidia.com>,
Matan Azrad <matan@nvidia.com>,
Potnuri Bharat Teja <bharat@chelsio.com>,
Andrew Boyer <andrew.boyer@amd.com>,
Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,
Cristian Dumitrescu <cristian.dumitrescu@intel.com>,
Stephen Hemminger <stephen@networkplumber.org>
Subject: [PATCH v4 00/11] devtools, lib, test, net, common: remove unused rte_bitmap_free() and plt_bitmap_free()
Date: Sun, 22 Dec 2024 13:49:20 +0100 [thread overview]
Message-ID: <20241222125725.1532157-1-otilibil@eurecom.fr> (raw)
Hello,
The first version was about the clearing of Coverity IDs 357712 & 357737; now this series is about the removal of rte_bitmap_free() and its alias plt_bitmap_free().
As of commit 07604f2644 ("maintainers: update for next-net tree"):
* rte_bitmap_free() returns an integer, and does nothing else
* all functions that call rte_bitmap_free() do not use this return value.
Details are in d5941e7269 ("devtools/cocci,lib/eal: remove unused rte_bitmap_free()").
Looking forward your feedback,
---
v4:
* fix style issues, https://patches.dpdk.org/project/dpdk/patch/20241222013328.1362225-3-otilibil@eurecom.fr/
* fix build failures, https://github.com/ovsrobot/dpdk/actions/runs/12450201352
v3:
* addressed feedback from Stephen Hemminger
* removed rte_bitmap_free() wherever it was used
v2:
* fix style issues.
Ariel Otilibili (11):
devtools/cocci,lib/eal: remove unused rte_bitmap_free()
app/test: remove unused rte_bitmap_free()
net/sfc: remove unused rte_bitmap_free()
crypto/ionic: remove unused rte_bitmap_free()
net/cxgbe: remove unused rte_bitmap_free()
net/mlx4: remove unused rte_bitmap_free()
common/mlx5: remove unused rte_bitmap_free()
net/bonding: remove unused rte_bitmap_free()
net/netvsc: remove unused rte_bitmap_free()
common/cnxk: remove unused plt_bitmap_free()
{common,net}/cnxk: remove unused plt_bitmap_free()
app/test/test_bitmap.c | 2 --
devtools/cocci/nullfree.cocci | 3 ---
drivers/common/cnxk/roc_mcs.c | 4 ----
drivers/common/cnxk/roc_nix_inl_dev.c | 2 --
drivers/common/cnxk/roc_nix_tm.c | 5 -----
drivers/common/cnxk/roc_npa.c | 4 ----
drivers/common/cnxk/roc_npc_aging.c | 1 -
drivers/common/cnxk/roc_platform.h | 1 -
drivers/common/mlx5/mlx5_common_mr.c | 1 -
drivers/crypto/ionic/ionic_crypto_main.c | 1 -
drivers/net/bonding/rte_eth_bond_pmd.c | 1 -
drivers/net/cnxk/cnxk_ethdev.c | 1 -
drivers/net/cxgbe/cxgbe_main.c | 1 -
drivers/net/mlx4/mlx4_mr.c | 1 -
drivers/net/netvsc/hn_rxtx.c | 1 -
drivers/net/sfc/sfc_sw_stats.c | 1 -
lib/eal/include/rte_bitmap.h | 19 -------------------
17 files changed, 49 deletions(-)
--
2.47.1
next reply other threads:[~2024-12-22 12:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-22 12:49 Ariel Otilibili [this message]
2024-12-22 12:49 ` [PATCH v4 01/11] devtools/cocci, lib/eal: remove unused rte_bitmap_free() Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 02/11] app/test: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 03/11] net/sfc: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 04/11] crypto/ionic: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 05/11] net/cxgbe: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 06/11] net/mlx4: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 07/11] common/mlx5: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 08/11] net/bonding: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 09/11] net/netvsc: " Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 10/11] common/cnxk: remove unused plt_bitmap_free() Ariel Otilibili
2024-12-22 12:49 ` [PATCH v4 11/11] {common,net}/cnxk: " Ariel Otilibili
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241222125725.1532157-1-otilibil@eurecom.fr \
--to=otilibil@eurecom.fr \
--cc=andrew.boyer@amd.com \
--cc=andrew.rybchenko@oktetlabs.ru \
--cc=bharat@chelsio.com \
--cc=bingz@nvidia.com \
--cc=chas3@att.com \
--cc=cristian.dumitrescu@intel.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=dsosnowski@nvidia.com \
--cc=hkalra@marvell.com \
--cc=humin29@huawei.com \
--cc=kirankumark@marvell.com \
--cc=longli@microsoft.com \
--cc=matan@nvidia.com \
--cc=ndabilpuram@marvell.com \
--cc=orika@nvidia.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
--cc=stable@dpdk.org \
--cc=stephen@networkplumber.org \
--cc=suanmingm@nvidia.com \
--cc=thomas@monjalon.net \
--cc=viacheslavo@nvidia.com \
--cc=weh@microsoft.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).