Merge branch 'netconsole-populate-dynamic-entry-even-if-netpoll-fails'
authorJakub Kicinski <kuba@kernel.org>
Mon, 26 Aug 2024 16:25:46 +0000 (09:25 -0700)
committerJakub Kicinski <kuba@kernel.org>
Mon, 26 Aug 2024 16:25:46 +0000 (09:25 -0700)
commit9f08ae4ffa39824b2e950fb57b7d8ccf07788e4f
tree123dde01978ee0c82056b2d38e0fa50aad0eb842
parentf086d18db11724f55e70e001ce5cc29fdf47bb55
parent908ee298c8fb3f9129a470735e8bb8037a95221e
Merge branch 'netconsole-populate-dynamic-entry-even-if-netpoll-fails'

Breno Leitao says:

====================
netconsole: Populate dynamic entry even if netpoll fails

The current implementation of netconsole removes the entry and fails
entirely if netpoll fails to initialize. This approach is suboptimal, as
it prevents reconfiguration or re-enabling of the target through
configfs.

While this issue might seem minor if it were rare, it actually occurs
frequently when the network module is configured as a loadable module.

In such cases, the network is unavailable when netconsole initializes,
causing netpoll to fail. This failure forces users to reconfigure the
target from scratch, discarding any settings provided via the command
line.

The proposed change would keep the target available in configfs, albeit
in a disabled state. This modification allows users to adjust settings
or simply re-enable the target once the network module has loaded,
providing a more flexible and user-friendly solution.

v2: https://lore.kernel.org/20240819103616.2260006-1-leitao@debian.org
v1: https://lore.kernel.org/20240809161935.3129104-1-leitao@debian.org
====================

Link: https://patch.msgid.link/20240822111051.179850-1-leitao@debian.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>