[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <063D6719AE5E284EB5DD2968C1650D6DD00490C0@AcuExch.aculab.com>
Date: Wed, 2 Aug 2017 13:48:18 +0000
From: David Laight <David.Laight@...LAB.COM>
To: 'Timur Tabi' <timur@...eaurora.org>,
"David S. Miller" <davem@...emloft.net>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [PATCH 1/2] [for 4.13] net: qcom/emac: disable flow control
autonegotiation by default
From: Timur Tabi
> Sent: 01 August 2017 22:38
> The EMAC has a curious qwirk when RX flow control is enabled and the
> kernel hangs. With the kernel hung, the EMAC's RX queue soon fills.
> If RX flow control is enabled, the EMAC will then send a non-stop
> stream of pause frames until the system is reset. The EMAC does not
> have a built-in watchdog.
>
> In various tests, the pause frame stream sometimes overloads nearby
> switches, effectively disabling the network.
...
If the nearby switches cannot handle pause frames, then the MAC shouldn't
be sending them at all.
They
I suspect that they should only ever be sent if the phy autonegotiation
indicates that they are supported.
You might want to avoid sending them even if allowed, or advertise
non-support on hardware that could support them, but sending them
anyway is likely to cause grief.
This is similar to the problems that arise when the 'speed' is forced
instead of limiting the advertised speed to one value.
David
Powered by blists - more mailing lists