Date: Tue, 11 Jul 2023 18:45:33 +0200 From: Piotr Kubaj <pkubaj@anongoth.pl> To: Navdeep Parhar <np@FreeBSD.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: a0302c923150 - main - ix, ixv: Update link status with autonegotiated baudrate value Message-ID: <ZK2HLX0kIZytPsNf@talos> In-Reply-To: <1e24a398-2a92-b84e-a5c6-710dc277fe51@FreeBSD.org> References: <202307111619.36BGJYdv023721@gitrepo.freebsd.org> <1e24a398-2a92-b84e-a5c6-710dc277fe51@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--hkD8xVjIhsAJGigs Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 23-07-11 09:34:08, Navdeep Parhar wrote: > On 7/11/23 9:19=E2=80=AFAM, Piotr Kubaj wrote: > > The branch main has been updated by pkubaj: > >=20 > > URL: https://cgit.FreeBSD.org/src/commit/?id=3Da0302c9231502bae8f43edbd= 5fb8d73132eb8da7 > >=20 > > 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 > >=20 > > ix, ixv: Update link status with autonegotiated baudrate value > > =20 > > Use autonegotiated link speed value while updating link status > > to iflib. > > =20 > > This patch is part of change made in NetBSD kernel > > by Masanobu Saitoh, NetBSD maintainer. > > =20 > > 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(-) > >=20 > > 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 */ >=20 > This comment should have been deleted too? Thanks, fixed now. >=20 > Regards, > Navdeep >=20 > > - 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) --hkD8xVjIhsAJGigs Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEycyIeNkkgohzsoorelmbhSCDnJ0FAmSthy0ACgkQelmbhSCD nJ1urhAAhwugy4vzFkyM/nQXK+gvRx+Ai07nWg2yX7cl82jCJsaHAAAN69KXK8m5 dGRCJmUliWwNz7SnEswvABYkfQEtHvkgo2gDEznVmLfo1DrfeWTO8D2gwBeL++Ao HPrYAgVrU2U7jGZVK4Tq7AWDiVdKqF7a73E6dYbN6mgdUthQmlJV3mnk9uyC8a09 xNxD/lrp1d8gIAjxPfsD0t0xPB5ZTwYWC05uMLBRuLoEzm0Kx6GFHVwzYtaLK2m1 CVDTv+Bt9NNOkjFtbAFSyoPGCEplIEQJ9VdVtge9D2Ly/Gapg2/sJIh6081km8rJ XFAlcVBl3ro6AyryCT2bpzkfCgzne3wJ/f+3uCFePBfLBINg8DydyNYBuNN3CoSb Ru8CTy8048kBB2tKNhv+TgWAP7r4VNxAzp+L9cPcCNXLcbxrlZ9v58O4esfTGn8x BJ9BfO9Avz2mU3aR3gdRVhv/I/Wui9VIn7tHlOH7R/0P1hjpgCKpD8sZpBBEieGW 4X54K1E/hMPFyq+X5srsByy1siSJ2KDjIvHZEIPK8jAz0D/zEdeDPpY+asGQIkEC iBIZDwh5p0ROax6mQyf2AGsJF822Nlg8ueqOxgsAhZxJ91GiXrGMGhURTMHKAWbW j0QHvtcQGiooMO/hXUV9i3cxAzutL2kS04ARwUijvZKxvEuVcUA= =SJJE -----END PGP SIGNATURE----- --hkD8xVjIhsAJGigs--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ZK2HLX0kIZytPsNf>