Skip to content

Conversation

@ShacharKagan
Copy link
Contributor

Fix Addressed a segmentation fault in DV CQ creation, exposed SRQ/QP pointers for DV objects, and updated the Context return type for Cython 3.x compatibility.
This series introduces improvements to DC QP bad flow testing.
Additionally, added CX10 to the MLX5_DEVS list.

Daniel Hayon and others added 5 commits December 1, 2025 13:41
Add DC QP stream error handling tests with recovery validation:
- test_dc_stream_qp_recovery: Tests QP error state transition when
multiple stream channels accumulate errors and validates QP recovery
after reset
- test_dc_stream_ids_recovery: Tests stream ID reset functionality to
recover from individual stream errors without affecting overall QP
state

Signed-off-by: Daniel Hayon <dhayon@nvidia.com>
Signed-off-by: Shachar Kagan <skagan@nvidia.com>
Remove old DC bad flow test functions and supporting infrastructure:
- test_dc_send_stream_bad_flow() and test_dc_send_stream_bad_flow_qp()
- DCI_TEST_* flow constants and bad flow simulation functions
- Complex error tracking arrays and MR re-registration logic
- bad_flow_handler_qp(), set_bad_flow(), traffic_with_bad_flow()
  and related methods
Simplify dci_reset_stream_id() by removing stream validation logic.
Remove bad flow check from utils.py send path.

Signed-off-by: Daniel Hayon <dhayon@nvidia.com>
Signed-off-by: Shachar Kagan <skagan@nvidia.com>
Change Context object return type to be compatible with Cython 3.x.

Signed-off-by: Daria Velikovsky <daria@nvidia.com>
Signed-off-by: Shachar Kagan <skagan@nvidia.com>
- Correctly cast the CQ pointer when creating a DV CQ from a CQ to
  prevent a segmentation fault.
- Expose SRQ and QP pointers from their classes to allow creating
  corresponding DV objects.

Signed-off-by: elyashivc <elyashivc@nvidia.com>
Signed-off-by: Shachar Kagan <skagan@nvidia.com>
Add CX10 to MLX5_DEVS list.

Signed-off-by: Mazal Or Yakuel <moyakuel@nvidia.com>
Signed-off-by: Shachar Kagan <skagan@nvidia.com>
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