On Wed, Nov 15, 2017 at 11:34:01AM +0200, Yishai Hadas wrote: > >>ibv_td should expose in verbs.h the 'context' as the inline function > >>ibv_alloc_td() needs it to get the verbs_context and call the driver > >>function if was set. > > > >You mean ibv_dealloc_td, OK.. > > > >Related to my other point, ibv_alloc_td should probably accept a > >ibv_pd as an argument instead. > > > > Why ibv_alloc_td() should get an ibv_pd ? do you have a typo here ? No.. Since ibv_alloc_td returns a 'ibv_pd' with additional information, it makes no sense to have an API that can return an 'ibv_pd' that is not actually a PD. So, all TD's have to be created from a PD, and the concept of a TD without a PD should be entirely removed from the API. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
