kassert: Turn 'poisoned_buf' into a 'void *'
Needs ReviewPublic
Actions

Authored by olce on Wed, Apr 23, 10:04 AM.

Tags

None

Referenced Files

	F115426808: D49971.id.diff
	Wed, Apr 23, 5:10 PM

	F115425402: D49971.id.diff
	Wed, Apr 23, 4:43 PM

	F115425036: D49971.id154149.diff
	Wed, Apr 23, 4:37 PM

	F115424917: D49971.id154149.diff
	Wed, Apr 23, 4:35 PM

	F115423420: D49971.diff
	Wed, Apr 23, 4:08 PM

	F115423039: D49971.diff
	Wed, Apr 23, 4:05 PM

Subscribers

imp

Details

Reviewers

markj
emaste

Summary

This is a step to make <sys/kassert.h> usable without the need to
explicitly include other headers.

'poisoned_buf' is only used for its value and never subject to pointer
arithmetics. Keeping 'caddr_t' would require <sys/kassert.h> to include
<sys/types.h> (at least on INVARIANTS).

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63658
Build 60542: arc lint + arc unit

Event Timeline

olce created this revision.Wed, Apr 23, 10:04 AM

Herald added a subscriber: imp. · View Herald TranscriptWed, Apr 23, 10:04 AM

olce requested review of this revision.Wed, Apr 23, 10:04 AM

Harbormaster completed remote builds in B63658: Diff 154149.Wed, Apr 23, 10:04 AM

olce added a parent revision: D49970: queue: Consistent single space after all #define.Wed, Apr 23, 10:19 AM

olce added a child revision: D49972: kassert: Explicitly include <sys/_types>.

Revision Contents
Changeset List

Path

Size

sys/

kern/

vfs_bio.c

2 lines

sys/

kassert.h

4 lines

Diff 154149

View Options

sys/kern/vfs_bio.c