Page MenuHomeFreeBSD

Add KASSERT_WARN
Needs ReviewPublic

Authored by bdrewery on Sep 25 2014, 1:04 AM.
Tags
None
Referenced Files
F157045451: D829.id1749.diff
Mon, May 18, 2:56 AM
F156949449: D829.diff
Sun, May 17, 2:17 PM
Unknown Object (File)
Sat, Apr 18, 8:44 PM
Unknown Object (File)
Apr 17 2026, 1:27 PM
Unknown Object (File)
Apr 16 2026, 11:33 AM
Unknown Object (File)
Apr 6 2026, 11:57 PM
Unknown Object (File)
Mar 10 2026, 2:36 PM
Unknown Object (File)
Feb 21 2026, 3:43 AM
Subscribers
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

KASSERT_WARN is for adding KASSERTS where the impact is unknown but it makes
sense to check for the condition. It is expected that the use would
eventually be changed to a full KASSERT.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bdrewery retitled this revision from to Add KASSERT_WARN.
bdrewery updated this object.
bdrewery edited the test plan for this revision. (Show Details)
ngie added inline comments.
sys/kern/kern_shutdown.c
617

Should "fatal" be boolean?

627–628

I would update the comment here to add in "fatal".

sys/sys/systm.h
91

Existing style(9) bugs in the macro with a space missing between exp and msg?

sys/kern/kern_shutdown.c
617

I'm not sure. I almost went with boolean but am unclear on its acceptance in kernel code.

emaste added inline comments.
sys/kern/kern_shutdown.c
617

style(9) now mentions boolean, and for new code I believe we should use it.