On Mon, Nov 28, 2016 at 10:29:38PM -0800, Vishwanathapura, Niranjana wrote: > On Thu, Nov 24, 2016 at 09:15:45AM -0700, Jason Gunthorpe wrote: > >>And will move the hfi_vnic module under > >>???drivers/infiniband/ulp/hfi_vnic???. > > > >I would prefer drivers/net/ethernet > > > >This is clearly not a ULP since it doesn't use verbs. > > > > I understand it is not using verbs, but the control path (ib_device client) > is using verbs (IB MAD). > Our prefernce is to keep it somewhere under drivers/infiniband. Summarizing > reasons again here, > > - VNIC control driver (ib_device client) is an IB MAD agent. > - It is purly a software construct, encapsualtes ethernet packets in > Omni-path packet and depends on hfi1 driver here for HW access. Is the majority of the code MAD focused or net stack focused? I'm not sure it matters, it isn't like we can review Intel's proprietary mad stuff anyhow. :\ 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
