From: Chaoyong He <chaoyong.he@corigine.com>
To: dev@dpdk.org
Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com,
Chaoyong He <chaoyong.he@corigine.com>
Subject: [PATCH v7] net/nfp: fix the promiscuous mode functions
Date: Sat, 22 Oct 2022 16:11:48 +0800 [thread overview]
Message-ID: <1666426308-9806-1-git-send-email-chaoyong.he@corigine.com> (raw)
In-Reply-To: <1666339323-24016-3-git-send-email-chaoyong.he@corigine.com>
The original functions of promiscuous mode can't process the
representor port rightly, revise the logic to do that.
Fixes: e1124c4f8a45 ("net/nfp: add flower representor framework")
Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
---
drivers/net/nfp/flower/nfp_flower.h | 2 ++
drivers/net/nfp/flower/nfp_flower_representor.c | 47 +++----------------------
drivers/net/nfp/flower/nfp_flower_representor.h | 2 ++
drivers/net/nfp/nfp_common.c | 10 +++++-
4 files changed, 17 insertions(+), 44 deletions(-)
diff --git a/drivers/net/nfp/flower/nfp_flower.h b/drivers/net/nfp/flower/nfp_flower.h
index 12a0fb5..f199741 100644
--- a/drivers/net/nfp/flower/nfp_flower.h
+++ b/drivers/net/nfp/flower/nfp_flower.h
@@ -6,6 +6,8 @@
#ifndef _NFP_FLOWER_H_
#define _NFP_FLOWER_H_
+#include "../nfp_common.h"
+
/*
* Flower fallback and ctrl path always adds and removes
* 8 bytes of prepended data. Tx descriptors must point
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c b/drivers/net/nfp/flower/nfp_flower_representor.c
index f1cd298..5809c83 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.c
+++ b/drivers/net/nfp/flower/nfp_flower_representor.c
@@ -414,45 +414,6 @@
}
static int
-nfp_flower_repr_promiscuous_enable(struct rte_eth_dev *dev)
-{
- struct nfp_net_hw *pf_hw;
- struct nfp_flower_representor *repr;
-
- repr = (struct nfp_flower_representor *)dev->data->dev_private;
- pf_hw = repr->app_fw_flower->pf_hw;
-
- if (!(pf_hw->cap & NFP_NET_CFG_CTRL_PROMISC)) {
- PMD_DRV_LOG(INFO, "Promiscuous mode not supported");
- return -ENOTSUP;
- }
-
- if (pf_hw->ctrl & NFP_NET_CFG_CTRL_PROMISC) {
- PMD_DRV_LOG(INFO, "Promiscuous mode already enabled");
- return 0;
- }
-
- return nfp_net_promisc_enable(pf_hw->eth_dev);
-}
-
-static int
-nfp_flower_repr_promiscuous_disable(struct rte_eth_dev *dev)
-{
- struct nfp_net_hw *pf_hw;
- struct nfp_flower_representor *repr;
-
- repr = (struct nfp_flower_representor *)dev->data->dev_private;
- pf_hw = repr->app_fw_flower->pf_hw;
-
- if ((pf_hw->ctrl & NFP_NET_CFG_CTRL_PROMISC) == 0) {
- PMD_DRV_LOG(INFO, "Promiscuous mode already disabled");
- return 0;
- }
-
- return nfp_net_promisc_disable(pf_hw->eth_dev);
-}
-
-static int
nfp_flower_repr_mac_addr_set(struct rte_eth_dev *ethdev,
struct rte_ether_addr *mac_addr)
{
@@ -566,8 +527,8 @@
.stats_get = nfp_flower_repr_stats_get,
.stats_reset = nfp_flower_repr_stats_reset,
- .promiscuous_enable = nfp_flower_repr_promiscuous_enable,
- .promiscuous_disable = nfp_flower_repr_promiscuous_disable,
+ .promiscuous_enable = nfp_net_promisc_enable,
+ .promiscuous_disable = nfp_net_promisc_enable,
.mac_addr_set = nfp_flower_repr_mac_addr_set,
};
@@ -587,8 +548,8 @@
.stats_get = nfp_flower_repr_stats_get,
.stats_reset = nfp_flower_repr_stats_reset,
- .promiscuous_enable = nfp_flower_repr_promiscuous_enable,
- .promiscuous_disable = nfp_flower_repr_promiscuous_disable,
+ .promiscuous_enable = nfp_net_promisc_enable,
+ .promiscuous_disable = nfp_net_promisc_enable,
.mac_addr_set = nfp_flower_repr_mac_addr_set,
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.h b/drivers/net/nfp/flower/nfp_flower_representor.h
index af44ef3..685cbe4 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.h
+++ b/drivers/net/nfp/flower/nfp_flower_representor.h
@@ -6,6 +6,8 @@
#ifndef _NFP_FLOWER_REPRESENTOR_H_
#define _NFP_FLOWER_REPRESENTOR_H_
+#include "nfp_flower.h"
+
/*
* enum nfp_repr_type - type of representor
* @NFP_REPR_TYPE_PHYS_PORT: external NIC port
diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c
index 277ea23..463eb64 100644
--- a/drivers/net/nfp/nfp_common.c
+++ b/drivers/net/nfp/nfp_common.c
@@ -37,6 +37,8 @@
#include "nfpcore/nfp_rtsym.h"
#include "nfpcore/nfp_nsp.h"
+#include "flower/nfp_flower_representor.h"
+
#include "nfp_common.h"
#include "nfp_ctrl.h"
#include "nfp_rxtx.h"
@@ -412,10 +414,16 @@
uint32_t new_ctrl, update = 0;
struct nfp_net_hw *hw;
int ret;
+ struct nfp_flower_representor *repr;
PMD_DRV_LOG(DEBUG, "Promiscuous mode enable");
- hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+ if ((dev->data->dev_flags & RTE_ETH_DEV_REPRESENTOR) != 0) {
+ repr = dev->data->dev_private;
+ hw = repr->app_fw_flower->pf_hw;
+ } else {
+ hw = NFP_NET_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+ }
if (!(hw->cap & NFP_NET_CFG_CTRL_PROMISC)) {
PMD_INIT_LOG(INFO, "Promiscuous mode not supported");
--
1.8.3.1
next prev parent reply other threads:[~2022-10-22 8:12 UTC|newest]
Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 11:26 [PATCH v4 00/25] add the basic rte_flow offload support of nfp PMD Chaoyong He
2022-10-18 11:26 ` [PATCH v4 01/25] net/nfp: fix the requirement of cpp bridge service Chaoyong He
2022-10-18 12:39 ` Ferruh Yigit
2022-10-18 11:26 ` [PATCH v4 02/25] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-18 11:26 ` [PATCH v4 03/25] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-18 11:26 ` [PATCH v4 04/25] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-18 11:26 ` [PATCH v4 05/25] net/nfp: support basic flow items Chaoyong He
2022-10-18 11:26 ` [PATCH v4 06/25] net/nfp: support basic flow actions Chaoyong He
2022-10-18 11:26 ` [PATCH v4 07/25] net/nfp: support VLAN flow item Chaoyong He
2022-10-18 11:26 ` [PATCH v4 08/25] net/nfp: support IPv4 " Chaoyong He
2022-10-18 11:26 ` [PATCH v4 09/25] net/nfp: support IPv6 " Chaoyong He
2022-10-18 11:26 ` [PATCH v4 10/25] net/nfp: support TCP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 11/25] net/nfp: support UDP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 12/25] net/nfp: support SCTP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 13/25] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-18 11:27 ` [PATCH v4 14/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 15/25] net/nfp: support pop VLAN " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 16/25] net/nfp: support push " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 17/25] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 18/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 19/25] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 20/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 21/25] net/nfp: support TP SRC " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 22/25] net/nfp: support TP DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 23/25] net/nfp: support TTL " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 24/25] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 25/25] net/nfp: support IPv6 " Chaoyong He
2022-10-18 12:27 ` [PATCH v4 00/25] add the basic rte_flow offload support of nfp PMD Ferruh Yigit
2022-10-20 2:19 ` [PATCH v5 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 01/25] net/nfp: fix CPP bridge service requirement Chaoyong He
2022-10-20 2:19 ` [PATCH v5 02/25] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-20 2:19 ` [PATCH v5 03/25] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-20 2:19 ` [PATCH v5 04/25] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-20 11:09 ` Ferruh Yigit
2022-10-20 11:42 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 05/25] net/nfp: support basic flow items Chaoyong He
2022-10-20 11:11 ` Ferruh Yigit
2022-10-20 11:43 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 06/25] net/nfp: support basic flow actions Chaoyong He
2022-10-20 11:12 ` Ferruh Yigit
2022-10-20 11:41 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 07/25] net/nfp: support VLAN flow item Chaoyong He
2022-10-20 2:19 ` [PATCH v5 08/25] net/nfp: support IPv4 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 09/25] net/nfp: support IPv6 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 10/25] net/nfp: support TCP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 11/25] net/nfp: support UDP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 12/25] net/nfp: support SCTP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 13/25] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-20 11:12 ` Ferruh Yigit
2022-10-20 11:48 ` Chaoyong He
2022-10-20 11:55 ` Ferruh Yigit
2022-10-20 2:19 ` [PATCH v5 14/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 15/25] net/nfp: support pop VLAN " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 16/25] net/nfp: support push " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 17/25] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 18/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 19/25] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 20/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 21/25] net/nfp: support TP SRC " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 22/25] net/nfp: support TP DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 23/25] net/nfp: support TTL " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 24/25] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 25/25] net/nfp: support IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 00/27] add the basic rte_flow offload support of nfp PMD Chaoyong He
2022-10-21 8:01 ` [PATCH v6 01/27] net/nfp: fix CPP bridge service requirement Chaoyong He
2022-10-21 8:01 ` [PATCH v6 02/27] net/nfp: fix the promiscuous mode control functions Chaoyong He
2022-10-21 13:03 ` Ferruh Yigit
2022-10-22 6:49 ` Chaoyong He
2022-10-22 8:11 ` Chaoyong He [this message]
2022-10-24 12:31 ` [PATCH v7] net/nfp: fix the promiscuous mode functions Ferruh Yigit
2022-10-21 8:01 ` [PATCH v6 03/27] net/nfp: fix the service stuck the app end Chaoyong He
2022-10-21 8:01 ` [PATCH v6 04/27] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-21 8:01 ` [PATCH v6 05/27] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-21 8:01 ` [PATCH v6 06/27] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-21 8:01 ` [PATCH v6 07/27] net/nfp: support basic flow items Chaoyong He
2022-10-21 8:01 ` [PATCH v6 08/27] net/nfp: support basic flow actions Chaoyong He
2022-10-21 8:01 ` [PATCH v6 09/27] net/nfp: support VLAN flow item Chaoyong He
2022-10-21 8:01 ` [PATCH v6 10/27] net/nfp: support IPv4 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 11/27] net/nfp: support IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 12/27] net/nfp: support TCP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 13/27] net/nfp: support UDP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 14/27] net/nfp: support SCTP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 15/27] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-21 8:01 ` [PATCH v6 16/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 17/27] net/nfp: support pop VLAN " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 18/27] net/nfp: support push " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 19/27] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 20/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 21/27] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 22/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 23/27] net/nfp: support TP SRC " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 24/27] net/nfp: support TP DST " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 25/27] net/nfp: support TTL " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 26/27] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 27/27] net/nfp: support IPv6 " Chaoyong He
2022-10-21 13:04 ` [PATCH v6 00/27] add the basic rte_flow offload support of nfp PMD Ferruh Yigit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1666426308-9806-1-git-send-email-chaoyong.he@corigine.com \
--to=chaoyong.he@corigine.com \
--cc=dev@dpdk.org \
--cc=niklas.soderlund@corigine.com \
--cc=oss-drivers@corigine.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).