[dpdk-stable] patch 'net/bonding: delete redundant code' has been queued to stable release 19.11.4

[luca.boccassi at gmail.com]

Hi,

FYI, your patch has been queued to stable release 19.11.4

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 07/26/20. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Thanks.

Luca Boccassi

---
>From 2ac0580bcb2c757bd54cf0e2c3252781660368a4 Mon Sep 17 00:00:00 2001
From: Dongyang Pan <197020236 at qq.com>
Date: Sat, 4 Jul 2020 09:15:26 +0800
Subject: [PATCH] net/bonding: delete redundant code

[ upstream commit 2db4cf9d01c5920be0397e5c7aba61779c997753 ]

The function valid_bonded_port_id() has already contains function
rte_eth_dev_is_valid_port(), so delete redundant check.

Fixes: 588ae95e7983 ("net/bonding: fix port ID check")

Signed-off-by: Dongyang Pan <197020236 at qq.com>
Reviewed-by: Wei Hu (Xavier) <xavier.huwei at huawei.com>
---
 drivers/net/bonding/rte_eth_bond_8023ad.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c
index 3991825ad..b24a44686 100644
--- a/drivers/net/bonding/rte_eth_bond_8023ad.c
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.c
@@ -1676,9 +1676,6 @@ rte_eth_bond_8023ad_dedicated_queues_enable(uint16_t port)
 	dev = &rte_eth_devices[port];
 	internals = dev->data->dev_private;
 
-	if (check_for_bonded_ethdev(dev) != 0)
-		return -1;
-
 	if (bond_8023ad_slow_pkt_hw_filter_supported(port) != 0)
 		return -1;
 
@@ -1705,9 +1702,6 @@ rte_eth_bond_8023ad_dedicated_queues_disable(uint16_t port)
 	dev = &rte_eth_devices[port];
 	internals = dev->data->dev_private;
 
-	if (check_for_bonded_ethdev(dev) != 0)
-		return -1;
-
 	/* Device must be stopped to set up slow queue */
 	if (dev->data->dev_started)
 		return -1;
-- 
2.20.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2020-07-24 12:53:55.007550473 +0100
+++ 0167-net-bonding-delete-redundant-code.patch	2020-07-24 12:53:48.563011602 +0100
@@ -1,13 +1,14 @@
-From 2db4cf9d01c5920be0397e5c7aba61779c997753 Mon Sep 17 00:00:00 2001
+From 2ac0580bcb2c757bd54cf0e2c3252781660368a4 Mon Sep 17 00:00:00 2001
 From: Dongyang Pan <197020236 at qq.com>
 Date: Sat, 4 Jul 2020 09:15:26 +0800
 Subject: [PATCH] net/bonding: delete redundant code
 
+[ upstream commit 2db4cf9d01c5920be0397e5c7aba61779c997753 ]
+
 The function valid_bonded_port_id() has already contains function
 rte_eth_dev_is_valid_port(), so delete redundant check.
 
 Fixes: 588ae95e7983 ("net/bonding: fix port ID check")
-Cc: stable at dpdk.org
 
 Signed-off-by: Dongyang Pan <197020236 at qq.com>
 Reviewed-by: Wei Hu (Xavier) <xavier.huwei at huawei.com>