* [PATCH 0/3] announce bonding macro and function change
@ 2023-07-14 8:15 Chaoyong He
2023-07-14 8:15 ` [PATCH 1/3] doc: announce bonding macro change Chaoyong He
` (4 more replies)
0 siblings, 5 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-14 8:15 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, some of the macro and function in
DPDK will need to be renamed. Do this through deprecation process now
for 23.07.
Chaoyong He (2):
doc: announce bonding data change
doc: announce bonding function change
Long Wu (1):
doc: announce bonding macro change
app/test-pmd/testpmd.c | 6 +-
app/test/test_link_bonding.c | 100 +++++++++++-----------
app/test/test_link_bonding_mode4.c | 8 +-
app/test/test_link_bonding_rssconf.c | 8 +-
doc/guides/rel_notes/deprecation.rst | 19 ++++
drivers/net/bonding/bonding_testpmd.c | 4 +-
drivers/net/bonding/rte_eth_bond.h | 42 ++++++++-
drivers/net/bonding/rte_eth_bond_8023ad.c | 4 +-
drivers/net/bonding/rte_eth_bond_8023ad.h | 13 ++-
drivers/net/bonding/rte_eth_bond_api.c | 14 +--
drivers/net/bonding/rte_eth_bond_pmd.c | 12 +--
drivers/net/bonding/version.map | 10 +++
examples/bond/main.c | 6 +-
lib/ethdev/rte_ethdev.h | 5 +-
14 files changed, 162 insertions(+), 89 deletions(-)
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH 1/3] doc: announce bonding macro change
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
@ 2023-07-14 8:15 ` Chaoyong He
2023-07-17 15:13 ` Ferruh Yigit
2023-07-14 8:15 ` [PATCH 2/3] doc: announce bonding data change Chaoyong He
` (3 subsequent siblings)
4 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-14 8:15 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Long Wu, Chaoyong He
From: Long Wu <long.wu@corigine.com>
In order to support inclusive naming, some of the macro in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
---
app/test-pmd/testpmd.c | 2 +-
doc/guides/rel_notes/deprecation.rst | 4 ++++
drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
lib/ethdev/rte_ethdev.h | 5 +++--
4 files changed, 11 insertions(+), 6 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index c6ad9b18bf..938ca035d4 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -4248,7 +4248,7 @@ uint8_t port_is_bonding_slave(portid_t slave_pid)
slave_pid);
return 0;
}
- if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDED_SLAVE) || (port->slave_flag == 1))
+ if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDING_MEMBER) || (port->slave_flag == 1))
return 1;
return 0;
}
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index fb771a0305..c9477dd0da 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -161,3 +161,7 @@ Deprecation Notices
The new port library API (functions rte_swx_port_*)
will gradually transition from experimental to stable status
starting with DPDK 23.07 release.
+
+* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
+ DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
+ ``RTE_ETH_DEV_BONDING_MEMBER``.
diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
index 85d0528b7c..8b6cdce34a 100644
--- a/drivers/net/bonding/rte_eth_bond_api.c
+++ b/drivers/net/bonding/rte_eth_bond_api.c
@@ -472,7 +472,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
return -1;
slave_eth_dev = &rte_eth_devices[slave_port_id];
- if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDED_SLAVE) {
+ if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDING_MEMBER) {
RTE_BOND_LOG(ERR, "Slave device is already a slave of a bonded device");
return -1;
}
@@ -615,7 +615,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
}
/* Add slave details to bonded device */
- slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDED_SLAVE;
+ slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDING_MEMBER;
slave_vlan_filter_set(bonded_port_id, slave_port_id);
@@ -724,7 +724,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
slave_eth_dev = &rte_eth_devices[slave_port_id];
slave_remove(internals, slave_eth_dev);
- slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDED_SLAVE);
+ slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDING_MEMBER);
/* first slave in the active list will be the primary by default,
* otherwise use first device in list */
diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h
index 3d44979b44..04a2564f22 100644
--- a/lib/ethdev/rte_ethdev.h
+++ b/lib/ethdev/rte_ethdev.h
@@ -2036,8 +2036,9 @@ struct rte_eth_dev_owner {
#define RTE_ETH_DEV_FLOW_OPS_THREAD_SAFE RTE_BIT32(0)
/** Device supports link state interrupt */
#define RTE_ETH_DEV_INTR_LSC RTE_BIT32(1)
-/** Device is a bonded slave */
-#define RTE_ETH_DEV_BONDED_SLAVE RTE_BIT32(2)
+/** Device is a bonding member */
+#define RTE_ETH_DEV_BONDING_MEMBER RTE_BIT32(2)
+#define RTE_ETH_DEV_BONDED_SLAVE RTE_DEPRECATED(RTE_ETH_DEV_BONDED_SLAVE) RTE_ETH_DEV_BONDING_MEMBER
/** Device supports device removal interrupt */
#define RTE_ETH_DEV_INTR_RMV RTE_BIT32(3)
/** Device is port representor */
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH 2/3] doc: announce bonding data change
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
2023-07-14 8:15 ` [PATCH 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-14 8:15 ` Chaoyong He
2023-07-17 15:03 ` Ferruh Yigit
2023-07-14 8:15 ` [PATCH 3/3] doc: announce bonding function change Chaoyong He
` (2 subsequent siblings)
4 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-14 8:15 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, the data structure of bonding 8023
info need to be renamed. Do this through deprecation process now for
23.07.
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
doc/guides/rel_notes/deprecation.rst | 3 +++
drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +-
drivers/net/bonding/rte_eth_bond_8023ad.h | 4 ++--
drivers/net/bonding/rte_eth_bond_pmd.c | 4 ++--
4 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index c9477dd0da..5b16b66267 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -165,3 +165,6 @@ Deprecation Notices
* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
``RTE_ETH_DEV_BONDING_MEMBER``.
+ The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
+ deprecated in DPDK 23.07, and removed in DPDK 23.11. The relevant code can be
+ updated using ``struct rte_eth_bond_8023ad_member_info``.
diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c
index 4a266bb2ca..49f22ffab1 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.c
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.c
@@ -1518,7 +1518,7 @@ rte_eth_bond_8023ad_setup(uint16_t port_id,
int
rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
- struct rte_eth_bond_8023ad_slave_info *info)
+ struct rte_eth_bond_8023ad_member_info *info)
{
struct rte_eth_dev *bond_dev;
struct bond_dev_private *internals;
diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.h b/drivers/net/bonding/rte_eth_bond_8023ad.h
index 7ad8d6d00b..ab6d0182a9 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.h
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.h
@@ -141,7 +141,7 @@ struct rte_eth_bond_8023ad_conf {
enum rte_bond_8023ad_agg_selection agg_selection;
};
-struct rte_eth_bond_8023ad_slave_info {
+struct rte_eth_bond_8023ad_member_info {
enum rte_bond_8023ad_selection selected;
uint8_t actor_state;
struct port_params actor;
@@ -195,7 +195,7 @@ rte_eth_bond_8023ad_setup(uint16_t port_id,
*/
int
rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
- struct rte_eth_bond_8023ad_slave_info *conf);
+ struct rte_eth_bond_8023ad_member_info *conf);
#ifdef __cplusplus
}
diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c
index 73205f78f4..0a595d427c 100644
--- a/drivers/net/bonding/rte_eth_bond_pmd.c
+++ b/drivers/net/bonding/rte_eth_bond_pmd.c
@@ -3471,7 +3471,7 @@ dump_lacp_port_param(const struct port_params *params, FILE *f)
}
static void
-dump_lacp_slave(const struct rte_eth_bond_8023ad_slave_info *info, FILE *f)
+dump_lacp_slave(const struct rte_eth_bond_8023ad_member_info *info, FILE *f)
{
char a_state[256] = { 0 };
char p_state[256] = { 0 };
@@ -3520,7 +3520,7 @@ dump_lacp_slave(const struct rte_eth_bond_8023ad_slave_info *info, FILE *f)
static void
dump_lacp(uint16_t port_id, FILE *f)
{
- struct rte_eth_bond_8023ad_slave_info slave_info;
+ struct rte_eth_bond_8023ad_member_info slave_info;
struct rte_eth_bond_8023ad_conf port_conf;
uint16_t slaves[RTE_MAX_ETHPORTS];
int num_active_slaves;
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH 3/3] doc: announce bonding function change
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
2023-07-14 8:15 ` [PATCH 1/3] doc: announce bonding macro change Chaoyong He
2023-07-14 8:15 ` [PATCH 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-14 8:15 ` Chaoyong He
2023-07-17 15:13 ` Ferruh Yigit
2023-07-17 15:14 ` [PATCH 0/3] announce bonding macro and " Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
4 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-14 8:15 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He, Long Wu
In order to support inclusive naming, some of the function in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
app/test-pmd/testpmd.c | 4 +-
app/test/test_link_bonding.c | 100 +++++++++++-----------
app/test/test_link_bonding_mode4.c | 8 +-
| 8 +-
doc/guides/rel_notes/deprecation.rst | 12 +++
drivers/net/bonding/bonding_testpmd.c | 4 +-
drivers/net/bonding/rte_eth_bond.h | 42 ++++++++-
drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +-
drivers/net/bonding/rte_eth_bond_8023ad.h | 11 ++-
drivers/net/bonding/rte_eth_bond_api.c | 8 +-
drivers/net/bonding/rte_eth_bond_pmd.c | 8 +-
drivers/net/bonding/version.map | 10 +++
examples/bond/main.c | 6 +-
13 files changed, 144 insertions(+), 79 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 938ca035d4..2dd4180bf9 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -612,7 +612,7 @@ change_bonding_slave_port_status(portid_t bond_pid, bool is_stop)
portid_t slave_pid;
int i;
- num_slaves = rte_eth_bond_slaves_get(bond_pid, slave_pids,
+ num_slaves = rte_eth_bond_members_get(bond_pid, slave_pids,
RTE_MAX_ETHPORTS);
if (num_slaves < 0) {
fprintf(stderr, "Failed to get slave list for port = %u\n",
@@ -3519,7 +3519,7 @@ close_port(portid_t pid)
flush_port_owned_resources(pi);
#ifdef RTE_NET_BOND
if (port->bond_flag == 1)
- num_slaves = rte_eth_bond_slaves_get(pi,
+ num_slaves = rte_eth_bond_members_get(pi,
slave_pids, RTE_MAX_ETHPORTS);
#endif
rte_eth_dev_close(pi);
diff --git a/app/test/test_link_bonding.c b/app/test/test_link_bonding.c
index 5c496352c2..a94644a831 100644
--- a/app/test/test_link_bonding.c
+++ b/app/test/test_link_bonding.c
@@ -281,14 +281,14 @@ test_create_bonded_device(void)
test_params->bonding_mode), "Failed to set ethdev %d to mode %d",
test_params->bonded_port_id, test_params->bonding_mode);
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, 0,
"Number of slaves %d is great than expected %d.",
current_slave_count, 0);
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, 0,
@@ -335,19 +335,19 @@ test_add_slave_to_bonded_device(void)
uint16_t slaves[RTE_MAX_ETHPORTS];
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params->bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params->bonded_port_id,
test_params->slave_port_ids[test_params->bonded_slave_count]),
"Failed to add slave (%d) to bonded port (%d).",
test_params->slave_port_ids[test_params->bonded_slave_count],
test_params->bonded_port_id);
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, test_params->bonded_slave_count + 1,
"Number of slaves (%d) is greater than expected (%d).",
current_slave_count, test_params->bonded_slave_count + 1);
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, 0,
"Number of active slaves (%d) is not as expected (%d).\n",
@@ -362,12 +362,12 @@ static int
test_add_slave_to_invalid_bonded_device(void)
{
/* Invalid port ID */
- TEST_ASSERT_FAIL(rte_eth_bond_slave_add(test_params->bonded_port_id + 5,
+ TEST_ASSERT_FAIL(rte_eth_bond_member_add(test_params->bonded_port_id + 5,
test_params->slave_port_ids[test_params->bonded_slave_count]),
"Expected call to failed as invalid port specified.");
/* Non bonded device */
- TEST_ASSERT_FAIL(rte_eth_bond_slave_add(test_params->slave_port_ids[0],
+ TEST_ASSERT_FAIL(rte_eth_bond_member_add(test_params->slave_port_ids[0],
test_params->slave_port_ids[test_params->bonded_slave_count]),
"Expected call to failed as invalid port specified.");
@@ -382,14 +382,14 @@ test_remove_slave_from_bonded_device(void)
struct rte_ether_addr read_mac_addr, *mac_addr;
uint16_t slaves[RTE_MAX_ETHPORTS];
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_remove(test_params->bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_remove(test_params->bonded_port_id,
test_params->slave_port_ids[test_params->bonded_slave_count-1]),
"Failed to remove slave %d from bonded port (%d).",
test_params->slave_port_ids[test_params->bonded_slave_count-1],
test_params->bonded_port_id);
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, test_params->bonded_slave_count - 1,
@@ -424,13 +424,13 @@ static int
test_remove_slave_from_invalid_bonded_device(void)
{
/* Invalid port ID */
- TEST_ASSERT_FAIL(rte_eth_bond_slave_remove(
+ TEST_ASSERT_FAIL(rte_eth_bond_member_remove(
test_params->bonded_port_id + 5,
test_params->slave_port_ids[test_params->bonded_slave_count - 1]),
"Expected call to failed as invalid port specified.");
/* Non bonded device */
- TEST_ASSERT_FAIL(rte_eth_bond_slave_remove(
+ TEST_ASSERT_FAIL(rte_eth_bond_member_remove(
test_params->slave_port_ids[0],
test_params->slave_port_ids[test_params->bonded_slave_count - 1]),
"Expected call to failed as invalid port specified.");
@@ -449,7 +449,7 @@ test_add_already_bonded_slave_to_bonded_device(void)
test_add_slave_to_bonded_device();
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, 1,
"Number of slaves (%d) is not that expected (%d).",
@@ -461,7 +461,7 @@ test_add_already_bonded_slave_to_bonded_device(void)
rte_socket_id());
TEST_ASSERT(port_id >= 0, "Failed to create bonded device.");
- TEST_ASSERT(rte_eth_bond_slave_add(port_id,
+ TEST_ASSERT(rte_eth_bond_member_add(port_id,
test_params->slave_port_ids[test_params->bonded_slave_count - 1])
< 0,
"Added slave (%d) to bonded port (%d) unexpectedly.",
@@ -482,34 +482,34 @@ test_get_slaves_from_bonded_device(void)
"Failed to add slave to bonded device");
/* Invalid port id */
- current_slave_count = rte_eth_bond_slaves_get(INVALID_PORT_ID, slaves,
+ current_slave_count = rte_eth_bond_members_get(INVALID_PORT_ID, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid port id unexpectedly succeeded");
- current_slave_count = rte_eth_bond_active_slaves_get(INVALID_PORT_ID,
+ current_slave_count = rte_eth_bond_active_members_get(INVALID_PORT_ID,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid port id unexpectedly succeeded");
/* Invalid slaves pointer */
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
NULL, RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid slave array unexpectedly succeeded");
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->bonded_port_id, NULL, RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid slave array unexpectedly succeeded");
/* non bonded device*/
- current_slave_count = rte_eth_bond_slaves_get(
+ current_slave_count = rte_eth_bond_members_get(
test_params->slave_port_ids[0], NULL, RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid port id unexpectedly succeeded");
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->slave_port_ids[0], NULL, RTE_MAX_ETHPORTS);
TEST_ASSERT(current_slave_count < 0,
"Invalid port id unexpectedly succeeded");
@@ -573,13 +573,13 @@ test_start_bonded_device(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[test_params->bonded_slave_count-1], 1);
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, test_params->bonded_slave_count,
"Number of slaves (%d) is not expected value (%d).",
current_slave_count, test_params->bonded_slave_count);
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, test_params->bonded_slave_count,
"Number of active slaves (%d) is not expected value (%d).",
@@ -627,13 +627,13 @@ test_stop_bonded_device(void)
"Bonded port (%d) status (%d) is not expected value (%d).",
test_params->bonded_port_id, link_status.link_status, 0);
- current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
+ current_slave_count = rte_eth_bond_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, test_params->bonded_slave_count,
"Number of slaves (%d) is not expected value (%d).",
current_slave_count, test_params->bonded_slave_count);
- current_slave_count = rte_eth_bond_active_slaves_get(
+ current_slave_count = rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(current_slave_count, 0,
"Number of active slaves (%d) is not expected value (%d).",
@@ -956,13 +956,13 @@ test_set_bonded_port_initialization_mac_assignment(void)
* 2. Add slave ethdevs to bonded device
*/
for (i = 0; i < BONDED_INIT_MAC_ASSIGNMENT_SLAVE_COUNT; i++) {
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(bonded_port_id,
slave_port_ids[i]),
"Failed to add slave (%d) to bonded port (%d).",
slave_port_ids[i], bonded_port_id);
}
- slave_count = rte_eth_bond_slaves_get(bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(BONDED_INIT_MAC_ASSIGNMENT_SLAVE_COUNT, slave_count,
"Number of slaves (%d) is not as expected (%d)",
@@ -1080,13 +1080,13 @@ test_set_bonded_port_initialization_mac_assignment(void)
bonded_port_id);
for (i = 0; i < BONDED_INIT_MAC_ASSIGNMENT_SLAVE_COUNT; i++) {
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_remove(bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_remove(bonded_port_id,
slave_port_ids[i]),
"Failed to remove slave %d from bonded port (%d).",
slave_port_ids[i], bonded_port_id);
}
- slave_count = rte_eth_bond_slaves_get(bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 0,
@@ -1169,7 +1169,7 @@ test_adding_slave_after_bonded_device_started(void)
test_params->slave_port_ids[i], 1);
}
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params->bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params->bonded_port_id,
test_params->slave_port_ids[test_params->bonded_slave_count]),
"Failed to add slave to bonded port.\n");
@@ -1253,7 +1253,7 @@ test_status_interrupt(void)
RTE_ETH_EVENT_INTR_LSC, test_bonding_lsc_event_callback,
&test_params->bonded_port_id);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, TEST_STATUS_INTERRUPT_SLAVE_COUNT,
@@ -1281,7 +1281,7 @@ test_status_interrupt(void)
TEST_ASSERT(test_lsc_interrupt_count > 0,
"Did not receive link status change interrupt");
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 0,
@@ -1909,13 +1909,13 @@ test_roundrobin_verify_slave_link_status_change_behaviour(void)
"Failed to initialize bonded device with slaves");
/* Verify Current Slaves Count /Active Slave Count is */
- slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(test_params->bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, TEST_RR_LINK_STATUS_SLAVE_COUNT,
"Number of slaves (%d) is not as expected (%d).",
slave_count, TEST_RR_LINK_STATUS_SLAVE_COUNT);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, TEST_RR_LINK_STATUS_SLAVE_COUNT,
"Number of active slaves (%d) is not as expected (%d).",
@@ -1927,7 +1927,7 @@ test_roundrobin_verify_slave_link_status_change_behaviour(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[3], 0);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count,
TEST_RR_LINK_STATUS_EXPECTED_ACTIVE_SLAVE_COUNT,
@@ -2053,7 +2053,7 @@ test_roundrobin_verfiy_polling_slave_link_status_change(void)
}
/* Add slave to bonded device */
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params->bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params->bonded_port_id,
polling_test_slaves[i]),
"Failed to add slave %s(%d) to bonded device %d",
slave_name, polling_test_slaves[i],
@@ -2104,7 +2104,7 @@ test_roundrobin_verfiy_polling_slave_link_status_change(void)
for (i = 0; i < TEST_RR_POLLING_LINK_STATUS_SLAVE_COUNT; i++) {
TEST_ASSERT_SUCCESS(
- rte_eth_bond_slave_remove(test_params->bonded_port_id,
+ rte_eth_bond_member_remove(test_params->bonded_port_id,
polling_test_slaves[i]),
"Failed to remove slave %d from bonded port (%d)",
polling_test_slaves[i], test_params->bonded_port_id);
@@ -2509,13 +2509,13 @@ test_activebackup_verify_slave_link_status_change_failover(void)
"Failed to initialize bonded device with slaves");
/* Verify Current Slaves Count /Active Slave Count is */
- slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(test_params->bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 4,
"Number of slaves (%d) is not as expected (%d).",
slave_count, 4);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 4,
"Number of active slaves (%d) is not as expected (%d).",
@@ -2531,7 +2531,7 @@ test_activebackup_verify_slave_link_status_change_failover(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[3], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS), 2,
"Number of active slaves (%d) is not as expected (%d).",
slave_count, 2);
@@ -2547,7 +2547,7 @@ test_activebackup_verify_slave_link_status_change_failover(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[0], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS),
3,
"Number of active slaves (%d) is not as expected (%d).",
@@ -3441,13 +3441,13 @@ test_balance_verify_slave_link_status_change_behaviour(void)
/* Verify Current Slaves Count /Active Slave Count is */
- slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(test_params->bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, TEST_BALANCE_LINK_STATUS_SLAVE_COUNT,
"Number of slaves (%d) is not as expected (%d).",
slave_count, TEST_BALANCE_LINK_STATUS_SLAVE_COUNT);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, TEST_BALANCE_LINK_STATUS_SLAVE_COUNT,
"Number of active slaves (%d) is not as expected (%d).",
@@ -3459,7 +3459,7 @@ test_balance_verify_slave_link_status_change_behaviour(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[3], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS), 2,
"Number of active slaves (%d) is not as expected (%d).",
slave_count, 2);
@@ -3508,7 +3508,7 @@ test_balance_verify_slave_link_status_change_behaviour(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[2], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS), 1,
"Number of active slaves (%d) is not as expected (%d).",
slave_count, 1);
@@ -4023,13 +4023,13 @@ test_broadcast_verify_slave_link_status_change_behaviour(void)
1), "Failed to initialise bonded device");
/* Verify Current Slaves Count /Active Slave Count is */
- slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(test_params->bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 4,
"Number of slaves (%d) is not as expected (%d).",
slave_count, 4);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 4,
"Number of active slaves (%d) is not as expected (%d).",
@@ -4041,7 +4041,7 @@ test_broadcast_verify_slave_link_status_change_behaviour(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[3], 0);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 2,
"Number of active slaves (%d) is not as expected (%d).",
@@ -4581,13 +4581,13 @@ test_tlb_verify_slave_link_status_change_failover(void)
"Failed to initialize bonded device with slaves");
/* Verify Current Slaves Count /Active Slave Count is */
- slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id, slaves,
+ slave_count = rte_eth_bond_members_get(test_params->bonded_port_id, slaves,
RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, 4,
"Number of slaves (%d) is not as expected (%d).\n",
slave_count, 4);
- slave_count = rte_eth_bond_active_slaves_get(test_params->bonded_port_id,
+ slave_count = rte_eth_bond_active_members_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
TEST_ASSERT_EQUAL(slave_count, (int)4,
"Number of slaves (%d) is not as expected (%d).\n",
@@ -4603,7 +4603,7 @@ test_tlb_verify_slave_link_status_change_failover(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[3], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS), 2,
"Number of active slaves (%d) is not as expected (%d).",
slave_count, 2);
@@ -4619,7 +4619,7 @@ test_tlb_verify_slave_link_status_change_failover(void)
virtual_ethdev_simulate_link_status_interrupt(
test_params->slave_port_ids[0], 0);
- TEST_ASSERT_EQUAL(rte_eth_bond_active_slaves_get(
+ TEST_ASSERT_EQUAL(rte_eth_bond_active_members_get(
test_params->bonded_port_id, slaves, RTE_MAX_ETHPORTS), 3,
"Number of active slaves (%d) is not as expected (%d).",
slave_count, 3);
diff --git a/app/test/test_link_bonding_mode4.c b/app/test/test_link_bonding_mode4.c
index 21c512c94b..98d7cf9f54 100644
--- a/app/test/test_link_bonding_mode4.c
+++ b/app/test/test_link_bonding_mode4.c
@@ -238,7 +238,7 @@ add_slave(struct slave_conf *slave, uint8_t start)
TEST_ASSERT_SUCCESS(rte_eth_dev_mac_addr_add(slave->port_id, &addr, 0),
"Failed to set slave MAC address");
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params.bonded_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params.bonded_port_id,
slave->port_id),
"Failed to add slave (idx=%u, id=%u) to bonding (id=%u)",
(uint8_t)(slave - test_params.slave_ports), slave->port_id,
@@ -279,7 +279,7 @@ remove_slave(struct slave_conf *slave)
"Slave %u tx queue not empty while removing from bonding.",
slave->port_id);
- TEST_ASSERT_EQUAL(rte_eth_bond_slave_remove(test_params.bonded_port_id,
+ TEST_ASSERT_EQUAL(rte_eth_bond_member_remove(test_params.bonded_port_id,
slave->port_id), 0,
"Failed to remove slave (idx=%u, id=%u) from bonding (id=%u)",
(uint8_t)slave_idx, slave->port_id,
@@ -359,7 +359,7 @@ remove_slaves_and_stop_bonded_device(void)
FOR_EACH_SLAVE(i, slave)
remove_slave(slave);
- retval = rte_eth_bond_slaves_get(test_params.bonded_port_id, slaves,
+ retval = rte_eth_bond_members_get(test_params.bonded_port_id, slaves,
RTE_DIM(slaves));
TEST_ASSERT_EQUAL(retval, 0,
@@ -1540,7 +1540,7 @@ check_environment(void)
break;
}
- slaves_count = rte_eth_bond_slaves_get(test_params.bonded_port_id,
+ slaves_count = rte_eth_bond_members_get(test_params.bonded_port_id,
slaves, RTE_DIM(slaves));
if (slaves_count != 0)
--git a/app/test/test_link_bonding_rssconf.c b/app/test/test_link_bonding_rssconf.c
index 464fb2dbd0..5d1b73f719 100644
--- a/app/test/test_link_bonding_rssconf.c
+++ b/app/test/test_link_bonding_rssconf.c
@@ -162,7 +162,7 @@ remove_slaves(void)
FOR_EACH_PORT(n, port) {
port = &test_params.slave_ports[n];
if (port->is_slave) {
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_remove(
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_remove(
test_params.bond_port_id, port->port_id),
"Cannot remove slave %d from bonding", port->port_id);
port->is_slave = 0;
@@ -193,7 +193,7 @@ bond_slaves(void)
FOR_EACH_PORT(n, port) {
port = &test_params.slave_ports[n];
if (!port->is_slave) {
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params.bond_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params.bond_port_id,
port->port_id), "Cannot attach slave %d to the bonding",
port->port_id);
port->is_slave = 1;
@@ -289,7 +289,7 @@ slave_remove_and_add(void)
struct slave_conf *port = &(test_params.slave_ports[0]);
/* 1. Remove first slave from bonding */
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_remove(test_params.bond_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_remove(test_params.bond_port_id,
port->port_id), "Cannot remove slave #d from bonding");
/* 2. Change removed (ex-)slave and bonding configuration to different
@@ -305,7 +305,7 @@ slave_remove_and_add(void)
"Removed slave didn't should be synchronized with bonding port");
/* 3. Add (ex-)slave and check if configuration changed*/
- TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(test_params.bond_port_id,
+ TEST_ASSERT_SUCCESS(rte_eth_bond_member_add(test_params.bond_port_id,
port->port_id), "Cannot add slave");
bond_reta_fetch();
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 5b16b66267..2f504e4c78 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -168,3 +168,15 @@ Deprecation Notices
The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
deprecated in DPDK 23.07, and removed in DPDK 23.11. The relevant code can be
updated using ``struct rte_eth_bond_8023ad_member_info``.
+ The following functions will be deprecated in DPDK 23.07, and removed in
+ DPDK 23.11. The old functions:
+ ``rte_eth_bond_8023ad_slave_info``,
+ ``rte_eth_bond_active_slaves_get``,
+ ``rte_eth_bond_slave_add``,
+ ``rte_eth_bond_slave_remove``, and
+ ``rte_eth_bond_slaves_get`` will be replaced by:
+ ``rte_eth_bond_8023ad_member_info``,
+ ``rte_eth_bond_active_members_get``,
+ ``rte_eth_bond_member_add``,
+ ``rte_eth_bond_member_remove``,
+ and ``rte_eth_bond_members_get``.
diff --git a/drivers/net/bonding/bonding_testpmd.c b/drivers/net/bonding/bonding_testpmd.c
index b3c12cada0..6049d5c865 100644
--- a/drivers/net/bonding/bonding_testpmd.c
+++ b/drivers/net/bonding/bonding_testpmd.c
@@ -347,7 +347,7 @@ static void cmd_add_bonding_slave_parsed(void *parsed_result,
portid_t slave_port_id = res->slave_id;
/* add the slave for a bonded device. */
- if (rte_eth_bond_slave_add(master_port_id, slave_port_id) != 0) {
+ if (rte_eth_bond_member_add(master_port_id, slave_port_id) != 0) {
fprintf(stderr,
"\t Failed to add slave %d to master port = %d.\n",
slave_port_id, master_port_id);
@@ -406,7 +406,7 @@ static void cmd_remove_bonding_slave_parsed(void *parsed_result,
portid_t slave_port_id = res->slave_id;
/* remove the slave from a bonded device. */
- if (rte_eth_bond_slave_remove(master_port_id, slave_port_id) != 0) {
+ if (rte_eth_bond_member_remove(master_port_id, slave_port_id) != 0) {
fprintf(stderr,
"\t Failed to remove slave %d from master port = %d.\n",
slave_port_id, master_port_id);
diff --git a/drivers/net/bonding/rte_eth_bond.h b/drivers/net/bonding/rte_eth_bond.h
index 874aa91a5f..6783c5b342 100644
--- a/drivers/net/bonding/rte_eth_bond.h
+++ b/drivers/net/bonding/rte_eth_bond.h
@@ -121,8 +121,16 @@ rte_eth_bond_free(const char *name);
* @return
* 0 on success, negative value otherwise
*/
+__rte_experimental
int
-rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id);
+rte_eth_bond_member_add(uint16_t bonded_port_id, uint16_t member_port_id);
+
+__rte_deprecated
+static inline int
+rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id)
+{
+ return rte_eth_bond_member_add(bonded_port_id, slave_port_id);
+}
/**
* Remove a slave rte_eth_dev device from the bonded device
@@ -133,8 +141,16 @@ rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id);
* @return
* 0 on success, negative value otherwise
*/
+__rte_experimental
int
-rte_eth_bond_slave_remove(uint16_t bonded_port_id, uint16_t slave_port_id);
+rte_eth_bond_member_remove(uint16_t bonded_port_id, uint16_t member_port_id);
+
+__rte_deprecated
+static inline int
+rte_eth_bond_slave_remove(uint16_t bonded_port_id, uint16_t slave_port_id)
+{
+ return rte_eth_bond_member_remove(bonded_port_id, slave_port_id);
+}
/**
* Set link bonding mode of bonded device
@@ -193,9 +209,18 @@ rte_eth_bond_primary_get(uint16_t bonded_port_id);
* Number of slaves associated with bonded device on success,
* negative value otherwise
*/
+__rte_experimental
int
+rte_eth_bond_members_get(uint16_t bonded_port_id, uint16_t members[],
+ uint16_t len);
+
+__rte_deprecated
+static inline int
rte_eth_bond_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
- uint16_t len);
+ uint16_t len)
+{
+ return rte_eth_bond_members_get(bonded_port_id, slaves, len);
+}
/**
* Populate an array with list of the active slaves port id's of the bonded
@@ -209,9 +234,18 @@ rte_eth_bond_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
* Number of active slaves associated with bonded device on success,
* negative value otherwise
*/
+__rte_experimental
int
+rte_eth_bond_active_members_get(uint16_t bonded_port_id, uint16_t members[],
+ uint16_t len);
+
+__rte_deprecated
+static inline int
rte_eth_bond_active_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
- uint16_t len);
+ uint16_t len)
+{
+ return rte_eth_bond_active_members_get(bonded_port_id, slaves, len);
+}
/**
* Set explicit MAC address to use on bonded device and it's slaves.
diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c
index 49f22ffab1..50618d9a9a 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.c
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.c
@@ -1517,7 +1517,7 @@ rte_eth_bond_8023ad_setup(uint16_t port_id,
int
-rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
+rte_eth_bond_8023ad_member_info(uint16_t port_id, uint16_t slave_id,
struct rte_eth_bond_8023ad_member_info *info)
{
struct rte_eth_dev *bond_dev;
diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.h b/drivers/net/bonding/rte_eth_bond_8023ad.h
index ab6d0182a9..54ef2db6c0 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.h
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.h
@@ -193,10 +193,19 @@ rte_eth_bond_8023ad_setup(uint16_t port_id,
* -EINVAL if conf is NULL or slave id is invalid (not a slave of given
* bonded device or is not inactive).
*/
+__rte_experimental
int
-rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
+rte_eth_bond_8023ad_member_info(uint16_t port_id, uint16_t member_id,
struct rte_eth_bond_8023ad_member_info *conf);
+__rte_deprecated
+static inline int
+rte_eth_bond_8023ad_slave_info(uint16_t port_id, uint16_t slave_id,
+ struct rte_eth_bond_8023ad_member_info *conf)
+{
+ return rte_eth_bond_8023ad_member_info(port_id, slave_id, conf);
+}
+
#ifdef __cplusplus
}
#endif
diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
index 8b6cdce34a..998059cb29 100644
--- a/drivers/net/bonding/rte_eth_bond_api.c
+++ b/drivers/net/bonding/rte_eth_bond_api.c
@@ -624,7 +624,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
}
int
-rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id)
+rte_eth_bond_member_add(uint16_t bonded_port_id, uint16_t slave_port_id)
{
struct rte_eth_dev *bonded_eth_dev;
struct bond_dev_private *internals;
@@ -760,7 +760,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
}
int
-rte_eth_bond_slave_remove(uint16_t bonded_port_id, uint16_t slave_port_id)
+rte_eth_bond_member_remove(uint16_t bonded_port_id, uint16_t slave_port_id)
{
struct rte_eth_dev *bonded_eth_dev;
struct bond_dev_private *internals;
@@ -849,7 +849,7 @@ rte_eth_bond_primary_get(uint16_t bonded_port_id)
}
int
-rte_eth_bond_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
+rte_eth_bond_members_get(uint16_t bonded_port_id, uint16_t slaves[],
uint16_t len)
{
struct bond_dev_private *internals;
@@ -873,7 +873,7 @@ rte_eth_bond_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
}
int
-rte_eth_bond_active_slaves_get(uint16_t bonded_port_id, uint16_t slaves[],
+rte_eth_bond_active_members_get(uint16_t bonded_port_id, uint16_t slaves[],
uint16_t len)
{
struct bond_dev_private *internals;
diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c
index 0a595d427c..92fba4e6c2 100644
--- a/drivers/net/bonding/rte_eth_bond_pmd.c
+++ b/drivers/net/bonding/rte_eth_bond_pmd.c
@@ -2203,7 +2203,7 @@ bond_ethdev_cfg_cleanup(struct rte_eth_dev *dev, bool remove)
continue;
}
- if (rte_eth_bond_slave_remove(bond_port_id, port_id) != 0) {
+ if (rte_eth_bond_member_remove(bond_port_id, port_id) != 0) {
RTE_BOND_LOG(ERR,
"Failed to remove port %d from bonded device %s",
port_id, dev->device->name);
@@ -3528,7 +3528,7 @@ dump_lacp(uint16_t port_id, FILE *f)
fprintf(f, " - Lacp info:\n");
- num_active_slaves = rte_eth_bond_active_slaves_get(port_id, slaves,
+ num_active_slaves = rte_eth_bond_active_members_get(port_id, slaves,
RTE_MAX_ETHPORTS);
if (num_active_slaves < 0) {
fprintf(f, "\tFailed to get active slave list for port %u\n",
@@ -3546,7 +3546,7 @@ dump_lacp(uint16_t port_id, FILE *f)
dump_lacp_conf(&port_conf, f);
for (i = 0; i < num_active_slaves; i++) {
- ret = rte_eth_bond_8023ad_slave_info(port_id, slaves[i],
+ ret = rte_eth_bond_8023ad_member_info(port_id, slaves[i],
&slave_info);
if (ret) {
fprintf(f, "\tGet slave device %u 8023ad info failed\n",
@@ -4075,7 +4075,7 @@ bond_ethdev_configure(struct rte_eth_dev *dev)
}
for (i = 0; i < slave_ports.slave_count; i++) {
- if (rte_eth_bond_slave_add(port_id, slave_ports.slaves[i]) != 0) {
+ if (rte_eth_bond_member_add(port_id, slave_ports.slaves[i]) != 0) {
RTE_BOND_LOG(ERR,
"Failed to add port %d as slave to bonded device %s",
slave_ports.slaves[i], name);
diff --git a/drivers/net/bonding/version.map b/drivers/net/bonding/version.map
index 9333923b4e..f3f0a776ff 100644
--- a/drivers/net/bonding/version.map
+++ b/drivers/net/bonding/version.map
@@ -31,3 +31,13 @@ DPDK_23 {
local: *;
};
+
+EXPERIMENTAL {
+ # added in 23.07
+ global:
+ rte_eth_bond_8023ad_member_info;
+ rte_eth_bond_active_members_get;
+ rte_eth_bond_member_add;
+ rte_eth_bond_member_remove;
+ rte_eth_bond_members_get;
+};
diff --git a/examples/bond/main.c b/examples/bond/main.c
index 9b076bb39f..f5514e5361 100644
--- a/examples/bond/main.c
+++ b/examples/bond/main.c
@@ -253,7 +253,7 @@ bond_port_init(struct rte_mempool *mbuf_pool)
"failed (res=%d)\n", BOND_PORT, retval);
for (i = 0; i < slaves_count; i++) {
- if (rte_eth_bond_slave_add(BOND_PORT, slaves[i]) == -1)
+ if (rte_eth_bond_member_add(BOND_PORT, slaves[i]) == -1)
rte_exit(-1, "Oooops! adding slave (%u) to bond (%u) failed!\n",
slaves[i], BOND_PORT);
@@ -286,7 +286,7 @@ bond_port_init(struct rte_mempool *mbuf_pool)
printf("Waiting for slaves to become active...");
while (wait_counter) {
uint16_t act_slaves[16] = {0};
- if (rte_eth_bond_active_slaves_get(BOND_PORT, act_slaves, 16) ==
+ if (rte_eth_bond_active_members_get(BOND_PORT, act_slaves, 16) ==
slaves_count) {
printf("\n");
break;
@@ -765,7 +765,7 @@ static void cmd_show_parsed(__rte_unused void *parsed_result,
cmdline_printf(cl,
"Active_slaves:%d "
"packets received:Tot:%d Arp:%d IPv4:%d\n",
- rte_eth_bond_active_slaves_get(BOND_PORT, slaves, len),
+ rte_eth_bond_active_members_get(BOND_PORT, slaves, len),
global_flag_stru_p->port_packets[0],
global_flag_stru_p->port_packets[1],
global_flag_stru_p->port_packets[2]);
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH 2/3] doc: announce bonding data change
2023-07-14 8:15 ` [PATCH 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-17 15:03 ` Ferruh Yigit
2023-07-18 1:13 ` Chaoyong He
0 siblings, 1 reply; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-17 15:03 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund
On 7/14/2023 9:15 AM, Chaoyong He wrote:
> In order to support inclusive naming, the data structure of bonding 8023
> info need to be renamed. Do this through deprecation process now for
> 23.07.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 3 +++
> drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +-
> drivers/net/bonding/rte_eth_bond_8023ad.h | 4 ++--
> drivers/net/bonding/rte_eth_bond_pmd.c | 4 ++--
> 4 files changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index c9477dd0da..5b16b66267 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -165,3 +165,6 @@ Deprecation Notices
> * bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
> DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
> ``RTE_ETH_DEV_BONDING_MEMBER``.
> + The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> + deprecated in DPDK 23.07, and removed in DPDK 23.11. The relevant code can be
> + updated using ``struct rte_eth_bond_8023ad_member_info``.
<...>
> --- a/drivers/net/bonding/rte_eth_bond_8023ad.h
> +++ b/drivers/net/bonding/rte_eth_bond_8023ad.h
> @@ -141,7 +141,7 @@ struct rte_eth_bond_8023ad_conf {
> enum rte_bond_8023ad_agg_selection agg_selection;
> };
>
> -struct rte_eth_bond_8023ad_slave_info {
> +struct rte_eth_bond_8023ad_member_info {
> enum rte_bond_8023ad_selection selected;
> uint8_t actor_state;
> struct port_params actor;
There is no good way to deprecate struct names.
For macros it is possible to keep both old and new ones, old ones will
give warning but still continue to work, so that is just a heads up for
the user.
But above is rename and will break the application, forcing user to
update their code.
And if we will force user to update their code, this should be done on
ABI break release, v23.11.
That is why I suggest just keep the deprecation notice update, saying
that struct will be renamed in v23.11, without mentioning from
deprecating struct in this release etc..
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH 3/3] doc: announce bonding function change
2023-07-14 8:15 ` [PATCH 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-17 15:13 ` Ferruh Yigit
2023-07-18 1:15 ` Chaoyong He
0 siblings, 1 reply; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-17 15:13 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/14/2023 9:15 AM, Chaoyong He wrote:
> In order to support inclusive naming, some of the function in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
<...>
> --- a/drivers/net/bonding/rte_eth_bond.h
> +++ b/drivers/net/bonding/rte_eth_bond.h
> @@ -121,8 +121,16 @@ rte_eth_bond_free(const char *name);
> * @return
> * 0 on success, negative value otherwise
> */
> +__rte_experimental
> int
> -rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id);
> +rte_eth_bond_member_add(uint16_t bonded_port_id, uint16_t member_port_id);
> +
> +__rte_deprecated
> +static inline int
> +rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id)
> +{
> + return rte_eth_bond_member_add(bonded_port_id, slave_port_id);
> +}
>
This will make old symbols disappear from shared library, since they are
static inline functions and not object in the shared library.
And this will break the ABI, you can see this from the CI test:
https://mails.dpdk.org/archives/test-report/2023-July/427987.html
One option is to add old functions to the .c file, and keep old function
declarations in the header file, with '__rte_deprecated' attribute.
But I think it is simpler/safer to rename in one go in v23.11 release,
so this patch can update only deprecation notice to list functions that
will be renamed in v23.11 release.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH 1/3] doc: announce bonding macro change
2023-07-14 8:15 ` [PATCH 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-17 15:13 ` Ferruh Yigit
2023-07-18 1:17 ` Chaoyong He
0 siblings, 1 reply; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-17 15:13 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/14/2023 9:15 AM, Chaoyong He wrote:
> From: Long Wu <long.wu@corigine.com>
>
> In order to support inclusive naming, some of the macro in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
<...>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index fb771a0305..c9477dd0da 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -161,3 +161,7 @@ Deprecation Notices
> The new port library API (functions rte_swx_port_*)
> will gradually transition from experimental to stable status
> starting with DPDK 23.07 release.
> +
> +* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
> + DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
> + ``RTE_ETH_DEV_BONDING_MEMBER``.
It can be better to move this below the ethdev entries, to group them, I
can do this while merging.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH 0/3] announce bonding macro and function change
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
` (2 preceding siblings ...)
2023-07-14 8:15 ` [PATCH 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-17 15:14 ` Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
4 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-17 15:14 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund
On 7/14/2023 9:15 AM, Chaoyong He wrote:
> In order to support inclusive naming, some of the macro and function in
> DPDK will need to be renamed. Do this through deprecation process now
> for 23.07.
>
> Chaoyong He (2):
> doc: announce bonding data change
> doc: announce bonding function change
>
> Long Wu (1):
> doc: announce bonding macro change
Hi Chaoyong,
OK to deprecate macro in this release (v23.07) and remove them in LTS
(v23.11) release,
but for struct and function names I think simpler to rename them
directly in LTS release without deprecating them in this release, more
details in the individual patches.
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH 2/3] doc: announce bonding data change
2023-07-17 15:03 ` Ferruh Yigit
@ 2023-07-18 1:13 ` Chaoyong He
0 siblings, 0 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:13 UTC (permalink / raw)
To: Ferruh Yigit, dev; +Cc: oss-drivers, Niklas Soderlund
> On 7/14/2023 9:15 AM, Chaoyong He wrote:
> > In order to support inclusive naming, the data structure of bonding
> > 8023 info need to be renamed. Do this through deprecation process now
> > for 23.07.
> >
> > Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> > ---
> > doc/guides/rel_notes/deprecation.rst | 3 +++
> > drivers/net/bonding/rte_eth_bond_8023ad.c | 2 +-
> > drivers/net/bonding/rte_eth_bond_8023ad.h | 4 ++--
> > drivers/net/bonding/rte_eth_bond_pmd.c | 4 ++--
> > 4 files changed, 8 insertions(+), 5 deletions(-)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> > b/doc/guides/rel_notes/deprecation.rst
> > index c9477dd0da..5b16b66267 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -165,3 +165,6 @@ Deprecation Notices
> > * bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated
> in
> > DPDK 23.07, and removed in DPDK 23.11. The relevant code can be
> updated using
> > ``RTE_ETH_DEV_BONDING_MEMBER``.
> > + The data structure ``struct rte_eth_bond_8023ad_slave_info`` will
> > + be deprecated in DPDK 23.07, and removed in DPDK 23.11. The
> > + relevant code can be updated using ``struct
> rte_eth_bond_8023ad_member_info``.
>
> <...>
>
> > --- a/drivers/net/bonding/rte_eth_bond_8023ad.h
> > +++ b/drivers/net/bonding/rte_eth_bond_8023ad.h
> > @@ -141,7 +141,7 @@ struct rte_eth_bond_8023ad_conf {
> > enum rte_bond_8023ad_agg_selection agg_selection; };
> >
> > -struct rte_eth_bond_8023ad_slave_info {
> > +struct rte_eth_bond_8023ad_member_info {
> > enum rte_bond_8023ad_selection selected;
> > uint8_t actor_state;
> > struct port_params actor;
>
> There is no good way to deprecate struct names.
>
> For macros it is possible to keep both old and new ones, old ones will give
> warning but still continue to work, so that is just a heads up for the user.
> But above is rename and will break the application, forcing user to update their
> code.
> And if we will force user to update their code, this should be done on ABI
> break release, v23.11.
>
> That is why I suggest just keep the deprecation notice update, saying that
> struct will be renamed in v23.11, without mentioning from deprecating struct
> in this release etc..
Got it. Thanks for point it out, I will revise in the v2 patch.
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH 3/3] doc: announce bonding function change
2023-07-17 15:13 ` Ferruh Yigit
@ 2023-07-18 1:15 ` Chaoyong He
0 siblings, 0 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:15 UTC (permalink / raw)
To: Ferruh Yigit, dev; +Cc: oss-drivers, Niklas Soderlund, Long Wu
> On 7/14/2023 9:15 AM, Chaoyong He wrote:
> > In order to support inclusive naming, some of the function in DPDK
> > will need to be renamed. Do this through deprecation process now for
> 23.07.
> >
> > Signed-off-by: Long Wu <long.wu@corigine.com>
> > Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
>
> <...>
>
> > --- a/drivers/net/bonding/rte_eth_bond.h
> > +++ b/drivers/net/bonding/rte_eth_bond.h
> > @@ -121,8 +121,16 @@ rte_eth_bond_free(const char *name);
> > * @return
> > * 0 on success, negative value otherwise
> > */
> > +__rte_experimental
> > int
> > -rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t
> > slave_port_id);
> > +rte_eth_bond_member_add(uint16_t bonded_port_id, uint16_t
> > +member_port_id);
> > +
> > +__rte_deprecated
> > +static inline int
> > +rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t
> > +slave_port_id) {
> > + return rte_eth_bond_member_add(bonded_port_id, slave_port_id); }
> >
>
> This will make old symbols disappear from shared library, since they are static
> inline functions and not object in the shared library.
> And this will break the ABI, you can see this from the CI test:
> https://mails.dpdk.org/archives/test-report/2023-July/427987.html
>
> One option is to add old functions to the .c file, and keep old function
> declarations in the header file, with '__rte_deprecated' attribute.
>
> But I think it is simpler/safer to rename in one go in v23.11 release, so this
> patch can update only deprecation notice to list functions that will be renamed
> in v23.11 release.
Okay. I will revise as your advice in the v2 patch, thanks.
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH 1/3] doc: announce bonding macro change
2023-07-17 15:13 ` Ferruh Yigit
@ 2023-07-18 1:17 ` Chaoyong He
0 siblings, 0 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:17 UTC (permalink / raw)
To: Ferruh Yigit, dev; +Cc: oss-drivers, Niklas Soderlund, Long Wu
> On 7/14/2023 9:15 AM, Chaoyong He wrote:
> > From: Long Wu <long.wu@corigine.com>
> >
> > In order to support inclusive naming, some of the macro in DPDK will
> > need to be renamed. Do this through deprecation process now for 23.07.
> >
> > Signed-off-by: Long Wu <long.wu@corigine.com>
> > Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
> >
>
> Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
>
> <...>
>
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> > b/doc/guides/rel_notes/deprecation.rst
> > index fb771a0305..c9477dd0da 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -161,3 +161,7 @@ Deprecation Notices
> > The new port library API (functions rte_swx_port_*)
> > will gradually transition from experimental to stable status
> > starting with DPDK 23.07 release.
> > +
> > +* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated
> > +in
> > + DPDK 23.07, and removed in DPDK 23.11. The relevant code can be
> > +updated using
> > + ``RTE_ETH_DEV_BONDING_MEMBER``.
>
>
> It can be better to move this below the ethdev entries, to group them, I can do
> this while merging.
Okay, I will do it in the v2 patch, thanks very much.
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v2 0/3] announce bonding macro and function change
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
` (3 preceding siblings ...)
2023-07-17 15:14 ` [PATCH 0/3] announce bonding macro and " Ferruh Yigit
@ 2023-07-18 1:48 ` Chaoyong He
2023-07-18 1:48 ` [PATCH v2 1/3] doc: announce bonding macro change Chaoyong He
` (3 more replies)
4 siblings, 4 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:48 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, some of the macro and function in
DPDK will need to be renamed. Do this through deprecation process now
for 23.07.
---
v2:
* Drop the rename of function and data struct in the logics.
---
Chaoyong He (2):
doc: announce bonding data change
doc: announce bonding function change
Long Wu (1):
doc: announce bonding macro change
app/test-pmd/testpmd.c | 2 +-
doc/guides/rel_notes/deprecation.rst | 18 ++++++++++++++++++
drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
lib/ethdev/rte_ethdev.h | 5 +++--
4 files changed, 25 insertions(+), 6 deletions(-)
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v2 1/3] doc: announce bonding macro change
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
@ 2023-07-18 1:48 ` Chaoyong He
2023-07-18 8:09 ` Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 2/3] doc: announce bonding data change Chaoyong He
` (2 subsequent siblings)
3 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:48 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Long Wu, Chaoyong He
From: Long Wu <long.wu@corigine.com>
In order to support inclusive naming, some of the macro in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
---
app/test-pmd/testpmd.c | 2 +-
doc/guides/rel_notes/deprecation.rst | 3 +++
drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
lib/ethdev/rte_ethdev.h | 5 +++--
4 files changed, 10 insertions(+), 6 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index c6ad9b18bf..938ca035d4 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -4248,7 +4248,7 @@ uint8_t port_is_bonding_slave(portid_t slave_pid)
slave_pid);
return 0;
}
- if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDED_SLAVE) || (port->slave_flag == 1))
+ if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDING_MEMBER) || (port->slave_flag == 1))
return 1;
return 0;
}
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index fb771a0305..fc50b73761 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -117,6 +117,9 @@ Deprecation Notices
superseded by the generic ``RTE_FLOW_ACTION_TYPE_MODIFY_FIELD``.
The legacy actions should be removed
once ``MODIFY_FIELD`` alternative is implemented in drivers.
+ The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
+ DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
+ ``RTE_ETH_DEV_BONDING_MEMBER``.
* cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
to have another parameter ``qp_id`` to return the queue pair ID
diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
index 85d0528b7c..8b6cdce34a 100644
--- a/drivers/net/bonding/rte_eth_bond_api.c
+++ b/drivers/net/bonding/rte_eth_bond_api.c
@@ -472,7 +472,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
return -1;
slave_eth_dev = &rte_eth_devices[slave_port_id];
- if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDED_SLAVE) {
+ if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDING_MEMBER) {
RTE_BOND_LOG(ERR, "Slave device is already a slave of a bonded device");
return -1;
}
@@ -615,7 +615,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
}
/* Add slave details to bonded device */
- slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDED_SLAVE;
+ slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDING_MEMBER;
slave_vlan_filter_set(bonded_port_id, slave_port_id);
@@ -724,7 +724,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
slave_eth_dev = &rte_eth_devices[slave_port_id];
slave_remove(internals, slave_eth_dev);
- slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDED_SLAVE);
+ slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDING_MEMBER);
/* first slave in the active list will be the primary by default,
* otherwise use first device in list */
diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h
index 3d44979b44..04a2564f22 100644
--- a/lib/ethdev/rte_ethdev.h
+++ b/lib/ethdev/rte_ethdev.h
@@ -2036,8 +2036,9 @@ struct rte_eth_dev_owner {
#define RTE_ETH_DEV_FLOW_OPS_THREAD_SAFE RTE_BIT32(0)
/** Device supports link state interrupt */
#define RTE_ETH_DEV_INTR_LSC RTE_BIT32(1)
-/** Device is a bonded slave */
-#define RTE_ETH_DEV_BONDED_SLAVE RTE_BIT32(2)
+/** Device is a bonding member */
+#define RTE_ETH_DEV_BONDING_MEMBER RTE_BIT32(2)
+#define RTE_ETH_DEV_BONDED_SLAVE RTE_DEPRECATED(RTE_ETH_DEV_BONDED_SLAVE) RTE_ETH_DEV_BONDING_MEMBER
/** Device supports device removal interrupt */
#define RTE_ETH_DEV_INTR_RMV RTE_BIT32(3)
/** Device is port representor */
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v2 2/3] doc: announce bonding data change
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
2023-07-18 1:48 ` [PATCH v2 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-18 1:48 ` Chaoyong He
2023-07-18 8:05 ` Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 3/3] doc: announce bonding function change Chaoyong He
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
3 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:48 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, the data structure of bonding 8023
info need to be renamed. Do this through deprecation process now for
23.07.
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
doc/guides/rel_notes/deprecation.rst | 3 +++
1 file changed, 3 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index fc50b73761..dc10c9020c 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -164,3 +164,6 @@ Deprecation Notices
The new port library API (functions rte_swx_port_*)
will gradually transition from experimental to stable status
starting with DPDK 23.07 release.
+
+* bonding: The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
+ renamed in DPDK 23.11.
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v2 3/3] doc: announce bonding function change
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
2023-07-18 1:48 ` [PATCH v2 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 1:48 ` [PATCH v2 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-18 1:48 ` Chaoyong He
2023-07-18 8:05 ` Ferruh Yigit
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
3 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 1:48 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He, Long Wu
In order to support inclusive naming, some of the function in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
doc/guides/rel_notes/deprecation.rst | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index dc10c9020c..32d54df743 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -167,3 +167,15 @@ Deprecation Notices
* bonding: The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
renamed in DPDK 23.11.
+ The following functions will be deprecated in DPDK 23.07, and removed in
+ DPDK 23.11. The old functions:
+ ``rte_eth_bond_8023ad_slave_info``,
+ ``rte_eth_bond_active_slaves_get``,
+ ``rte_eth_bond_slave_add``,
+ ``rte_eth_bond_slave_remove``, and
+ ``rte_eth_bond_slaves_get`` will be replaced by:
+ ``rte_eth_bond_8023ad_member_info``,
+ ``rte_eth_bond_active_members_get``,
+ ``rte_eth_bond_member_add``,
+ ``rte_eth_bond_member_remove``,
+ and ``rte_eth_bond_members_get``.
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v2 2/3] doc: announce bonding data change
2023-07-18 1:48 ` [PATCH v2 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-18 8:05 ` Ferruh Yigit
0 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 8:05 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund
On 7/18/2023 2:48 AM, Chaoyong He wrote:
> In order to support inclusive naming, the data structure of bonding 8023
> info need to be renamed. Do this through deprecation process now for
> 23.07.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index fc50b73761..dc10c9020c 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -164,3 +164,6 @@ Deprecation Notices
> The new port library API (functions rte_swx_port_*)
> will gradually transition from experimental to stable status
> starting with DPDK 23.07 release.
> +
> +* bonding: The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> + renamed in DPDK 23.11.
I think we know what new name will be, so what about documenting it?
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v2 3/3] doc: announce bonding function change
2023-07-18 1:48 ` [PATCH v2 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-18 8:05 ` Ferruh Yigit
0 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 8:05 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/18/2023 2:48 AM, Chaoyong He wrote:
> In order to support inclusive naming, some of the function in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index dc10c9020c..32d54df743 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -167,3 +167,15 @@ Deprecation Notices
>
> * bonding: The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> renamed in DPDK 23.11.
> + The following functions will be deprecated in DPDK 23.07, and removed in
> + DPDK 23.11. The old functions:
"deprecated in DPDK 23.07" part is no more in this patch.
> + ``rte_eth_bond_8023ad_slave_info``,
> + ``rte_eth_bond_active_slaves_get``,
> + ``rte_eth_bond_slave_add``,
> + ``rte_eth_bond_slave_remove``, and
> + ``rte_eth_bond_slaves_get`` will be replaced by:
> + ``rte_eth_bond_8023ad_member_info``,
> + ``rte_eth_bond_active_members_get``,
> + ``rte_eth_bond_member_add``,
> + ``rte_eth_bond_member_remove``,
> + and ``rte_eth_bond_members_get``.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v2 1/3] doc: announce bonding macro change
2023-07-18 1:48 ` [PATCH v2 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-18 8:09 ` Ferruh Yigit
0 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 8:09 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/18/2023 2:48 AM, Chaoyong He wrote:
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index fb771a0305..fc50b73761 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -117,6 +117,9 @@ Deprecation Notices
> superseded by the generic ``RTE_FLOW_ACTION_TYPE_MODIFY_FIELD``.
> The legacy actions should be removed
> once ``MODIFY_FIELD`` alternative is implemented in drivers.
> + The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
> + DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
> + ``RTE_ETH_DEV_BONDING_MEMBER``.
>
Just for ordering, can you please keep new bullet item "* bonding: " as
done in previous version, but just place it here (below ethdev, before
cryptodev), and append updates in next patches to this block?
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 0/3] announce bonding macro and function change
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
` (2 preceding siblings ...)
2023-07-18 1:48 ` [PATCH v2 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-18 8:28 ` Chaoyong He
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
` (4 more replies)
3 siblings, 5 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 8:28 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, some of the macro and function in
DPDK will need to be renamed. Do this through deprecation process now
for 23.07.
---
v3:
* Adjust the statement in 'deprecation.rst' document.
v2:
* Drop the rename of function and data struct in the logics.
---
Chaoyong He (2):
doc: announce bonding data change
doc: announce bonding function change
Long Wu (1):
doc: announce bonding macro change
app/test-pmd/testpmd.c | 2 +-
doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
lib/ethdev/rte_ethdev.h | 5 +++--
4 files changed, 24 insertions(+), 6 deletions(-)
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 1/3] doc: announce bonding macro change
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
@ 2023-07-18 8:28 ` Chaoyong He
2023-07-18 11:39 ` Ferruh Yigit
` (3 more replies)
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
` (3 subsequent siblings)
4 siblings, 4 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 8:28 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Long Wu, Chaoyong He
From: Long Wu <long.wu@corigine.com>
In order to support inclusive naming, some of the macro in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
---
app/test-pmd/testpmd.c | 2 +-
doc/guides/rel_notes/deprecation.rst | 4 ++++
drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
lib/ethdev/rte_ethdev.h | 5 +++--
4 files changed, 11 insertions(+), 6 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index c6ad9b18bf..938ca035d4 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -4248,7 +4248,7 @@ uint8_t port_is_bonding_slave(portid_t slave_pid)
slave_pid);
return 0;
}
- if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDED_SLAVE) || (port->slave_flag == 1))
+ if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDING_MEMBER) || (port->slave_flag == 1))
return 1;
return 0;
}
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index fb771a0305..f3f2baf0b9 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -118,6 +118,10 @@ Deprecation Notices
The legacy actions should be removed
once ``MODIFY_FIELD`` alternative is implemented in drivers.
+* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
+ DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
+ ``RTE_ETH_DEV_BONDING_MEMBER``.
+
* cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
to have another parameter ``qp_id`` to return the queue pair ID
which got error interrupt to the application,
diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
index 85d0528b7c..8b6cdce34a 100644
--- a/drivers/net/bonding/rte_eth_bond_api.c
+++ b/drivers/net/bonding/rte_eth_bond_api.c
@@ -472,7 +472,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
return -1;
slave_eth_dev = &rte_eth_devices[slave_port_id];
- if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDED_SLAVE) {
+ if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDING_MEMBER) {
RTE_BOND_LOG(ERR, "Slave device is already a slave of a bonded device");
return -1;
}
@@ -615,7 +615,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
}
/* Add slave details to bonded device */
- slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDED_SLAVE;
+ slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDING_MEMBER;
slave_vlan_filter_set(bonded_port_id, slave_port_id);
@@ -724,7 +724,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
slave_eth_dev = &rte_eth_devices[slave_port_id];
slave_remove(internals, slave_eth_dev);
- slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDED_SLAVE);
+ slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDING_MEMBER);
/* first slave in the active list will be the primary by default,
* otherwise use first device in list */
diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h
index 3d44979b44..04a2564f22 100644
--- a/lib/ethdev/rte_ethdev.h
+++ b/lib/ethdev/rte_ethdev.h
@@ -2036,8 +2036,9 @@ struct rte_eth_dev_owner {
#define RTE_ETH_DEV_FLOW_OPS_THREAD_SAFE RTE_BIT32(0)
/** Device supports link state interrupt */
#define RTE_ETH_DEV_INTR_LSC RTE_BIT32(1)
-/** Device is a bonded slave */
-#define RTE_ETH_DEV_BONDED_SLAVE RTE_BIT32(2)
+/** Device is a bonding member */
+#define RTE_ETH_DEV_BONDING_MEMBER RTE_BIT32(2)
+#define RTE_ETH_DEV_BONDED_SLAVE RTE_DEPRECATED(RTE_ETH_DEV_BONDED_SLAVE) RTE_ETH_DEV_BONDING_MEMBER
/** Device supports device removal interrupt */
#define RTE_ETH_DEV_INTR_RMV RTE_BIT32(3)
/** Device is port representor */
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 2/3] doc: announce bonding data change
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-18 8:28 ` Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
` (2 more replies)
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
` (2 subsequent siblings)
4 siblings, 3 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 8:28 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He
In order to support inclusive naming, the data structure of bonding 8023
info need to be renamed. Do this through deprecation process now for
23.07.
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
doc/guides/rel_notes/deprecation.rst | 2 ++
1 file changed, 2 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index f3f2baf0b9..8cf972a9ab 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -121,6 +121,8 @@ Deprecation Notices
* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
``RTE_ETH_DEV_BONDING_MEMBER``.
+ The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
+ renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
* cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
to have another parameter ``qp_id`` to return the queue pair ID
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* [PATCH v3 3/3] doc: announce bonding function change
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-18 8:28 ` Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
` (2 more replies)
2023-07-26 9:06 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
2023-07-28 13:54 ` Thomas Monjalon
4 siblings, 3 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-18 8:28 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, niklas.soderlund, Chaoyong He, Long Wu
In order to support inclusive naming, some of the function in DPDK will
need to be renamed. Do this through deprecation process now for 23.07.
Signed-off-by: Long Wu <long.wu@corigine.com>
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 8cf972a9ab..45da0361e6 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -123,6 +123,17 @@ Deprecation Notices
``RTE_ETH_DEV_BONDING_MEMBER``.
The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
+ The following functions will be removed in DPDK 23.11. The old functions:
+ ``rte_eth_bond_8023ad_slave_info``,
+ ``rte_eth_bond_active_slaves_get``,
+ ``rte_eth_bond_slave_add``,
+ ``rte_eth_bond_slave_remove``, and
+ ``rte_eth_bond_slaves_get`` will be replaced by:
+ ``rte_eth_bond_8023ad_member_info``,
+ ``rte_eth_bond_active_members_get``,
+ ``rte_eth_bond_member_add``,
+ ``rte_eth_bond_member_remove``, and
+ ``rte_eth_bond_members_get``.
* cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
to have another parameter ``qp_id`` to return the queue pair ID
--
2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/3] doc: announce bonding macro change
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
@ 2023-07-18 11:39 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
` (2 subsequent siblings)
3 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 11:39 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/18/2023 9:28 AM, Chaoyong He wrote:
> From: Long Wu <long.wu@corigine.com>
>
> In order to support inclusive naming, some of the macro in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 2/3] doc: announce bonding data change
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
@ 2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
2023-07-19 1:32 ` lihuisong (C)
2 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 11:40 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund
On 7/18/2023 9:28 AM, Chaoyong He wrote:
> In order to support inclusive naming, the data structure of bonding 8023
> info need to be renamed. Do this through deprecation process now for
> 23.07.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 3/3] doc: announce bonding function change
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:52 ` Stephen Hemminger
2023-07-19 1:31 ` lihuisong (C)
2 siblings, 0 replies; 36+ messages in thread
From: Ferruh Yigit @ 2023-07-18 11:40 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
On 7/18/2023 9:28 AM, Chaoyong He wrote:
> In order to support inclusive naming, some of the function in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 3/3] doc: announce bonding function change
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
@ 2023-07-18 15:52 ` Stephen Hemminger
2023-07-19 1:31 ` lihuisong (C)
2 siblings, 0 replies; 36+ messages in thread
From: Stephen Hemminger @ 2023-07-18 15:52 UTC (permalink / raw)
To: Chaoyong He; +Cc: dev, oss-drivers, niklas.soderlund, Long Wu
On Tue, 18 Jul 2023 16:28:56 +0800
Chaoyong He <chaoyong.he@corigine.com> wrote:
> In order to support inclusive naming, some of the function in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 2/3] doc: announce bonding data change
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
@ 2023-07-18 15:53 ` Stephen Hemminger
2023-07-19 1:32 ` lihuisong (C)
2 siblings, 0 replies; 36+ messages in thread
From: Stephen Hemminger @ 2023-07-18 15:53 UTC (permalink / raw)
To: Chaoyong He; +Cc: dev, oss-drivers, niklas.soderlund
On Tue, 18 Jul 2023 16:28:55 +0800
Chaoyong He <chaoyong.he@corigine.com> wrote:
> In order to support inclusive naming, the data structure of bonding 8023
> info need to be renamed. Do this through deprecation process now for
> 23.07.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/3] doc: announce bonding macro change
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 11:39 ` Ferruh Yigit
@ 2023-07-18 15:53 ` Stephen Hemminger
2023-07-19 1:24 ` lihuisong (C)
2023-07-26 11:47 ` Morten Brørup
3 siblings, 0 replies; 36+ messages in thread
From: Stephen Hemminger @ 2023-07-18 15:53 UTC (permalink / raw)
To: Chaoyong He; +Cc: dev, oss-drivers, niklas.soderlund, Long Wu
On Tue, 18 Jul 2023 16:28:54 +0800
Chaoyong He <chaoyong.he@corigine.com> wrote:
> From: Long Wu <long.wu@corigine.com>
>
> In order to support inclusive naming, some of the macro in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 1/3] doc: announce bonding macro change
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 11:39 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
@ 2023-07-19 1:24 ` lihuisong (C)
2023-07-26 11:47 ` Morten Brørup
3 siblings, 0 replies; 36+ messages in thread
From: lihuisong (C) @ 2023-07-19 1:24 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
+1 do not use slave in macro, data and function.
Acked-by: Huisong Li <lihuisong@huawei.com>
在 2023/7/18 16:28, Chaoyong He 写道:
> From: Long Wu <long.wu@corigine.com>
>
> In order to support inclusive naming, some of the macro in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> app/test-pmd/testpmd.c | 2 +-
> doc/guides/rel_notes/deprecation.rst | 4 ++++
> drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
> lib/ethdev/rte_ethdev.h | 5 +++--
> 4 files changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
> index c6ad9b18bf..938ca035d4 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -4248,7 +4248,7 @@ uint8_t port_is_bonding_slave(portid_t slave_pid)
> slave_pid);
> return 0;
> }
> - if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDED_SLAVE) || (port->slave_flag == 1))
> + if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDING_MEMBER) || (port->slave_flag == 1))
> return 1;
> return 0;
> }
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index fb771a0305..f3f2baf0b9 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -118,6 +118,10 @@ Deprecation Notices
> The legacy actions should be removed
> once ``MODIFY_FIELD`` alternative is implemented in drivers.
>
> +* bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
> + DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
> + ``RTE_ETH_DEV_BONDING_MEMBER``.
> +
> * cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
> to have another parameter ``qp_id`` to return the queue pair ID
> which got error interrupt to the application,
> diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
> index 85d0528b7c..8b6cdce34a 100644
> --- a/drivers/net/bonding/rte_eth_bond_api.c
> +++ b/drivers/net/bonding/rte_eth_bond_api.c
> @@ -472,7 +472,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
> return -1;
>
> slave_eth_dev = &rte_eth_devices[slave_port_id];
> - if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDED_SLAVE) {
> + if (slave_eth_dev->data->dev_flags & RTE_ETH_DEV_BONDING_MEMBER) {
> RTE_BOND_LOG(ERR, "Slave device is already a slave of a bonded device");
> return -1;
> }
> @@ -615,7 +615,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
> }
>
> /* Add slave details to bonded device */
> - slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDED_SLAVE;
> + slave_eth_dev->data->dev_flags |= RTE_ETH_DEV_BONDING_MEMBER;
>
> slave_vlan_filter_set(bonded_port_id, slave_port_id);
>
> @@ -724,7 +724,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
>
> slave_eth_dev = &rte_eth_devices[slave_port_id];
> slave_remove(internals, slave_eth_dev);
> - slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDED_SLAVE);
> + slave_eth_dev->data->dev_flags &= (~RTE_ETH_DEV_BONDING_MEMBER);
>
> /* first slave in the active list will be the primary by default,
> * otherwise use first device in list */
> diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h
> index 3d44979b44..04a2564f22 100644
> --- a/lib/ethdev/rte_ethdev.h
> +++ b/lib/ethdev/rte_ethdev.h
> @@ -2036,8 +2036,9 @@ struct rte_eth_dev_owner {
> #define RTE_ETH_DEV_FLOW_OPS_THREAD_SAFE RTE_BIT32(0)
> /** Device supports link state interrupt */
> #define RTE_ETH_DEV_INTR_LSC RTE_BIT32(1)
> -/** Device is a bonded slave */
> -#define RTE_ETH_DEV_BONDED_SLAVE RTE_BIT32(2)
> +/** Device is a bonding member */
> +#define RTE_ETH_DEV_BONDING_MEMBER RTE_BIT32(2)
> +#define RTE_ETH_DEV_BONDED_SLAVE RTE_DEPRECATED(RTE_ETH_DEV_BONDED_SLAVE) RTE_ETH_DEV_BONDING_MEMBER
> /** Device supports device removal interrupt */
> #define RTE_ETH_DEV_INTR_RMV RTE_BIT32(3)
> /** Device is port representor */
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 3/3] doc: announce bonding function change
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:52 ` Stephen Hemminger
@ 2023-07-19 1:31 ` lihuisong (C)
2023-07-19 1:37 ` Chaoyong He
2 siblings, 1 reply; 36+ messages in thread
From: lihuisong (C) @ 2023-07-19 1:31 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Long Wu
Hi Chaoyong,
All something about slave in variables and logs should be fixed.
Is this in your plan?
/Huisong
在 2023/7/18 16:28, Chaoyong He 写道:
> In order to support inclusive naming, some of the function in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 8cf972a9ab..45da0361e6 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -123,6 +123,17 @@ Deprecation Notices
> ``RTE_ETH_DEV_BONDING_MEMBER``.
> The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
> + The following functions will be removed in DPDK 23.11. The old functions:
> + ``rte_eth_bond_8023ad_slave_info``,
> + ``rte_eth_bond_active_slaves_get``,
> + ``rte_eth_bond_slave_add``,
> + ``rte_eth_bond_slave_remove``, and
> + ``rte_eth_bond_slaves_get`` will be replaced by:
> + ``rte_eth_bond_8023ad_member_info``,
> + ``rte_eth_bond_active_members_get``,
> + ``rte_eth_bond_member_add``,
> + ``rte_eth_bond_member_remove``, and
> + ``rte_eth_bond_members_get``.
>
> * cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
> to have another parameter ``qp_id`` to return the queue pair ID
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 2/3] doc: announce bonding data change
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
@ 2023-07-19 1:32 ` lihuisong (C)
2 siblings, 0 replies; 36+ messages in thread
From: lihuisong (C) @ 2023-07-19 1:32 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund
在 2023/7/18 16:28, Chaoyong He 写道:
> In order to support inclusive naming, the data structure of bonding 8023
> info need to be renamed. Do this through deprecation process now for
> 23.07.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> doc/guides/rel_notes/deprecation.rst | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index f3f2baf0b9..8cf972a9ab 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -121,6 +121,8 @@ Deprecation Notices
> * bonding: The macro ``RTE_ETH_DEV_BONDED_SLAVE`` will be deprecated in
> DPDK 23.07, and removed in DPDK 23.11. The relevant code can be updated using
> ``RTE_ETH_DEV_BONDING_MEMBER``.
> + The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> + renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
>
> * cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
> to have another parameter ``qp_id`` to return the queue pair ID
Acked-by: Huisong Li <lihuisong@huawei.com>
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH v3 3/3] doc: announce bonding function change
2023-07-19 1:31 ` lihuisong (C)
@ 2023-07-19 1:37 ` Chaoyong He
2023-07-19 1:53 ` lihuisong (C)
0 siblings, 1 reply; 36+ messages in thread
From: Chaoyong He @ 2023-07-19 1:37 UTC (permalink / raw)
To: lihuisong (C), dev; +Cc: oss-drivers, Niklas Soderlund, Long Wu
> [Some people who received this message don't often get email from
> lihuisong@huawei.com. Learn why this is important at
> https://aka.ms/LearnAboutSenderIdentification ]
>
> Hi Chaoyong,
>
> All something about slave in variables and logs should be fixed.
> Is this in your plan?
>
> /Huisong
>
Oh, yes, we have a patch series to do that (replace master/slave in everywhere for bonding PMD).
And that patch series has been delay to v23.11 version.
>
> 在 2023/7/18 16:28, Chaoyong He 写道:
> > In order to support inclusive naming, some of the function in DPDK
> > will need to be renamed. Do this through deprecation process now for
> 23.07.
> >
> > Signed-off-by: Long Wu <long.wu@corigine.com>
> > Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> > ---
> > doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> > b/doc/guides/rel_notes/deprecation.rst
> > index 8cf972a9ab..45da0361e6 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -123,6 +123,17 @@ Deprecation Notices
> > ``RTE_ETH_DEV_BONDING_MEMBER``.
> > The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
> > renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
> > + The following functions will be removed in DPDK 23.11. The old functions:
> > + ``rte_eth_bond_8023ad_slave_info``,
> > + ``rte_eth_bond_active_slaves_get``,
> > + ``rte_eth_bond_slave_add``,
> > + ``rte_eth_bond_slave_remove``, and
> > + ``rte_eth_bond_slaves_get`` will be replaced by:
> > + ``rte_eth_bond_8023ad_member_info``,
> > + ``rte_eth_bond_active_members_get``,
> > + ``rte_eth_bond_member_add``,
> > + ``rte_eth_bond_member_remove``, and ``rte_eth_bond_members_get``.
> >
> > * cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
> > to have another parameter ``qp_id`` to return the queue pair ID
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 3/3] doc: announce bonding function change
2023-07-19 1:37 ` Chaoyong He
@ 2023-07-19 1:53 ` lihuisong (C)
0 siblings, 0 replies; 36+ messages in thread
From: lihuisong (C) @ 2023-07-19 1:53 UTC (permalink / raw)
To: Chaoyong He, dev; +Cc: oss-drivers, Niklas Soderlund, Long Wu
在 2023/7/19 9:37, Chaoyong He 写道:
>> [Some people who received this message don't often get email from
>> lihuisong@huawei.com. Learn why this is important at
>> https://aka.ms/LearnAboutSenderIdentification ]
>>
>> Hi Chaoyong,
>>
>> All something about slave in variables and logs should be fixed.
>> Is this in your plan?
>>
>> /Huisong
>>
> Oh, yes, we have a patch series to do that (replace master/slave in everywhere for bonding PMD).
> And that patch series has been delay to v23.11 version.
great.
Acked-by: Huisong Li <lihuisong@huawei.com>
>
>> 在 2023/7/18 16:28, Chaoyong He 写道:
>>> In order to support inclusive naming, some of the function in DPDK
>>> will need to be renamed. Do this through deprecation process now for
>> 23.07.
>>> Signed-off-by: Long Wu <long.wu@corigine.com>
>>> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
>>> ---
>>> doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
>>> 1 file changed, 11 insertions(+)
>>>
>>> diff --git a/doc/guides/rel_notes/deprecation.rst
>>> b/doc/guides/rel_notes/deprecation.rst
>>> index 8cf972a9ab..45da0361e6 100644
>>> --- a/doc/guides/rel_notes/deprecation.rst
>>> +++ b/doc/guides/rel_notes/deprecation.rst
>>> @@ -123,6 +123,17 @@ Deprecation Notices
>>> ``RTE_ETH_DEV_BONDING_MEMBER``.
>>> The data structure ``struct rte_eth_bond_8023ad_slave_info`` will be
>>> renamed to ``struct rte_eth_bond_8023ad_member_info`` in DPDK 23.11.
>>> + The following functions will be removed in DPDK 23.11. The old functions:
>>> + ``rte_eth_bond_8023ad_slave_info``,
>>> + ``rte_eth_bond_active_slaves_get``,
>>> + ``rte_eth_bond_slave_add``,
>>> + ``rte_eth_bond_slave_remove``, and
>>> + ``rte_eth_bond_slaves_get`` will be replaced by:
>>> + ``rte_eth_bond_8023ad_member_info``,
>>> + ``rte_eth_bond_active_members_get``,
>>> + ``rte_eth_bond_member_add``,
>>> + ``rte_eth_bond_member_remove``, and ``rte_eth_bond_members_get``.
>>>
>>> * cryptodev: The function ``rte_cryptodev_cb_fn`` will be updated
>>> to have another parameter ``qp_id`` to return the queue pair ID
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH v3 0/3] announce bonding macro and function change
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
` (2 preceding siblings ...)
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
@ 2023-07-26 9:06 ` Chaoyong He
2023-07-28 13:54 ` Thomas Monjalon
4 siblings, 0 replies; 36+ messages in thread
From: Chaoyong He @ 2023-07-26 9:06 UTC (permalink / raw)
To: dev; +Cc: oss-drivers, Niklas Soderlund
A gentle ping~
Could anyone kindly enough give us a 4th 'Acked-by' to make it can be merged into v23.07?
Thanks very much!
> -----Original Message-----
> From: Chaoyong He
> Sent: Tuesday, July 18, 2023 4:29 PM
> To: dev@dpdk.org
> Cc: oss-drivers <oss-drivers@corigine.com>; Niklas Soderlund
> <niklas.soderlund@corigine.com>; Chaoyong He
> <chaoyong.he@corigine.com>
> Subject: [PATCH v3 0/3] announce bonding macro and function change
>
> In order to support inclusive naming, some of the macro and function in DPDK
> will need to be renamed. Do this through deprecation process now for 23.07.
>
> ---
> v3:
> * Adjust the statement in 'deprecation.rst' document.
> v2:
> * Drop the rename of function and data struct in the logics.
> ---
>
> Chaoyong He (2):
> doc: announce bonding data change
> doc: announce bonding function change
>
> Long Wu (1):
> doc: announce bonding macro change
>
> app/test-pmd/testpmd.c | 2 +-
> doc/guides/rel_notes/deprecation.rst | 17 +++++++++++++++++
> drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
> lib/ethdev/rte_ethdev.h | 5 +++--
> 4 files changed, 24 insertions(+), 6 deletions(-)
>
> --
> 2.39.1
^ permalink raw reply [flat|nested] 36+ messages in thread
* RE: [PATCH v3 1/3] doc: announce bonding macro change
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
` (2 preceding siblings ...)
2023-07-19 1:24 ` lihuisong (C)
@ 2023-07-26 11:47 ` Morten Brørup
3 siblings, 0 replies; 36+ messages in thread
From: Morten Brørup @ 2023-07-26 11:47 UTC (permalink / raw)
To: Chaoyong He, dev
Cc: oss-drivers, niklas.soderlund, Long Wu, stephen, Ferruh Yigit,
Huisong Li
> From: Chaoyong He [mailto:chaoyong.he@corigine.com]
> Sent: Tuesday, 18 July 2023 10.29
>
> From: Long Wu <long.wu@corigine.com>
>
> In order to support inclusive naming, some of the macro in DPDK will
> need to be renamed. Do this through deprecation process now for 23.07.
>
> Signed-off-by: Long Wu <long.wu@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
> ---
> app/test-pmd/testpmd.c | 2 +-
> doc/guides/rel_notes/deprecation.rst | 4 ++++
> drivers/net/bonding/rte_eth_bond_api.c | 6 +++---
> lib/ethdev/rte_ethdev.h | 5 +++--
> 4 files changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
> index c6ad9b18bf..938ca035d4 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -4248,7 +4248,7 @@ uint8_t port_is_bonding_slave(portid_t slave_pid)
> slave_pid);
We might want to get rid of the term "slave" in function, parameter and variable names in testpmd too. This is not important now, and can be postponed to a later patch.
> return 0;
> }
> - if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDED_SLAVE) || (port-
> >slave_flag == 1))
> + if ((*dev_info.dev_flags & RTE_ETH_DEV_BONDING_MEMBER) || (port-
> >slave_flag == 1))
Can we please standardize on using only "bond" and "BOND" everywhere in this driver, instead of both "bond" (in function names) and "BONDING" (in enum values)?
The source code file are also named "rte_eth_bond...", while the directory name is /drivers/net/bonding/.
We are about to rename anyway, so let's do it as good as we can.
Also, are the bonding and balancing modes defined in /drivers/net/bonding/rte_eth_bond.h missing the RTE_ prefix, or are they private?
PS: Sorry about joining the discussion late (and possibly ignoring previous discussions on this). I reacted to the poll for ACKs to this series.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [PATCH v3 0/3] announce bonding macro and function change
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
` (3 preceding siblings ...)
2023-07-26 9:06 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
@ 2023-07-28 13:54 ` Thomas Monjalon
4 siblings, 0 replies; 36+ messages in thread
From: Thomas Monjalon @ 2023-07-28 13:54 UTC (permalink / raw)
To: Chaoyong He; +Cc: dev, oss-drivers, niklas.soderlund
18/07/2023 10:28, Chaoyong He:
> In order to support inclusive naming, some of the macro and function in
> DPDK will need to be renamed. Do this through deprecation process now
> for 23.07.
>
> ---
> v3:
> * Adjust the statement in 'deprecation.rst' document.
> v2:
> * Drop the rename of function and data struct in the logics.
> ---
>
> Chaoyong He (2):
> doc: announce bonding data change
> doc: announce bonding function change
>
> Long Wu (1):
> doc: announce bonding macro change
Applied, thanks.
^ permalink raw reply [flat|nested] 36+ messages in thread
end of thread, other threads:[~2023-07-28 13:54 UTC | newest]
Thread overview: 36+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-14 8:15 [PATCH 0/3] announce bonding macro and function change Chaoyong He
2023-07-14 8:15 ` [PATCH 1/3] doc: announce bonding macro change Chaoyong He
2023-07-17 15:13 ` Ferruh Yigit
2023-07-18 1:17 ` Chaoyong He
2023-07-14 8:15 ` [PATCH 2/3] doc: announce bonding data change Chaoyong He
2023-07-17 15:03 ` Ferruh Yigit
2023-07-18 1:13 ` Chaoyong He
2023-07-14 8:15 ` [PATCH 3/3] doc: announce bonding function change Chaoyong He
2023-07-17 15:13 ` Ferruh Yigit
2023-07-18 1:15 ` Chaoyong He
2023-07-17 15:14 ` [PATCH 0/3] announce bonding macro and " Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 " Chaoyong He
2023-07-18 1:48 ` [PATCH v2 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 8:09 ` Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 2/3] doc: announce bonding data change Chaoyong He
2023-07-18 8:05 ` Ferruh Yigit
2023-07-18 1:48 ` [PATCH v2 3/3] doc: announce bonding function change Chaoyong He
2023-07-18 8:05 ` Ferruh Yigit
2023-07-18 8:28 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
2023-07-18 8:28 ` [PATCH v3 1/3] doc: announce bonding macro change Chaoyong He
2023-07-18 11:39 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
2023-07-19 1:24 ` lihuisong (C)
2023-07-26 11:47 ` Morten Brørup
2023-07-18 8:28 ` [PATCH v3 2/3] doc: announce bonding data change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:53 ` Stephen Hemminger
2023-07-19 1:32 ` lihuisong (C)
2023-07-18 8:28 ` [PATCH v3 3/3] doc: announce bonding function change Chaoyong He
2023-07-18 11:40 ` Ferruh Yigit
2023-07-18 15:52 ` Stephen Hemminger
2023-07-19 1:31 ` lihuisong (C)
2023-07-19 1:37 ` Chaoyong He
2023-07-19 1:53 ` lihuisong (C)
2023-07-26 9:06 ` [PATCH v3 0/3] announce bonding macro and " Chaoyong He
2023-07-28 13:54 ` Thomas Monjalon
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).