Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mlx5: Add RDMA TRANSPORT steering domain #1537

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yishaih
Copy link
Member

@yishaih yishaih commented Jan 2, 2025

This series extends mlx5dv_create_flow_matcher() API to support two new flow table types:
RDMA_TRANSPORT_RX and RDMA_TRANSPORT_TX.

Packets will traverse through RDMA_TRANSPORT_RX after RDMA_RX and through RDMA_TRANSPORT_TX before RDMA_TX, ensuring proper control and management.

Further details exist as part of the commit log and the man page.

The matching kernel series was sent already to rdma-next.

Yishai Hadas and others added 2 commits January 2, 2025 15:35
To commit: a7cdd04ad2bf ("RDMA/mlx5: Expose RDMA TRANSPORT flow table types to userspace").

Signed-off-by: Yishai Hadas <[email protected]>
…ypes

Extend mlx5dv_create_flow_matcher() verb to support two new flow table
types: RDMA_TRANSPORT_RX and RDMA_TRANSPORT_TX.

These new domains are created on the vport group manager for each vport.
Hence, the ib_port argument is needed when creating such a matcher to
specify its corresponding port.

In switchdev mode the ib_port argument is used to denote the vport on
which to create the matcher via the ib_port representor.
Whereas in legacy/baremetal mode ib_port=1 must be provided which
indicates to create on your own domain.

Packets will traverse through RDMA_TRANSPORT_RX after RDMA_RX and
through RDMA_TRANSPORT_TX before RDMA_TX, ensuring proper control and
management.

Signed-off-by: Patrisious Haddad <[email protected]>
Reviewed-by: Mark Bloch <[email protected]>
Signed-off-by: Yishai Hadas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants