From: Prashant Gupta <prashant.gupta_3@nxp.com>
To: stephen@networkplumber.org, dev@dpdk.org
Subject: [PATCH 42/45] net/dpaa2: read MC version from device private data
Date: Thu, 3 Sep 2026 19:23:50 +0530 [thread overview]
Message-ID: <20260903135353.3358303-43-prashant.gupta_3@nxp.com> (raw)
In-Reply-To: <20260903135353.3358303-1-prashant.gupta_3@nxp.com>
The flow engine reached back to the bus device to read the MC firmware
version. The bus_info pointer behind it is only attached by the primary
process, so a secondary process that creates a flow dereferences a
pointer it never set up.
The version is already cached in priv->mc_rev at probe time and the rest
of the driver reads it from there, so use that here too. This removes the
last user of DPAA2_DEV_PRIV_TO_DPAA2_DEV(), so drop the macro.
Signed-off-by: Prashant Gupta <prashant.gupta_3@nxp.com>
---
drivers/net/dpaa2/dpaa2_ethdev.h | 4 ---
drivers/net/dpaa2/dpaa2_flow.c | 59 ++++++++++----------------------
2 files changed, 18 insertions(+), 45 deletions(-)
diff --git a/drivers/net/dpaa2/dpaa2_ethdev.h b/drivers/net/dpaa2/dpaa2_ethdev.h
index 7d93b75be3..e02fa9a622 100644
--- a/drivers/net/dpaa2/dpaa2_ethdev.h
+++ b/drivers/net/dpaa2/dpaa2_ethdev.h
@@ -43,10 +43,6 @@
#define MAX_DPNI 8
#define DPAA2_MAX_CHANNELS 16
-#define DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv) \
- container_of((((struct dpaa2_dev_priv *)priv)->eth_dev->device), \
- struct rte_dpaa2_device, device)
-
#define DPAA2_EXTRACT_PARAM_MAX_SIZE \
RTE_ALIGN(sizeof(struct dpni_ext_set_rx_tc_dist), 256)
diff --git a/drivers/net/dpaa2/dpaa2_flow.c b/drivers/net/dpaa2/dpaa2_flow.c
index 8456b3b80d..187381c5dd 100644
--- a/drivers/net/dpaa2/dpaa2_flow.c
+++ b/drivers/net/dpaa2/dpaa2_flow.c
@@ -781,13 +781,10 @@ static int
dpaa2_flow_add_fs_rule(struct dpaa2_dev_priv *priv,
struct dpaa2_generic_flow *flow)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct dpaa2_flow_tbl_profile *tbl_profile;
int ret;
struct fsl_mc_io *dpni = priv->hw;
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
-
tbl_profile = &priv->flow_profile.tc_profile[flow->tc_id];
if (dpaa2_flow_entry_map_get(tbl_profile->entry_map,
flow->entry_index)) {
@@ -798,7 +795,7 @@ dpaa2_flow_add_fs_rule(struct dpaa2_dev_priv *priv,
dpaa2_flow_fs_entry_log("Add", flow);
- if (dpaa2_dev->bus_info->mc_rev >= DPAA2_FLOW_FRM_REPLICATION_ACTION_MC_REV) {
+ if (priv->mc_rev >= DPAA2_FLOW_FRM_REPLICATION_ACTION_MC_REV) {
ret = dpni_add_fs_entry(dpni, CMD_PRI_LOW,
priv->token, flow->tc_id, flow->entry_index,
&flow->rule_cfg, &flow->flow_action.fs_action.fs_action_cfg);
@@ -3642,7 +3639,6 @@ dpaa2_flow_fs_action_config(struct dpaa2_dev_priv *priv,
struct dpaa2_generic_flow *flow,
const struct rte_flow_action *rte_action)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct rte_eth_dev *dest_dev;
struct dpaa2_dev_priv *dest_priv;
const struct rte_flow_action_queue *dest_queue;
@@ -3651,11 +3647,9 @@ dpaa2_flow_fs_action_config(struct dpaa2_dev_priv *priv,
uint16_t num_tokens;
struct dpaa2_dev_flow_fs_action *fs_action;
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
-
fs_action = &flow->flow_action.fs_action;
if (fs_action->action_type != RTE_FLOW_ACTION_TYPE_END) {
- if (dpaa2_dev->bus_info->mc_rev >= DPAA2_FLOW_FRM_REPLICATION_ACTION_MC_REV &&
+ if (priv->mc_rev >= DPAA2_FLOW_FRM_REPLICATION_ACTION_MC_REV &&
(fs_action->action_type == RTE_FLOW_ACTION_TYPE_PORT_ID ||
fs_action->action_type == RTE_FLOW_ACTION_TYPE_PORT_REPRESENTOR)) {
if (rte_action->type != RTE_FLOW_ACTION_TYPE_PORT_ID &&
@@ -3807,21 +3801,19 @@ dpaa2_flow_fs_table_set_default(struct dpaa2_dev_priv *priv,
uint8_t tc_id, int discard, uint16_t default_queue)
{
int ret;
- struct rte_dpaa2_device *dpaa2_dev;
struct dpni_rx_dist_cfg *tc_cfg;
struct fsl_mc_io *dpni = priv->hw;
struct dpaa2_flow_tbl_profile *tbl_profile;
struct dpaa2_queue *queue;
char mc_rev[1024];
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
tbl_profile = &priv->flow_profile.tc_profile[tc_id];
snprintf(mc_rev, 1024, "MC rev(%d.%d.%d)",
- RTE_FSL_MC_REV_MAJOR(dpaa2_dev->bus_info->mc_rev),
- RTE_FSL_MC_REV_MINOR(dpaa2_dev->bus_info->mc_rev),
- RTE_FSL_MC_REV_REVISION(dpaa2_dev->bus_info->mc_rev));
+ RTE_FSL_MC_REV_MAJOR(priv->mc_rev),
+ RTE_FSL_MC_REV_MINOR(priv->mc_rev),
+ RTE_FSL_MC_REV_REVISION(priv->mc_rev));
if (!tbl_profile->dpkg.num_extracts &&
- dpaa2_dev->bus_info->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV) {
+ priv->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV) {
DPAA2_PMD_DEBUG("%s can't set miss action of FS table indepentently.",
mc_rev);
return 0;
@@ -3946,26 +3938,24 @@ dpaa2_flow_qos_table_set_default(struct dpaa2_dev_priv *priv,
int discard, uint8_t default_tc, uint16_t default_flow)
{
int ret;
- struct rte_dpaa2_device *dpaa2_dev;
struct dpni_qos_tbl_cfg qos_cfg;
struct fsl_mc_io *dpni = priv->hw;
struct dpaa2_flow_tbl_profile *tbl_profile;
char mc_rev[1024];
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
tbl_profile = &priv->flow_profile.qos_profile;
snprintf(mc_rev, 1024, "MC rev(%d.%d.%d)",
- RTE_FSL_MC_REV_MAJOR(dpaa2_dev->bus_info->mc_rev),
- RTE_FSL_MC_REV_MINOR(dpaa2_dev->bus_info->mc_rev),
- RTE_FSL_MC_REV_REVISION(dpaa2_dev->bus_info->mc_rev));
+ RTE_FSL_MC_REV_MAJOR(priv->mc_rev),
+ RTE_FSL_MC_REV_MINOR(priv->mc_rev),
+ RTE_FSL_MC_REV_REVISION(priv->mc_rev));
if (!tbl_profile->dpkg.num_extracts &&
- dpaa2_dev->bus_info->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV) {
+ priv->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV) {
DPAA2_PMD_DEBUG("%s can't set miss action of QoS table indepentently.",
mc_rev);
return 0;
}
if (default_flow < priv->dist_queues &&
- dpaa2_dev->bus_info->mc_rev < DPAA2_QOS_FLOW_TABLE_MISS_FLOW_ACTION_MC_REV) {
+ priv->mc_rev < DPAA2_QOS_FLOW_TABLE_MISS_FLOW_ACTION_MC_REV) {
DPAA2_PMD_WARN("%s can't direct miss traffic to TC%d-flow%d by QoS table only.",
mc_rev, default_tc, default_flow);
return 0;
@@ -4002,7 +3992,6 @@ static int
dpaa2_flow_qos_table_config(struct dpaa2_dev_priv *priv,
int rss_dist)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct dpaa2_flow_tbl_profile *tbl_profile;
uint8_t *key_cfg_buf;
int ret;
@@ -4064,8 +4053,7 @@ dpaa2_flow_qos_table_config(struct dpaa2_dev_priv *priv,
qos_cfg->discard_on_miss = false;
}
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
- if (dpaa2_dev->bus_info->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV)
+ if (priv->mc_rev < DPAA2_QOS_FLOW_TABLE_SET_V3_MC_REV)
ret = dpni_set_qos_table_v2(dpni, CMD_PRI_LOW, priv->token, qos_cfg);
else
ret = dpni_set_qos_table(dpni, CMD_PRI_LOW, priv->token, qos_cfg);
@@ -4209,7 +4197,6 @@ static int
dpaa2_flow_set_police_action(struct dpaa2_dev_priv *priv,
uint8_t tc_id, const struct rte_flow_action_meter_mark *meter_mark)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct dpni_rx_tc_policing_cfg policing_cfg = {0};
const struct dpaa2_dev_meter_profile *dpaa2_profile;
const struct dpaa2_dev_meter_policy *dpaa2_policy = NULL;
@@ -4221,8 +4208,6 @@ dpaa2_flow_set_police_action(struct dpaa2_dev_priv *priv,
return -ENOTSUP;
}
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
-
dpaa2_profile = (void *)meter_mark->profile;
if (!dpaa2_profile) {
DPAA2_PMD_ERR("Meter profile not specified!");
@@ -4244,9 +4229,8 @@ dpaa2_flow_set_police_action(struct dpaa2_dev_priv *priv,
if (priv->flow_profile.mtr_flow[tc_id]) {
/** Update existing policer.*/
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
policing_cfg.options |= DPNI_POLICER_OPT_DO_NOT_RESET_COUNTERS;
- if (dpaa2_dev->bus_info->mc_rev < DPAA2_POLICER_NOT_RESET_COUNTER_MC_REV)
+ if (priv->mc_rev < DPAA2_POLICER_NOT_RESET_COUNTER_MC_REV)
DPAA2_PMD_WARN("The existing policer's counters will be cleaned.");
}
@@ -4269,7 +4253,7 @@ dpaa2_flow_set_police_action(struct dpaa2_dev_priv *priv,
policing_cfg.eir = dpaa2_profile->pir;
policing_cfg.ebs = dpaa2_profile->pbs;
- if (dpaa2_dev->bus_info->mc_rev < DPAA2_POLICER_SET_V2_MC_REV) {
+ if (priv->mc_rev < DPAA2_POLICER_SET_V2_MC_REV) {
ret = dpni_set_rx_tc_policing_v1(priv->hw, CMD_PRI_LOW,
priv->token, tc_id, &policing_cfg);
} else {
@@ -5400,7 +5384,6 @@ dpaa2_flow_generic_flow_create(struct rte_eth_dev *dev,
int mix_extract, int is_rss)
{
struct dpaa2_generic_flow *flow = NULL;
- struct rte_dpaa2_device *dpaa2_dev;
struct dpaa2_dev_priv *priv = dev->data->dev_private;
int ret, idx = -1;
uint64_t iova, mc_rev;
@@ -5416,8 +5399,7 @@ dpaa2_flow_generic_flow_create(struct rte_eth_dev *dev,
if (type != DPAA2_FLOW_QOS_TYPE && type != DPAA2_FLOW_FS_TYPE)
return NULL;
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
- mc_rev = dpaa2_dev->bus_info->mc_rev;
+ mc_rev = priv->mc_rev;
if (actions) {
ret = dpaa2_flow_qos_fs_action_set(actions, qos_actions,
@@ -5912,19 +5894,17 @@ static int
dpaa2_flow_destroy_meter_flow(struct rte_eth_dev *dev,
struct dpaa2_dev_flow *flow)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct dpaa2_dev_priv *priv = dev->data->dev_private;
struct dpni_rx_tc_policing_cfg cfg;
int ret;
uint8_t tc_id;
RTE_ASSERT(!flow->qos_flow && flow->fs_flow);
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
tc_id = flow->fs_flow->tc_id;
RTE_ASSERT(priv->flow_profile.mtr_flow[tc_id] == flow);
memset(&cfg, 0, sizeof(cfg));
cfg.mode = DPNI_POLICER_MODE_NONE;
- if (dpaa2_dev->bus_info->mc_rev < DPAA2_POLICER_SET_V2_MC_REV) {
+ if (priv->mc_rev < DPAA2_POLICER_SET_V2_MC_REV) {
ret = dpni_set_rx_tc_policing_v1(priv->hw, CMD_PRI_LOW,
priv->token, tc_id, &cfg);
} else {
@@ -6141,7 +6121,6 @@ dpaa2_flow_actions_update(struct rte_eth_dev *dev,
const struct rte_flow_action actions[],
struct rte_flow_error *error)
{
- struct rte_dpaa2_device *dpaa2_dev;
struct dpaa2_dev_priv *priv = dev->data->dev_private;
struct dpaa2_dev_flow *flow;
struct dpaa2_flow_tbl_profile *tbl_profile = NULL;
@@ -6162,8 +6141,6 @@ dpaa2_flow_actions_update(struct rte_eth_dev *dev,
const struct rte_flow_action_queue *action_q;
struct dpaa2_queue *rxq = NULL;
- dpaa2_dev = DPAA2_DEV_PRIV_TO_DPAA2_DEV(priv);
-
/* check for the valid flow */
flow = (void *)_flow;
if (flow->is_meter_flow) {
@@ -6275,7 +6252,7 @@ dpaa2_flow_actions_update(struct rte_eth_dev *dev,
if (fs_action_num > 0 &&
fs_actions[0].type != RTE_FLOW_ACTION_TYPE_PORT_ID &&
fs_actions[0].type != RTE_FLOW_ACTION_TYPE_REPRESENTED_PORT &&
- dpaa2_dev->bus_info->mc_rev >= DPAA2_FS_FLOW_HW_ACTION_UPDATE_MC_REV) {
+ priv->mc_rev >= DPAA2_FS_FLOW_HW_ACTION_UPDATE_MC_REV) {
/** Action HW update doesn't support redirecting frames to other DPNIs.*/
hw_update = true;
goto skip_remove_fs_entry;
@@ -6365,7 +6342,7 @@ dpaa2_flow_actions_update(struct rte_eth_dev *dev,
}
hw_update = false;
- if (dpaa2_dev->bus_info->mc_rev >= DPAA2_QOS_FLOW_HW_ACTION_UPDATE_MC_REV) {
+ if (priv->mc_rev >= DPAA2_QOS_FLOW_HW_ACTION_UPDATE_MC_REV) {
/** Action HW update doesn't support redirecting frames to other DPNIs.*/
hw_update = true;
goto skip_remove_qos_entry;
--
2.43.0
next prev parent reply other threads:[~2026-09-03 13:59 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-03 13:53 [PATCH 00/45] net/dpaa2: features and fixes for NXP DPAA2 drivers Prashant Gupta
2026-09-03 13:53 ` [PATCH 01/45] crypto/dpaa2_sec: fix buffer overflow in GCM decrypt Prashant Gupta
2026-09-03 13:53 ` [PATCH 02/45] crypto/dpaa2_sec: fix FLE pool leak on sec FD build failure Prashant Gupta
2026-09-03 13:53 ` [PATCH 03/45] crypto/dpaa2_sec: support AES-GMAC Prashant Gupta
2026-09-03 13:53 ` [PATCH 04/45] crypto/dpaa2_sec: increase ivsize range for AES-CTR Prashant Gupta
2026-09-03 13:53 ` [PATCH 05/45] crypto/dpaa2_sec: add missing ECN capability Prashant Gupta
2026-09-03 13:53 ` [PATCH 06/45] crypto/dpaa2_sec: add support for env variables Prashant Gupta
2026-09-03 13:53 ` [PATCH 07/45] drivers: fix double free of dpaa2 device on uninit Prashant Gupta
2026-09-03 14:05 ` David Marchand
2026-09-03 13:53 ` [PATCH 08/45] net/dpaa2: fix integer overflow in CCSR region mapping Prashant Gupta
2026-09-03 13:53 ` [PATCH 09/45] dma/dpaa2: fix array-bounds warning in dequeue path Prashant Gupta
2026-09-03 13:53 ` [PATCH 10/45] bus/fslmc: defer bus initialization to probe Prashant Gupta
2026-09-03 13:53 ` [PATCH 11/45] dma/dpaa2: validate IOVA in pre-populate helpers Prashant Gupta
2026-09-03 13:53 ` [PATCH 12/45] dma/dpaa2: optimize context index ring enqueue Prashant Gupta
2026-09-03 13:53 ` [PATCH 13/45] drivers: add dpaa2 DMA bypass memory translation option Prashant Gupta
2026-09-03 13:53 ` [PATCH 14/45] mempool/dpaa2: support ops index from primary in secondary Prashant Gupta
2026-09-03 13:53 ` [PATCH 15/45] net/dpaa2: set Tx confirmation on device init Prashant Gupta
2026-09-03 13:53 ` [PATCH 16/45] drivers: optimize dpaa2 Tx queue and channel mapping Prashant Gupta
2026-09-03 13:53 ` [PATCH 17/45] net/dpaa2: support larger burst size Prashant Gupta
2026-09-03 13:53 ` [PATCH 18/45] net/dpaa2: support MPLS and PPPoE flow distribution Prashant Gupta
2026-09-03 13:53 ` [PATCH 19/45] net/dpaa2: support meter and policing Prashant Gupta
2026-09-03 13:53 ` [PATCH 20/45] net/dpaa2: support flow drop action Prashant Gupta
2026-09-03 13:53 ` [PATCH 21/45] net/dpaa2: set default flow miss action per device Prashant Gupta
2026-09-03 13:53 ` [PATCH 22/45] net/dpaa2: identify Rx mbuf hash information by FLC Prashant Gupta
2026-09-03 13:53 ` [PATCH 23/45] net/dpaa2: add minimum key size support Prashant Gupta
2026-09-03 13:53 ` [PATCH 24/45] net/dpaa2: restructure dpaa2 parser processing Prashant Gupta
2026-09-03 13:53 ` [PATCH 25/45] net/dpaa2: parse tunnel and fragmented packet types Prashant Gupta
2026-09-03 13:53 ` [PATCH 26/45] net/dpaa2: remove unused soft parser driver Prashant Gupta
2026-09-03 13:53 ` [PATCH 27/45] drivers: refresh dpaa2 MC and SoC version info Prashant Gupta
2026-09-03 13:53 ` [PATCH 28/45] drivers: identify dpaa2 soft parser protocol Prashant Gupta
2026-09-03 13:53 ` [PATCH 29/45] drivers: assign dpaa2 Rx CGID per traffic class Prashant Gupta
2026-09-03 13:53 ` [PATCH 30/45] drivers: inherit dpaa2 rxq config for event queue Prashant Gupta
2026-09-03 13:53 ` [PATCH 31/45] net/dpaa2: rename Rx queue flags Prashant Gupta
2026-09-03 13:53 ` [PATCH 32/45] drivers: rework dpaa2 Tx confirmation Prashant Gupta
2026-09-03 13:53 ` [PATCH 33/45] net/dpaa2: ptp enhancements Prashant Gupta
2026-09-03 13:53 ` [PATCH 34/45] net/dpaa2: remove unused soft parser Tx code Prashant Gupta
2026-09-03 13:53 ` [PATCH 35/45] net/dpaa2: update MC dpni QoS and flow steering API Prashant Gupta
2026-09-03 13:53 ` [PATCH 36/45] net/dpaa2: enhance xstat implementation Prashant Gupta
2026-09-03 13:53 ` [PATCH 37/45] net/dpaa2: rework flow engine Prashant Gupta
2026-09-03 13:53 ` [PATCH 38/45] net/dpaa2: support Rx mempool per traffic class Prashant Gupta
2026-09-03 13:53 ` [PATCH 39/45] drivers: consume dpaa2 DQRR entries in batches Prashant Gupta
2026-09-03 13:53 ` [PATCH 40/45] drivers: resolve dpaa2 endpoint in the net driver Prashant Gupta
2026-09-03 13:53 ` [PATCH 41/45] drivers: align dpaa2 event port depths with hardware rings Prashant Gupta
2026-09-03 13:53 ` Prashant Gupta [this message]
2026-09-03 13:53 ` [PATCH 43/45] net/dpaa2: do not overwrite mbuf hash with drop priority Prashant Gupta
2026-09-03 13:53 ` [PATCH 44/45] bus/fslmc: reduce probe-time logging and MC traffic Prashant Gupta
2026-09-03 13:53 ` [PATCH 45/45] net/dpaa2: reject Rx queue deferred start Prashant Gupta
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=20260903135353.3358303-43-prashant.gupta_3@nxp.com \
--to=prashant.gupta_3@nxp.com \
--cc=dev@dpdk.org \
--cc=stephen@networkplumber.org \
/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