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
F103302876: D34393.diff
Sat, Nov 23, 7:17 AM
Unknown Object (File)
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

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
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44600
Build 41488: arc lint + arc unit

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.