On Mon, Dec 7, 2015 at 11:32 AM, Jason Gunthorpe <jgunthorpe@xxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, Dec 03, 2015 at 04:20:50PM +0000, Liran Liss wrote: >> > From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma- >> >> > Subject: Re: [PATCH for-next V2 05/11] IB/core: Add rdma_network_type to >> > wc >> > >> > Bloating the WC with a field that's not really useful for the ULPs seems pretty >> > sad.. >> >> You need per packet (read per-WC) network type both for handling incoming connections over QP1 and in UD QPs. >> It looks like this patch doesn't extend the structure size due to alignment, so no real harm in any case... > > Why? The sgid index must tell you the network type. > User space might not have access to internal properties of gid entry like kernel. Not sure if more plumbing needs to be added in user space to get such property and async updates of it. So Liran might be thinking to have unified way to report required fields via wc? -- 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