On Thu, Jun 12, 2025 at 03:05:42PM -0700, Jakub Kicinski wrote:
On Wed, 11 Jun 2025 07:18:56 -0700 Breno Leitao wrote:
Add a new selftest to verify netconsole module loading with command line arguments. This test exercises the init_netconsole() path and validates proper parsing of the netconsole= parameter format.
The test:
- Loads netconsole module with cmdline configuration instead of dynamic reconfiguration
- Validates message transmission through the configured target
- Adds helper functions for cmdline string generation and module validation
This complements existing netconsole selftests by covering the module initialization code path that processes boot-time parameters. This test is useful to test issues like the one described in [1].
I think this leaks the IP address, because if netcons_overflow.sh runs afterwards it skips with:
# inet 192.0.2.1/24 scope global eni370np1 # SKIP: IPs already in use. Skipping it
if netcons_overflow.sh runs first everything is fine.
Thanks for the report. I acknowledge there is a bug in the cleanup part of the code. Basically `trap cleanup_all_ns EXIT` is not enought because it was not removing the local netdevsim interface, which must be removed as well. I will update and resend. Good catch, again.
--breno