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

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: <20141118.133829.2246481313976384106.davem@davemloft.net>
Date:	Tue, 18 Nov 2014 13:38:29 -0500 (EST)
From:	David Miller <davem@...emloft.net>
To:	jeffrey.t.kirsher@...el.com
Cc:	netdev@...r.kernel.org, nhorman@...hat.com, sassmann@...hat.com,
	jogreene@...hat.com
Subject: Re: [net-next v2 00/11][pull request] Intel Wired LAN Driver
 Updates 2014-11-18

From: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
Date: Tue, 18 Nov 2014 01:25:20 -0800

> This series contains updates to i40e only.
> 
> Shannon provides a patch to clean up the driver to only warn once that
> PTP is not supported when linked at 100Mbps.
> 
> Mitch provides a fix for i40e where the VF interrupt processing takes
> a long time and it is possible that we could lose a VFLR event if it
> happens while processing a VFLR on another VF.  To correct this situation,
> we enable the VFLR interrupt cause before we begin processing any pending
> resets.
> 
> Neerav provides several patches to update DCB support in i40e.  When
> there are DCB configuration changes based on DCBx, the firmware suspends
> the port's Tx and generates an event to the PF.  The PF is then
> responsible to reconfigure the PF VSIs and switching topology as per the
> updated DCB configuration and then resume the port's Tx by calling the
> "Resume Port Tx" AQ command, so add this call to the flow that handles
> DCB re-configuration in the PF.  Allow the driver to query and use DCB
> configuration from firmware when firmware DCBx agent is in CEE mode.
> Add a check whether LLDP Agent's default AdminStatus is enabled or
> disabled on a given port, and sets DCBx status to disabled if the
> status is disabled.  Fix an issue when the port TC configuration
> changes as a result of DCBx and the driver modifies the enabled TCs for
> the VEBs it manages but does not update the enabled_tc value that 
> was cached on a per VEB basis.  Add a new PF state so that if a port's
> Tx is in suspended state the Tx queue disable flow would just put the
> request for the queue to be disabled and return without waiting for the
> queue to be actually disabled.  Allows the driver to enable/disable
> the XPS based on the number of TCs being enabled for the given VSI.

Pulled, thanks Jeff.

> v2: Dropped patch "i40e: Handle a single mss packet with more than 8 frags"
>     while we rework the patch after we test a bit more based on feedback from 
>     Eric Dumazet.

Since, as Eric showed, it's easy to hang the card with a simple
netperf command line due to this issue, you might want to consider
sending that fix to 'net' instead.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html