On Sun, Nov 12, 2017 at 11:41:39PM +0200, Yishai Hadas wrote: > +struct ibv_td { > + struct ibv_context *context; > +}; As much as possible, I would like to see any new objects be 'opaque' to the application, so this should just be struct ibv_td; And ibv_td should be defined in driver.h or something This avoids leaking internal details and means we don't have to commit to an ABI for the insides of these structs. It was a mistake that so many structs were entirely exposed in the original verbs :( 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