Add KASSERT_WARN
Needs ReviewPublic
Actions

Authored by bdrewery on Sep 25 2014, 1:04 AM.

Tags

None

Referenced Files

	F165738954: D829.id1749.diff
	Sun, Aug 9, 2:31 PM

	F165727326: D829.diff
	Sun, Aug 9, 12:49 PM

	Unknown Object (File)
	Wed, Aug 5, 9:41 PM

	Unknown Object (File)
	Tue, Aug 4, 6:12 AM

	Unknown Object (File)
	Tue, Jul 14, 10:25 PM

	Unknown Object (File)
	Mon, Jul 13, 8:29 PM

	Unknown Object (File)
	Jul 7 2026, 2:15 PM

	Unknown Object (File)
	Jul 7 2026, 8:15 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 updated this revision to Diff 1749.Sep 25 2014, 1:04 AM

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 a subscriber: ngie.Sep 25 2014, 1:42 AM

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?