Re: [OMPI devel] RFC: Add extra_state field to ompi_request_t

2009-12-11 Thread Jeff Squyres
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

Re: [OMPI devel] RFC: Add extra_state field to ompi_request_t

2009-11-29 Thread Barrett, Brian W
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

Re: [OMPI devel] RFC: Add extra_state field to ompi_request_t

2009-11-27 Thread George Bosilca
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

[OMPI devel] RFC: Add extra_state field to ompi_request_t

2009-11-25 Thread Barrett, Brian W
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