On Thu, Sep 01, 2016 at 07:22:38PM +0200, Knut Omang wrote: > On Thu, 2016-09-01 at 10:49 -0600, Jason Gunthorpe wrote: > > On Thu, Sep 01, 2016 at 08:59:51AM +0200, Knut Omang wrote: > > > +++ b/src/libibverbs.map > > > @@ -64,6 +64,7 @@ IBVERBS_1.0 { > > > ibv_cmd_post_recv; > > > ibv_cmd_post_srq_recv; > > > ibv_cmd_create_ah; > > > + ibv_cmd_create_ah_ex; > > > > I should also point out that this is not the proper way to use symbol > > versions. Typically one would tag new symbols with the version number > > of the release that introduces them, and not just keep re-using 1.0 > > > > I know we haven't been doing that, but perhaps we should start. > > Yes, I noticed the never changed 1.0 and tried just to follow the usage > pattern ;-) Actually, thinking about this more, not doing this properly breaks RPM since it only summarizes the tag not the actual symbol into the package data. So, no this is not OK, and maybe we need to retroactively review things :( :( 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