On Thu, Mar 25, 2010 at 02:00:25PM -0700, Sean Hefty wrote:

> >IMHO, after thinking about it for a bit, I would prefer to see the
> >port space be unused from a user-space perspective when used with
> >AF_IB.
> 
> The rdma_cm_id is associated with a port space on creation, before
> it's known what address family will be used.  The kernel code
> enforces that the SID is formatted correctly for the port space that
> was selected.

Does this early association with the port space have any effect?

> >If a ps is needed in the kernel then it should pick the ps based on
> >the SID prefix that user space provided.. I guess prior to adding the
> >mask bits this wouldn't have made sense, but now that they are in it
> >seems like the way to go.
> 
> I added sib_mask to sockaddr_ib, but it's unused at this point.  I
> think what you're saying makes sense, but for RDMA_PS_IB, once it's
> defined.  RDMA_PS_IB may need to behave as RDMA_PS_TCP or
> RDMA_PS_UDP based on the sib_mask and SID, and ensure that any
> selected SID is reserved from the correct underlying SID region.

I would be inclined to ditch the port space concept entirely for
AF_IB. Just ignore the input parameter and always base the selection
on the SID region. It is confusing to that there are two ways to
specify the same thing.

Is there any reason the port space has to be known when the cm_id is
created but before bind?

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to