Page MenuHomeFreeBSD

debugnet: remove spurious message on boot

Authored by on Feb 28 2022, 11:33 AM.
Referenced Files
Unknown Object (File)
Sat, Mar 30, 11:47 PM
Unknown Object (File)
Tue, Mar 19, 8:33 AM
Unknown Object (File)
Feb 26 2024, 2:42 PM
Unknown Object (File)
Feb 18 2024, 7:22 PM
Unknown Object (File)
Jan 27 2024, 1:01 PM
Unknown Object (File)
Jan 24 2024, 1:06 PM
Unknown Object (File)
Jan 21 2024, 12:58 PM
Unknown Object (File)
Jan 13 2024, 7:37 AM



In 2019 it was said to be added to fix the drivers but that has not
happened so it does not make sense to present a non-error to users.

No response on recent mailing list question:

Diff Detail

rG FreeBSD src repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

I think it's important to leave this sort of message in for kernel developers to see, but agreed it has little user-facing value. What about something like this:

if (bootverbose)
	printf("%s: Bad dn_init result from %s (ifp %p), ignoring.\n",
	    __func__, if_name(ifp), ifp);

i.e., always print it in an INVARIANTS kernel, otherwise hide it under bootverbose

Either way seems fine to me. I’d ask vangyzen or bdrewery for input.

This revision is now accepted and ready to land.Feb 28 2022, 8:14 PM

Since I don't have a commit bit... anyone willing to commit this? Thanks in advance.

This revision was automatically updated to reflect the committed changes.