This is the first of several BTL interface changes I am proposing for the 1.9/2.0 release series.
What: Change naming of btl descriptor members. I propose we change des_src and des_dst (and their associated counts) to be des_local and des_remote. For receive callbacks the des_local member will be used to communicate the segment information to the callback. The proposed change will include updating all of the doxygen in btl.h as well as updating all BTLs and BTL users to use the new naming scheme. Why: My btl usage makes use of both put and get operations on the same descriptor. With the current naming scheme I need to ensure that there is consistency beteen the segments described in des_src and des_dst depending on whether a put or get operation is executed. Additionally, the current naming prevents BTLs that do not require prepare/RMA matched operations (do not set MCA_BTL_FLAGS_RDMA_MATCHED) from executing multiple simultaneous put AND get operations. At the moment the descriptor can only be used with one or the other. The naming change makes it easier for BTL users to setup/modify descriptors for RMA operations as the local segment and remote segment are always in the same member field. The only issue I forsee with this change is that it will require a little more work to move BTL fixes to the 1.8 release series. When: I received feedback on this change last week at the developer's meeting. I want to keep the timeout short as I intend to move the enhance one-sided into the trunk by mid-August. Timeout set for a week from today. Feedback on this proposal is greatly appreciated. -Nathan Hjelm Application Readiness, HPC-5, LANL
pgp5VeqMnGZW1.pgp
Description: PGP signature
