netdev - Re: [PATCH net-next 0/6] Simplify Intel Wired LAN driver init/exit routine

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1409721460.2384.27.camel@jtkirshe-mobl>
Date:	Tue, 02 Sep 2014 22:17:40 -0700
From:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>
To:	Jean Sacren <sakiwit@...il.com>
Cc:	linux.nics@...el.com, e1000-devel@...ts.sourceforge.net,
	netdev@...r.kernel.org
Subject: Re: [PATCH net-next 0/6] Simplify Intel Wired LAN driver init/exit
 routine

On Tue, 2014-09-02 at 03:36 -0600, Jean Sacren wrote:
> This series uses module_pci_driver() to simplify the init/exit
> routine of some Intel Wired LAN drivers.
> 
> Jean Sacren (6):
>   ixgb: remove the boilerplate of ixgb module for init/exit
>   ixgbevf: remove the boilerplate of ixgbevf module for init/exit
>   e100: remove the boilerplate of e100 module for init/exit
>   igbvf: remove the boilerplate of igbvf module for init/exit
>   e1000e: remove the boilerplate of e1000e module for init/exit
>   i40evf: remove the boilerplate of i40evf module for init/exit
> 
>  drivers/net/ethernet/intel/e100.c                 | 22 ++++---------
>  drivers/net/ethernet/intel/e1000e/netdev.c        | 35 +++-----------------
>  drivers/net/ethernet/intel/i40evf/i40evf_main.c   | 37 +++------------------
>  drivers/net/ethernet/intel/igbvf/netdev.c         | 35 +++-----------------
>  drivers/net/ethernet/intel/ixgb/ixgb_main.c       | 39 +++--------------------
>  drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 38 ++++------------------
>  6 files changed, 32 insertions(+), 174 deletions(-)
> 

I have gone ahead and added this series to my queue.  We have some
concerns that these changes will make it more difficult to debug the
drivers, so I will have these go through our validation as well.

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)