Skip to content

Conversation

@pakmarkthub
Copy link
Contributor

Prerequisite:

Problem:

  • exp-verbs is being used throughout the entire project. In order to prepare for moving to DirectVerbs / DevX, we would need to consolidate the use of exp-verbs and make gds api more generic.

This PR:

  • introduces src/mlx5-exp.cpp and src/mlx5-exp.hpp to contain exp-verbs related functions / structures.
  • moves gds_create_cq and gds_create_qp, which use exp-verbs, to mlx5-exp.cpp.
  • changes the typedef of gds_qp_init_attr_t and gds_send_wr to ib-verbs. We actually don't need exp-verbs there.
  • modifies gds_kernel_latency and gds_kernel_loopback_latency to support the new data structures.

Presubmit testing:

  • compile and run gds_kernel_latency and gds_kernel_loopback_latency on Summit.

@pakmarkthub pakmarkthub requested a review from drossetti August 6, 2021 06:23
@pakmarkthub pakmarkthub changed the base branch from master to devel-dv August 6, 2021 06:23
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