[Intel-wired-lan] [PATCH 2/2] e1000e: start network tx queue only when link is up
Brown, Aaron F
aaron.f.brown at intel.com
Fri Apr 26 00:05:02 UTC 2019
> From: Konstantin Khlebnikov [mailto:khlebnikov at yandex-team.ru]
> Sent: Wednesday, April 17, 2019 1:13 AM
> To: netdev at vger.kernel.org; intel-wired-lan at lists.osuosl.org; linux-
> kernel at vger.kernel.org; Kirsher, Jeffrey T <jeffrey.t.kirsher at intel.com>
> Cc: Sasha Levin <sashal at kernel.org>; Joseph Yasi <joe.yasi at gmail.com>;
> Brown, Aaron F <aaron.f.brown at intel.com>; Alexander Duyck
> <alexander.duyck at gmail.com>; e1000-devel at lists.sourceforge.net
> Subject: [PATCH 2/2] e1000e: start network tx queue only when link is up
>
> Driver does not want to keep packets in tx queue when link is lost.
> But present code only reset NIC to flush them, but does not prevent
> queuing new packets. Moreover reset sequence itself could generate
> new packets via netconsole and NIC falls into endless reset loop.
>
> This patch wakes tx queue only when NIC is ready to send packets.
>
> This is proper fix for problem addressed by commit 0f9e980bf5ee
> ("e1000e: fix cyclic resets at link up with active tx").
>
> Signed-off-by: Konstantin Khlebnikov <khlebnikov at yandex-team.ru>
> Suggested-by: Alexander Duyck <alexander.duyck at gmail.com>
> ---
> drivers/net/ethernet/intel/e1000e/netdev.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
Tested-by: Aaron Brown <aaron.f.brown at intel.com>
Again, more of a regression check than a test that the patch solves the problem as I did not manage to trigger the hang.
More information about the Intel-wired-lan
mailing list