]> www.infradead.org Git - users/willy/linux.git/commit
net: ethtool: let drivers remove lost RSS contexts
authorJakub Kicinski <kuba@kernel.org>
Thu, 11 Jul 2024 22:07:03 +0000 (15:07 -0700)
committerJakub Kicinski <kuba@kernel.org>
Sat, 13 Jul 2024 05:16:21 +0000 (22:16 -0700)
commitd69ba6bbaf1f606ac354e925571a54d025e32aae
tree06e8fb4031386840f96bdf6f35ab56ce5ffd9ef1
parent338a93cf4a18c2036b567e9f613367f7a52f2511
net: ethtool: let drivers remove lost RSS contexts

RSS contexts may get lost from a device, in various extreme circumstances.
Specifically if the firmware leaks resources and resets, or crashes and
either recovers in partially working state or the crash causes a
different FW version to run - creating the context again may fail.

Drivers should do their absolute best to prevent this from happening.
When it does, however, telling user that a context exists, when it can't
possibly be used any more is counter productive. Add a helper for
drivers to discard contexts. Print an error, in the future netlink
notification will also be sent.

More robust approaches were proposed, like keeping the contexts
but marking them as "dead" (but possibly resurrected by next reset).
That may be better but it's unclear at this stage whether the
effort is worth the benefits.

Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Link: https://patch.msgid.link/20240711220713.283778-2-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/linux/ethtool.h
net/ethtool/common.c