On Mon, 2016-09-19 at 06:29 +0200, Knut Omang wrote: > On Sun, 2016-09-18 at 22:08 -0600, Jason Gunthorpe wrote: > > On Mon, Sep 19, 2016 at 05:51:03AM +0200, Knut Omang wrote: > > > > > I was a bit concerned about the "structure", but ultimately decided to > > > follow the pattern since I could not find any documentation with this syntax > > > (which is probably because it is not supported?) > > > > It is something carried over from Solaris, it is not implemented in > > linux today, but it is good practice to follow the solaris > > method. Drepper talks about it in his various documentations of symbol > > versioning. > > > > Each stanza should refer to its predecessor. > > Ok, I see... > That also means that this patch would have to be based on the IBVERBS_1.2 and IBVERBS_1.3 fixups > to avoid introducing conflicts. I tested this, which seems to work fine, but I am not sure you want it ;-) : diff --git a/src/libibverbs.map b/src/libibverbs.map index 5134bd9..979980d 100644 --- a/src/libibverbs.map +++ b/src/libibverbs.map @@ -118,5 +118,15 @@ IBVERBS_1.1 { ibv_cmd_create_qp_ex2; ibv_cmd_open_qp; ibv_cmd_rereg_mr; } IBVERBS_1.0; +IBVERBS_1.2 { +} IBVERBS_1.1; + + +IBVERBS_1.3 { +} IBVERBS_1.2; + +IBVERBS_1.4 { + global: + ibv_cmd_create_ah_ex; +} IBVERBS_1.3; Knut -- 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