Re: ethtool pause mode clarifications

[Andrew Lunn <andrew@lunn.ch>]

On Fri, Dec 13, 2019 at 11:49:35AM +0000, Russell King - ARM Linux admin wrote:
> Hi,
> 
> Please can someone explain the ethtool pause mode settings?  The man
> page isn't particularly clear, it says:
> 
>        -A --pause
>               Changes the pause parameters of the specified Ethernet device.
> 
>            autoneg on|off
>                   Specifies whether pause autonegotiation should be enabled.
> 
>            rx on|off
>                   Specifies whether RX pause should be enabled.
> 
>            tx on|off
>                   Specifies whether TX pause should be enabled.
> 
> 
> "autoneg" states whether pause autonegotiation should be enabled, but
> how is this possible, when pause autonegotiation happens as part of the
> rest of the autonegotiation as a matter of course, and the only control
> we have at the PHY is the value of the pause and asym pause bits?

Hi Russell

Yah, this is not clear. How i've interpreted it is:

autoneg on:

The driver should validate rx and tx with its capabilities, and then
tell the PHY what to advertise, kick off an auto-neg, and wait for the
result to program the MAC with the negotiated value.

If autoneg in general is off, return an error.

autoneg off:

Forget about the PHY, program the MAC directly, and potentially shoot
yourself in the foot. But it can be useful it auto-neg in general is
off, or there is no PHY.


> So, would it be possible to clarify what these settings mean in the
> ethtool man page please?

I suspect the first step would be to survey current implementations
and find out what is the most popular interpretation of this
text. Then expand the document, and maybe list some of the alternative
meanings which are currently in use?

Clearly, the more of this we can handle in phylink/phylib, the more
uniform it will be. But there is also a trend at the moment for
firmware to control the PHY, and it seems like a few MAC driver
writers have no idea what their firmware is actually doing with the
PHY for things like this.

    Andrew