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

tests: Extend mlx5 flow steering coverage #1585

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

Conversation

EdwardSro
Copy link
Contributor

Extend mlx5 flow steering tests, mainly covering newly added RDMA Transport steering domain support (PR #1537 )

dhayon and others added 5 commits March 20, 2025 17:39
Add support for the ib_port attribute, this attribute is needed when
creating a matcher over vport to specify its corresponding port.
Add ft_type and ib_port enums to support the attribute comp mask.
Add support for the RDMA transport domain.

Signed-off-by: dhayon <[email protected]>
Signed-off-by: Edward Srouji <[email protected]>
Add pyverb to support MLX5DV_FLOW_ACTION_DEST_DEVX flow action.

Signed-off-by: Linoy Ganti <[email protected]>
Signed-off-by: dhayon <[email protected]>
Signed-off-by: Edward Srouji <[email protected]>
Renamed the class from FlowTableEntryMatchParam to
FlowTableEntryMatchParamSW, to better reflect the differentiation in
parameter structures used in SW steering.

Signed-off-by: dhayon <[email protected]>
Signed-off-by: Edward Srouji <[email protected]>
Add a test that verifies the creation of the RDMA transport domain
and ensures the rule on this table is correctly applied.
Updated `tests/mlx5_prm_structs.py` to include the necessary
PRM structure definitions for the RDMA transport domain.

Signed-off-by: dhayo <[email protected]>
Signed-off-by: Edward Srouji <[email protected]>
Add test that creates DevX objects for RDMA RX and TX domains.
This test can be run on IB and ETH.

Signed-off-by: Linoy Ganti <[email protected]>
Signed-off-by: dhayon <[email protected]>
Signed-off-by: Edward Srouji <[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.

1 participant