On Mon, Feb 15, 2021 at 04:19:19PM +0000, Stefan Chulski wrote:
quoted
quoted
quoted
I discussed it with Andrew earlier last year, and his response was:
DT configuration of pause for fixed link probably is sufficient. I
don't remember it ever been really discussed for DSA. It was a
Melanox discussion about limiting pause for the CPU. So I think it
is safe to not implement ethtool -A, at least until somebody has a
real use case for it.
So I chose not to support it - no point supporting features that
people aren't using. If you have a "real use case" then it can be added.
This patch may be sufficient - I haven't fully considered all the
implications of changing this though.
Did you try this patch? What's the outcome?
For me patch worked as expected.
Great, thanks. Andrew, do you have any further opinions on this
subject, or shall I sent a proper patch for net-next?
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!