From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Vishwanathapura, Niranjana" Subject: Re: [RFC 02/10] IB/hfi-vnic: Virtual Network Interface Controller (VNIC) Bus driver Date: Mon, 28 Nov 2016 22:31:06 -0800 Message-ID: <20161129063106.GB84990@knc-06.sc.intel.com> References: <20161121232629.GA67988@knc-06.sc.intel.com> <20161121233118.GA31132@obsidianresearch.com> <20161122015304.GB67988@knc-06.sc.intel.com> <20161122170407.GE3956@obsidianresearch.com> <20161123000502.GA27968@phlsvsds.ph.intel.com> <20161123004932.GA13598@obsidianresearch.com> <20161124000825.GA73280@knc-06.sc.intel.com> <20161124161545.GA20818@obsidianresearch.com> <20161125021350.GA74946@knc-06.sc.intel.com> <20161125190509.GB16504@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Cc: "ira.weiny" , Doug Ledford , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Dennis Dalessandro To: Jason Gunthorpe Return-path: Content-Disposition: inline In-Reply-To: <20161125190509.GB16504-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On Fri, Nov 25, 2016 at 12:05:09PM -0700, Jason Gunthorpe wrote: >On Thu, Nov 24, 2016 at 06:13:50PM -0800, Vishwanathapura, Niranjana wrote: > >> In order to be truely device independent the hfi_vnic ULP should not depend >> on a device exported symbol. Instead device should register its functions >> with the ULP. Hence the approaches a) and b). > >It is not device independent, it is hard linked to hfi1, just like our >other multi-component drivers.. So don't worry about that. > We would like to keep the design clean and avoid any tight coupling here (our original design in this series tackled these). Any strong reason not to go with a) or b) ? Niranjana >Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html