On Mon, Jul 27, 2020 at 03:49:37PM -0700, Jacob Keller wrote: > When displaying the FEC link modes advertised by the peer, we used the > string "No" to indicate when nothing was provided. This does not match > the IOCTL output which indicates "Not reported". It also doesn't match > the local advertised FEC modes, which also used the "Not reported" > string. > > This is especially confusing for FEC, because the FEC bits include > a "None" bit which indicates that FEC is definitely not supported. Avoid > this confusion and match both the local advertised settings display and > the old IOCTL output by using "Not reported" when FEC settings aren't > reported. > > Reported-by: Andrew Lunn <[email protected]> > Signed-off-by: Jacob Keller <[email protected]>
Applied with
Fixes: 10cc3ea337d1 ("netlink: partial netlink handler for gset (no option)")
Michal
> ---
> netlink/settings.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/netlink/settings.c b/netlink/settings.c
> index 66b0d4892cdd..726259d83702 100644
> --- a/netlink/settings.c
> +++ b/netlink/settings.c
> @@ -481,7 +481,7 @@ static int dump_peer_modes(struct nl_context *nlctx,
> const struct nlattr *attr)
>
> ret = dump_link_modes(nlctx, attr, false, LM_CLASS_FEC,
> "Link partner advertised FEC modes: ",
> - " ", "\n", "No");
> + " ", "\n", "Not reported");
> return ret;
> }
>
> --
> 2.26.2
>
signature.asc
Description: PGP signature
