Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-29 Thread kbuild test robot
Hi Catherine, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Catherine-Sullivan/Add-gve-driver/20190629-070444 config: i386-allyesconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 reprodu

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread kbuild test robot
Hi Catherine, I love your patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Catherine-Sullivan/Add-gve-driver/20190629-070444 reproduce: # apt-get install sparse # sparse version: v0.6.1-rc1-7-g2b96

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread Catherine Sullivan
On Fri, Jun 28, 2019 at 1:06 PM Andrew Lunn wrote: > > On Fri, Jun 28, 2019 at 11:46:15AM -0700, Jakub Kicinski wrote: > > On Fri, 28 Jun 2019 10:52:27 -0700, Catherine Sullivan wrote: > > > > > +if NET_VENDOR_GOOGLE > > > > > + > > > > > +config GVE > > > > > + tristate "Google Virtual NIC (g

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread Andrew Lunn
On Fri, Jun 28, 2019 at 11:46:15AM -0700, Jakub Kicinski wrote: > On Fri, 28 Jun 2019 10:52:27 -0700, Catherine Sullivan wrote: > > > > +if NET_VENDOR_GOOGLE > > > > + > > > > +config GVE > > > > + tristate "Google Virtual NIC (gVNIC) support" > > > > + depends on (PCI_MSI && X86) > > > >

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread Jakub Kicinski
On Fri, 28 Jun 2019 10:52:27 -0700, Catherine Sullivan wrote: > > > +if NET_VENDOR_GOOGLE > > > + > > > +config GVE > > > + tristate "Google Virtual NIC (gVNIC) support" > > > + depends on (PCI_MSI && X86) > > > > We usually prefer for drivers not to depend on the platform unless > > real

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread Catherine Sullivan
On Wed, Jun 26, 2019 at 12:35 PM Andrew Lunn wrote: > > On Wed, Jun 26, 2019 at 11:52:48AM -0700, Catherine Sullivan wrote: > > Add a driver framework for the Compute Engine Virtual NIC that will be > > available in the future. > > > > +static int __init gvnic_init_module(void) > > +{ > > + re

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-28 Thread Catherine Sullivan
On Wed, Jun 26, 2019 at 4:08 PM Jakub Kicinski wrote: > > On Wed, 26 Jun 2019 11:52:48 -0700, Catherine Sullivan wrote: > > Add a driver framework for the Compute Engine Virtual NIC that will be > > available in the future. > > > > At this point the only functionality is loading the driver. > > >

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-26 Thread Jakub Kicinski
On Wed, 26 Jun 2019 11:52:48 -0700, Catherine Sullivan wrote: > Add a driver framework for the Compute Engine Virtual NIC that will be > available in the future. > > At this point the only functionality is loading the driver. > > Signed-off-by: Catherine Sullivan > Signed-off-by: Sagi Shahar >

Re: [net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-26 Thread Andrew Lunn
On Wed, Jun 26, 2019 at 11:52:48AM -0700, Catherine Sullivan wrote: > Add a driver framework for the Compute Engine Virtual NIC that will be > available in the future. > > +static int __init gvnic_init_module(void) > +{ > + return pci_register_driver(&gvnic_driver); > +} > + > +static void __e

[net-next 1/4] gve: Add basic driver framework for Compute Engine Virtual NIC

2019-06-26 Thread Catherine Sullivan
Add a driver framework for the Compute Engine Virtual NIC that will be available in the future. At this point the only functionality is loading the driver. Signed-off-by: Catherine Sullivan Signed-off-by: Sagi Shahar Signed-off-by: Jon Olson Acked-by: Willem de Bruijn Reviewed-by: Luigi Rizzo