netdev - Re: [net-next v2 00/12][pull request] Intel Wired LAN Driver Updates

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1394228248.2101.6.camel@jtkirshe-mobl.jf.intel.com>
Date:	Fri, 07 Mar 2014 13:37:28 -0800
From:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>
To:	David Miller <davem@...emloft.net>
Cc:	netdev@...r.kernel.org, gospo@...hat.com, sassmann@...hat.com
Subject: Re: [net-next v2 00/12][pull request] Intel Wired LAN Driver Updates

On Fri, 2014-03-07 at 16:19 -0500, David Miller wrote:
> From: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
> Date: Thu,  6 Mar 2014 18:59:12 -0800
> 
> > This series contains updates to i40e and i40evf.
> > 
> > Most notable are:
> > Joseph completes the implementation of the ethtool ntuple rule
> > management interface by adding the get, update and delete interface
> > reset.
> > 
> > Akeem provides a fix to prevent a possible overflow due to multiplication
> > of number and size by using kzalloc, so use kcalloc.
> > 
> > Jesse provides an implementation for skb_set_hash() and adds the L4 type
> > return when we know it is an L4 hash.  He also adds a counter to
> > statistics for Tx timeouts to help users.  Lastly he provides a change
> > to stay away from the cache line where the done bit may be getting
> > written back for the transmit ring since the hardware may be writing the
> > whole cache line for a partial update.
> > 
> > Shannon cleans up code comments.
> > 
> > Anjali removes a firmware workaround for newer firmware since the number
> > of MSIx vectors are being reported correctly.
> > 
> > v2:
> >  -  dropped patch 01 of the series based on feedback from the author
> >     Joe Perches and Shannon Nelson.
> 
> Pulled, thanks Jeff.
> 
> And I'd like to make a broad long-term comment, actually about vf
> drivers in general....
> 
> There is so much common code between pf and vf drivers, as a quick
> example even in this patch set the skb_set_hash() stuff is pretty
> much the same for i40e and i40evf.
> 
> I realize there are subtle differences between vf and pf, however
> you can't say that there isn't a metric ton of common code.
> 
> Please consider seriously making a common layer for these kinds of
> driver pairs.  I absolutely do not care, as an initial step, if the
> common code just gets linked directly into the final module object
> for the two drivers.  Although eventually it would be nice if the
> common layer driver was a shared module object on it's own that is
> simply depended upon by the pf and vf drivers.
> 
> Thanks.

Thanks Dave!  I will work with i40e developers to get a solution put
together for the shared code.

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