Page MenuHomeFreeBSD

Add KASSERT_WARN
Needs ReviewPublic

Authored by bdrewery on Sep 25 2014, 1:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 1:04 AM
Unknown Object (File)
Feb 26 2024, 4:17 AM
Unknown Object (File)
Jan 5 2024, 11:18 PM
Unknown Object (File)
Dec 19 2023, 11:02 PM
Unknown Object (File)
Nov 29 2023, 1:26 AM
Unknown Object (File)
Nov 21 2023, 6:50 PM
Unknown Object (File)
Nov 15 2023, 1:06 PM
Unknown Object (File)
Nov 13 2023, 4:48 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.