Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-23 Thread David Miller
From: Jeff Kirsher Date: Tue, 22 May 2018 13:38:31 -0700 > From: Sindhu Devale > > Currently i40iw is dependent on i40e symbols > i40e_register_client and i40e_unregister_client due to > which i40iw cannot be loaded without i40e being loaded. > > This patch allows RDMA driver to build and load

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-23 Thread Jason Gunthorpe
On Wed, May 23, 2018 at 08:03:44AM -0700, Alexander Duyck wrote: > On Tue, May 22, 2018 at 11:19 PM, Christoph Hellwig > wrote: > > On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: > >> > Why would you want to do this? The rdma driver is non-functional > >> > without the ethernet dri

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-23 Thread Alexander Duyck
On Tue, May 22, 2018 at 11:19 PM, Christoph Hellwig wrote: > On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: >> > Why would you want to do this? The rdma driver is non-functional >> > without the ethernet driver, so why on earth would we want to defeat >> > the module dependency mech

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Christoph Hellwig
On Tue, May 22, 2018 at 02:50:32PM -0700, Jeff Kirsher wrote: > The ABI rarely changes, if at all. The issue OSV's are seeing is that > upgrading i40e, requires that i40iw be recompiled even though there > were no updates/changes to the ABI. So fscking what. If you upgrade one part of the kernel

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Christoph Hellwig
On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: > > Why would you want to do this? The rdma driver is non-functional > > without the ethernet driver, so why on earth would we want to defeat > > the module dependency mechanism? > > This change is driven by the OSV's like Red Hat, wher

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jason Gunthorpe
On Tue, May 22, 2018 at 02:50:32PM -0700, Jeff Kirsher wrote: > On Tue, 2018-05-22 at 15:33 -0600, Jason Gunthorpe wrote: > > On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: > > > On Tue, 2018-05-22 at 14:56 -0600, Jason Gunthorpe wrote: > > > > On Tue, May 22, 2018 at 01:38:31PM -070

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jeff Kirsher
On Tue, 2018-05-22 at 15:33 -0600, Jason Gunthorpe wrote: > On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: > > On Tue, 2018-05-22 at 14:56 -0600, Jason Gunthorpe wrote: > > > On Tue, May 22, 2018 at 01:38:31PM -0700, Jeff Kirsher wrote: > > > > From: Sindhu Devale > > > > > > > > C

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jason Gunthorpe
On Tue, May 22, 2018 at 02:04:06PM -0700, Jeff Kirsher wrote: > On Tue, 2018-05-22 at 14:56 -0600, Jason Gunthorpe wrote: > > On Tue, May 22, 2018 at 01:38:31PM -0700, Jeff Kirsher wrote: > > > From: Sindhu Devale > > > > > > Currently i40iw is dependent on i40e symbols > > > i40e_register_client

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jeff Kirsher
On Tue, 2018-05-22 at 14:56 -0600, Jason Gunthorpe wrote: > On Tue, May 22, 2018 at 01:38:31PM -0700, Jeff Kirsher wrote: > > From: Sindhu Devale > > > > Currently i40iw is dependent on i40e symbols > > i40e_register_client and i40e_unregister_client due to > > which i40iw cannot be loaded withou

Re: [net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jason Gunthorpe
On Tue, May 22, 2018 at 01:38:31PM -0700, Jeff Kirsher wrote: > From: Sindhu Devale > > Currently i40iw is dependent on i40e symbols > i40e_register_client and i40e_unregister_client due to > which i40iw cannot be loaded without i40e being loaded. > > This patch allows RDMA driver to build and l

[net-next] i40iw/i40e: Remove link dependency on i40e

2018-05-22 Thread Jeff Kirsher
From: Sindhu Devale Currently i40iw is dependent on i40e symbols i40e_register_client and i40e_unregister_client due to which i40iw cannot be loaded without i40e being loaded. This patch allows RDMA driver to build and load without linking to LAN driver and without LAN driver being loaded first.