Page MenuHomeFreeBSD

debugnet: remove spurious message on boot
ClosedPublic

Authored by franco_opnsense.org on Feb 28 2022, 11:33 AM.
Tags
None
Referenced Files
F103199972: D34393.diff
Fri, Nov 22, 4:28 AM
Unknown Object (File)
Fri, Nov 22, 3:37 AM
Unknown Object (File)
Wed, Nov 20, 3:41 AM
Unknown Object (File)
Wed, Nov 20, 3:33 AM
Unknown Object (File)
Wed, Nov 20, 3:32 AM
Unknown Object (File)
Wed, Nov 20, 1:40 AM
Unknown Object (File)
Tue, Nov 19, 9:28 PM
Unknown Object (File)
Tue, Nov 19, 9:58 AM

Details

Summary

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:

https://lists.freebsd.org/archives/freebsd-net/2022-February/001254.html

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
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:

#ifndef INVARIANTS
if (bootverbose)
#endif
	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.