On Thu, Dec 01, 2016 at 06:30:05PM +0200, Yishai Hadas wrote: > >> int ibv_cmd_create_ah(struct ibv_pd *pd, struct ibv_ah *ah, > >>- struct ibv_ah_attr *attr) > >>+ struct ibv_ah_attr *attr, > >>+ struct ibv_create_ah_resp *resp, > >>+ size_t resp_size) > > > >NAK without changing the name of this function.. > > > >We need to decide if we want to have comapt for ibv_cmd_ driver facing > >entry points, or discard that idea now that we only have one tree. > > The idea was that moving to one tree can drop the need to maintain 2 > different symbols, all vendors around where changed to use the new command. > As this API is not used directly by an applications I don't see a real need > for 2 APIs. I'm fine with that, but we still need to change the dynamic link symbol name when the signature is changed to ensure dynamlic linking of wrong providers fails. This might just be as simple as moving ibv_cmd_create_ah from IBVERBS_1.0 to IBVERBS_1.4 in the .map file. Please test and confirm. 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