We talked about this on the phone yesterday.
Brian, Jeff, Terry, and Rolf were there. George was able to join for a short
period of time.
George's objections seemed to be twofold (please correct if wrong!):
1. Why not change all 3 function pointers to take an additional (void*)?
George misun
George --
Sure. Since I had talked to you and Jeff about it a year ago (when you
added the callback) and you didn't complain, I assumed you two would be the
only ones to care and wouldn't complain this time. Guess I should have
known better :).
Brian
On 11/27/09 18:24 , "George Bosilca" wrot
Brian,
This is a pretty big change to be done with a so short notice, especially over
the Thanksgiving weekend. I do have a lots of concerns about this approach, but
I lack the time to expand on this right now. I'll be back at work on Monday and
I'll give detailed informations. Please delay the
WHAT: Add a void* extra_state field to ompi_request_t
WHY: When we added the req_complete_cb field so that internal pieces of OMPI
who generated requests (such as the OSC components using the PML) could be
async notified when the request completed (ie, the PML request the OSC
component had initiat