Re: git: a0302c923150 - main - ix, ixv: Update link status with autonegotiated baudrate value
Date: Tue, 11 Jul 2023 16:34:08 UTC
On 7/11/23 9:19 AM, Piotr Kubaj wrote: > The branch main has been updated by pkubaj: > > URL: https://cgit.FreeBSD.org/src/commit/?id=a0302c9231502bae8f43edbd5fb8d73132eb8da7 > > commit a0302c9231502bae8f43edbd5fb8d73132eb8da7 > Author: Piotr Pietruszewski <piotr.pietruszewski@intel.com> > AuthorDate: 2023-07-11 16:14:26 +0000 > Commit: Piotr Kubaj <pkubaj@FreeBSD.org> > CommitDate: 2023-07-11 16:14:26 +0000 > > ix, ixv: Update link status with autonegotiated baudrate value > > Use autonegotiated link speed value while updating link status > to iflib. > > This patch is part of change made in NetBSD kernel > by Masanobu Saitoh, NetBSD maintainer. > > Differential Revision: https://reviews.freebsd.org/D19176 > Approved by: erj > --- > sys/dev/ixgbe/if_ix.c | 3 ++- > sys/dev/ixgbe/if_ixv.c | 2 +- > sys/dev/ixgbe/ixgbe.h | 2 ++ > sys/dev/ixgbe/ixgbe_osdep.c | 33 +++++++++++++++++++++++++++++++++ > 4 files changed, 38 insertions(+), 2 deletions(-) > > diff --git a/sys/dev/ixgbe/if_ix.c b/sys/dev/ixgbe/if_ix.c > index 0179c6456de8..09c0a82279ed 100644 > --- a/sys/dev/ixgbe/if_ix.c > +++ b/sys/dev/ixgbe/if_ix.c > @@ -3679,7 +3679,8 @@ ixgbe_if_update_admin_status(if_ctx_t ctx) > /* Update DMA coalescing config */ > ixgbe_config_dmac(sc); > /* should actually be negotiated value */ This comment should have been deleted too? Regards, Navdeep > - iflib_link_state_change(ctx, LINK_STATE_UP, IF_Gbps(10)); > + iflib_link_state_change(ctx, LINK_STATE_UP, > + ixgbe_link_speed_to_baudrate(adapter->link_speed)); > > if (sc->feat_en & IXGBE_FEATURE_SRIOV) > ixgbe_ping_all_vfs(sc)