-
Notifications
You must be signed in to change notification settings - Fork 2
Add transport connection ID accessors #88
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -448,16 +448,16 @@ void bind_config_types(py::module_ &m) { | |||||||||||||||||||||||||||||
| .def(py::init<>()) | ||||||||||||||||||||||||||||||
| .def_readwrite("hdr", &BurstParams::hdr) | ||||||||||||||||||||||||||||||
| .def_property( | ||||||||||||||||||||||||||||||
| "rdma_conn_id", | ||||||||||||||||||||||||||||||
| [](const BurstParams &burst) { return burst.rdma_hdr.conn_id; }, | ||||||||||||||||||||||||||||||
| "connection_id", | ||||||||||||||||||||||||||||||
| [](const BurstParams &burst) { return get_connection_id(&burst); }, | ||||||||||||||||||||||||||||||
| [](BurstParams &burst, uintptr_t conn_id) { | ||||||||||||||||||||||||||||||
| burst.rdma_hdr.conn_id = conn_id; | ||||||||||||||||||||||||||||||
| set_connection_id(&burst, conn_id); | ||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||
| .def_property( | ||||||||||||||||||||||||||||||
| "rdma_wr_id", | ||||||||||||||||||||||||||||||
| [](const BurstParams &burst) { return burst.rdma_hdr.wr_id; }, | ||||||||||||||||||||||||||||||
| [](const BurstParams &burst) { return burst.transport_hdr.wr_id; }, | ||||||||||||||||||||||||||||||
| [](BurstParams &burst, uint64_t wr_id) { | ||||||||||||||||||||||||||||||
| burst.rdma_hdr.wr_id = wr_id; | ||||||||||||||||||||||||||||||
| burst.transport_hdr.wr_id = wr_id; | ||||||||||||||||||||||||||||||
| }); | ||||||||||||||||||||||||||||||
|
Comment on lines
456
to
461
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| py::class_<RDMAConfig>(m, "RDMAConfig") | ||||||||||||||||||||||||||||||
|
|
@@ -728,6 +728,8 @@ PYBIND11_MODULE(_daqiri, m) { | |||||||||||||||||||||||||||||
| m.def("set_num_packets", &set_num_packets, "burst"_a, "num"_a); | ||||||||||||||||||||||||||||||
| m.def("get_num_packets", &get_num_packets, "burst"_a); | ||||||||||||||||||||||||||||||
| m.def("get_q_id", &get_q_id, "burst"_a); | ||||||||||||||||||||||||||||||
| m.def("set_connection_id", &set_connection_id, "burst"_a, "conn_id"_a); | ||||||||||||||||||||||||||||||
| m.def("get_connection_id", &get_connection_id, "burst"_a); | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| m.def( | ||||||||||||||||||||||||||||||
| "get_segment_packet_ptr", | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit title is
#0 - Add transport connection ID accessors. GitHub issue numbers start at#1;#0does not correspond to a real issue. Per CONTRIBUTING.md, an approved issue must exist before coding starts. Please link to the actual tracking issue and update the commit title accordingly (or amend if needed).Rule Used: Commit titles must use imperative mood and be pref... (source)