Re: [PATCH libibverbs v2 1/3] Add new call ibv_cmd_create_ah_ex which supports extra parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux