On Mon, Dec 7, 2015 at 8:48 PM, Jason Gunthorpe <jgunthorpe@xxxxxxxxxxxxxxxxxxxx> wrote: > On Mon, Dec 07, 2015 at 08:34:43PM +0200, Moni Shoua wrote: >> Well, just tell me how you want to discover gid_index when you poll >> the WC out of the CQ. > > Hey, I'm not desiging this rocev2 stuff, this is something the rocev2 > community needs to sort out. > >> Like I said, the sgid itself is in the GRH that is scattered to the >> buffers in the receive queue. When ib_poll_cq() is called the pointer >> to GRH is not passed so there is no way to determine the gid_index >> inside the polling context. > > Then have an API to let the consumer recover it. We do but it has it disadvantages. We'll use it when there it no other choice (i.e. not supporting HW) > >> BTW, why do you argue about this only now? This is not RoCE specific >> issue. sgid_index was always resolved outside the polling context. The >> only difference between then and now is that with InfiniBand there was >> no conflict about sgid_index once you had the sgid. > > I think you answered your own question. In IB and rocev1 the sgid > index is not ambiguous, rocev2 breaks that requirement without > adaquately fixing it. ambiguity is not the issue here. You say that sgid_index needs to be known during poll so why raise it now? We want to add information to the CQE to help resolve the conflict but we leave the resolution of sgid index to the caller, like for RoCEv1 and like for native InfiniBand > > It is probably also the case that rocev1 has similar problems, > depending on how vland and macvlan ended up being implemented. > >> Now, when same gid value can be present in multiple entries you need >> an extra parameter to get distinguish between them - that would be >> the netwrok_type > > Eh? You are only worried about duplicates between rocev1 mode which > uses the GUID and v2 mode which uses the host's IP address? > > What about duplicates entirely within v2 mode where vlan and macvlan > can create duplicate host IP addresses. > > You can also just not create duplicate entries in the first place. For > instance there probably isn't a really a good reason to use rocev2 for > IPv6 link local addreses, that trivially eliminates the v1/v2 > collision. > > Jason > -- > 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 -- 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
