[ofa-general] [RMDA CM IPv6 support. PATCHv4 1/6] AF_INET6 support for rdma_bind_addr

Roland Dreier rdreier at cisco.com
Fri Nov 28 21:14:09 PST 2008


I would like to get some input from Sean before proceeding on this, but
one thing does jump out at me: the order of the patches seems strange to
me (or maybe it's the way the patches are split).  Starting with this
change only:

 > @@ -2073,7 +2073,7 @@ int rdma_bind_addr(struct rdma_cm_id *id, struct sockaddr *addr)
 > -	if (addr->sa_family != AF_INET)
 > +	if (addr->sa_family != AF_INET && addr->sa_family != AF_INET6)
 >  		return -EAFNOSUPPORT;

seems wrong to me.  If I just have this patch applied (eg if I'm doing a
bisection to track down a bug) then it seems I'll get some very strange
results if I try to bind an IPv6 address.

It seems to me we would want all the prep work like using
sockaddr_storage where needed, etc. before we actually enable IPv6 in
the API.

- R.



More information about the general mailing list