[Intel-wired-lan] [PATCH iwl-next v2] ice: Disable Cage Max Power override

Drewek, Wojciech wojciech.drewek at intel.com
Fri Sep 1 13:34:04 UTC 2023



> -----Original Message-----
> From: Ido Schimmel <idosch at idosch.org>
> Sent: środa, 30 sierpnia 2023 17:17
> To: Drewek, Wojciech <wojciech.drewek at intel.com>
> Cc: Jakub Kicinski <kuba at kernel.org>; intel-wired-lan at lists.osuosl.org;
> netdev at vger.kernel.org; Kitszel, Przemyslaw <przemyslaw.kitszel at intel.com>;
> idosch at nvidia.com
> Subject: Re: [PATCH iwl-next v2] ice: Disable Cage Max Power override
> 
> On Tue, Aug 29, 2023 at 09:12:22AM +0000, Drewek, Wojciech wrote:
> > In some cases users are trying to use media with power exceeding max
> allowed value.
> > Port split require system reboot so it feels natural to me to restore default
> settings.
> 
> I don't believe it's the kernel's responsibility to undo changes done by
> external tools. Given that the tool is able to change this setting, I
> assume it can also restore it back to default.

I agree with that, but we can end up with no link if we don't restore
default settings. Let me explain how.

> 
> Moreover, it doesn't sound like port split won't work without this
> change, so placing this change there only because we assume that a
> reboot will follow seems random.

After port split, we might end up with no link in one of the ports.
In dual port card if we increase max pwr on the 1st cage the 2nd one
will have max pwr decreased automatically. This might be useful if we have port
option with count 1, the second cage is not used in this case. If we then split and
use two ports now, the second port will use second cage which has decreased max pwr, default module
used there will not work.

So, should we leave the restoration of the default settings to the user?

> 
> I think the best way forward is to extend ethtool as was already
> suggested. It should allow you to avoid the split brain situation where
> the hardware is configured by both the kernel and an external tool.

I'll try to follow up with the ethtool extension.


More information about the Intel-wired-lan mailing list