[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <becaaeaf-e76a-43d2-b6e1-e7cc330d8cae@lunn.ch>
Date: Fri, 6 Sep 2024 17:54:49 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Jeff Daly <jeffd@...icom-usa.com>
Cc: anthony.l.nguyen@...el.com, przemyslaw.kitszel@...el.com,
davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
pabeni@...hat.com, intel-wired-lan@...ts.osuosl.org,
netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ixgbe: Manual AN-37 for troublesome link partners for
X550 SFI
On Fri, Sep 06, 2024 at 06:41:45AM -0400, Jeff Daly wrote:
> Resubmit commit 565736048bd5 ("ixgbe: Manual AN-37 for troublesome link
> partners for X550 SFI")
>
> Some (Juniper MX5) SFP link partners exhibit a disinclination to
> autonegotiate with X550 configured in SFI mode. This patch enables
> a manual AN-37 restart to work around the problem.
>
> Resubmitted patch includes a module parameter (default disabled) to
> isolate changes.
Module parameters are not liked in networking code. They are very user
unfriendly, and poorly documented.
Why do you need it? Is this change risky?
Andrew
Powered by blists - more mailing lists