<html><body>
<p>Ok thanks for the explanation. So, I presume that means rdma_ptr will now point to ib_device? <br>
<br>
If so, one issue that strikes me as significant would be backward compatability. My view is that one could continue to<br>
use the IPoIB private data.<br>
<br>
Pradeep<br>
pradeep@us.ibm.com<br>
<br>
<tt>Hal Rosenstock <halr@voltaire.com> wrote on 10/03/2005 04:17:17 PM:<br>
<br>
> On Mon, 2005-10-03 at 19:05, Pradeep Satyanarayana wrote:<br>
> > My understanding is that the refcnt will still need to be held (even<br>
> > after this change) even if SDP would not poke at IPoIB's private data.<br>
> > Is that true?<br>
> <br>
> Yes, that's an independent issue.<br>
> <br>
> > Moreover there was discussion about getting this data from the CM REQ<br>
> > private data. So, what is the exact rationale for adding this to the <br>
> > net_device structure? <br>
> <br>
> To get at the ib_device, port, and PKey which are needed for a<br>
> subsequent SA path record request.<br>
> <br>
> -- Hal<br>
> <br>
> > Pradeep<br>
> > pradeep@us.ibm.com<br>
> > <br>
> > openib-general-bounces@openib.org wrote on 10/03/2005 02:50:21 PM:<br>
> > <br>
> > > On Mon, 2005-10-03 at 17:35, Roland Dreier wrote:<br>
> > > > Hal> We will have this shortly. I have been waiting for this<br>
> > to<br>
> > > > Hal> propose the changes to SDP et al.<br>
> > > > <br>
> > > > OK, but I don't think it makes sense to merge this upstream until<br>
> > > > there is in-tree code that will use it.<br>
> > > <br>
> > > I wanted to get this in so I could add the code to IPoIB to use this<br>
> > so SDP<br>
> > > and others no longer poke at IPoIB's private data. This is a small<br>
> > > change. Should this change be made locally (in OpenIB) first (and<br>
> > we'll<br>
> > > have our own modified netdevice.h for a short time) ?<br>
> > > <br>
> > > -- Hal<br>
> > > <br>
> > > _______________________________________________<br>
> > > openib-general mailing list<br>
> > > openib-general@openib.org<br>
> > > <a href="http://openib.org/mailman/listinfo/openib-general">http://openib.org/mailman/listinfo/openib-general</a><br>
> > > <br>
> > > To unsubscribe, please visit<br>
> > <a href="http://openib.org/mailman/listinfo/openib-general">http://openib.org/mailman/listinfo/openib-general</a><br>
> <br>
</tt></body></html>