This patch adds the following internal functions for creation of
unicast DMAC flow rules:
- mlx5_legacy_dmac_flow_create() - simple wrapper over
mlx5_ctrl_flow().
- mlx5_legacy_dmac_vlan_flow_create() - simple wrapper over
mlx5_ctrl_flow_vlan().
These will be used as a basis for implementing dynamic
additions of unicast DMAC or unicast DMAC with VLAN
control flow rules when new addresses/VLANs are added.
Also, this path adds the following internal functions
for destructions of unicast DMAC flow rules:
- mlx5_legacy_ctrl_flow_destroy() - assuming a flow rule is on the
control flow rule list, destroy it.
- mlx5_legacy_dmac_flow_destroy() - find and destroy a flow rule
with given unicast DMAC.
- mlx5_legacy_dmac_flow_destroy() - find and destroy a flow rule
with given unicast DMAC and VLAN ID.
These will be used as a basis for implementing dynamic
removals of unicast DMAC or unicast DMAC with VLAN
control flow rules when addresses/VLANs are removed.
At the moment, no relevant flow rules are registered on the list
when working with Verbs or DV flow engine.
This will be added in the follow up commit.
Signed-off-by: Dariusz Sosnowski <dsosnow...@nvidia.com>
---
drivers/net/mlx5/mlx5_flow.c | 80 ++++++++++++++++++++++++++++++++++++
drivers/net/mlx5/mlx5_flow.h | 19 +++++++++
2 files changed, 99 insertions(+)
diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index af79956eaa..463edae70e 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -8534,6 +8534,86 @@ mlx5_ctrl_flow(struct rte_eth_dev *dev,
return mlx5_ctrl_flow_vlan(dev, eth_spec, eth_mask, NULL, NULL);
}
+int
+mlx5_legacy_dmac_flow_create(struct rte_eth_dev *dev, const struct
rte_ether_addr *addr)
+{
+ struct rte_flow_item_eth unicast = {
+ .hdr.dst_addr = *addr,
+ };
+ struct rte_flow_item_eth unicast_mask = {
+ .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+ };
+
+ return mlx5_ctrl_flow(dev, &unicast, &unicast_mask);
+}
+
+int
+mlx5_legacy_dmac_vlan_flow_create(struct rte_eth_dev *dev,
+ const struct rte_ether_addr *addr,
+ const uint16_t vid)
+{
+ struct rte_flow_item_eth unicast_spec = {
+ .hdr.dst_addr = *addr,
+ };
+ struct rte_flow_item_eth unicast_mask = {
+ .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+ };
+ struct rte_flow_item_vlan vlan_spec = {
+ .hdr.vlan_tci = rte_cpu_to_be_16(vid),
+ };
+ struct rte_flow_item_vlan vlan_mask = rte_flow_item_vlan_mask;
+
+ return mlx5_ctrl_flow_vlan(dev, &unicast_spec, &unicast_mask,
&vlan_spec, &vlan_mask);
+}
+
+void
+mlx5_legacy_ctrl_flow_destroy(struct rte_eth_dev *dev, struct
mlx5_ctrl_flow_entry *entry)
+{
+ uintptr_t flow_idx;
+
+ flow_idx = (uintptr_t)entry->flow;
+ mlx5_flow_list_destroy(dev, MLX5_FLOW_TYPE_CTL, flow_idx);
+ LIST_REMOVE(entry, next);
+ mlx5_free(entry);
+}
+
+int
+mlx5_legacy_dmac_flow_destroy(struct rte_eth_dev *dev, const struct
rte_ether_addr *addr)
+{
+ struct mlx5_priv *priv = dev->data->dev_private;
+ struct mlx5_ctrl_flow_entry *entry;
+
+ LIST_FOREACH(entry, &priv->hw_ctrl_flows, next) {
+ if (entry->info.type !=
MLX5_CTRL_FLOW_TYPE_DEFAULT_RX_RSS_UNICAST_DMAC ||
+ !rte_is_same_ether_addr(addr, &entry->info.uc.dmac))
+ continue;
+
+ mlx5_legacy_ctrl_flow_destroy(dev, entry);
+ return 0;
+ }
+ return 0;
+}
+
+int
+mlx5_legacy_dmac_vlan_flow_destroy(struct rte_eth_dev *dev,
+ const struct rte_ether_addr *addr,
+ const uint16_t vid)
+{
+ struct mlx5_priv *priv = dev->data->dev_private;
+ struct mlx5_ctrl_flow_entry *entry;
+
+ LIST_FOREACH(entry, &priv->hw_ctrl_flows, next) {
+ if (entry->info.type !=
MLX5_CTRL_FLOW_TYPE_DEFAULT_RX_RSS_UNICAST_DMAC_VLAN ||
+ !rte_is_same_ether_addr(addr, &entry->info.uc.dmac) ||
+ vid != entry->info.uc.vlan)
+ continue;
+
+ mlx5_legacy_ctrl_flow_destroy(dev, entry);
+ return 0;
+ }
+ return 0;
+}
+
/**
* Create default miss flow rule matching lacp traffic
*
diff --git a/drivers/net/mlx5/mlx5_flow.h b/drivers/net/mlx5/mlx5_flow.h
index 165d17e40a..db56ae051d 100644
--- a/drivers/net/mlx5/mlx5_flow.h
+++ b/drivers/net/mlx5/mlx5_flow.h
@@ -2991,6 +2991,25 @@ struct mlx5_flow_hw_ctrl_fdb {
int mlx5_flow_hw_ctrl_flows(struct rte_eth_dev *dev, uint32_t flags);
+/** Create a control flow rule for matching unicast DMAC with VLAN (Verbs and
DV). */
+int mlx5_legacy_dmac_flow_create(struct rte_eth_dev *dev, const struct
rte_ether_addr *addr);
+
+/** Destroy a control flow rule for matching unicast DMAC with VLAN (Verbs and
DV). */
+int mlx5_legacy_dmac_flow_destroy(struct rte_eth_dev *dev, const struct
rte_ether_addr *addr);
+
+/** Create a control flow rule for matching unicast DMAC with VLAN (Verbs and
DV). */
+int mlx5_legacy_dmac_vlan_flow_create(struct rte_eth_dev *dev,
+ const struct rte_ether_addr *addr,
+ const uint16_t vid);
+
+/** Destroy a control flow rule for matching unicast DMAC with VLAN (Verbs and
DV). */
+int mlx5_legacy_dmac_vlan_flow_destroy(struct rte_eth_dev *dev,
+ const struct rte_ether_addr *addr,
+ const uint16_t vid);
+
+/** Destroy a control flow rule registered on port level control flow rule
type. */
+void mlx5_legacy_ctrl_flow_destroy(struct rte_eth_dev *dev, struct
mlx5_ctrl_flow_entry *entry);
+
/** Create a control flow rule for matching unicast DMAC (HWS). */
int mlx5_flow_hw_ctrl_flow_dmac(struct rte_eth_dev *dev, const struct
rte_ether_addr *addr);
--
2.39.5
