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
