Re: [PATCH net-next v2 7/7] selftest: netconsole: add test for cmdline configuration

[Breno Leitao <leitao@debian.org>]

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