kernel decls to sysunit tests
AbandonedPublic
Actions

Authored by rstone on Jan 25 2022, 9:36 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 21, 1:54 PM

	Unknown Object (File)
	Thu, May 21, 8:41 AM

	Unknown Object (File)
	Wed, May 20, 5:15 PM

	Unknown Object (File)
	Sun, May 17, 2:24 PM

	Unknown Object (File)
	Sat, May 16, 8:32 PM

	Unknown Object (File)
	Wed, May 13, 5:36 PM

	Unknown Object (File)
	Sun, May 10, 11:12 PM

	Unknown Object (File)
	Sun, May 10, 11:08 PM

View All 89 Files

Subscribers

imp

Details

Reviewers

vangyzen
imp

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 44093
Build 40981: arc lint + arc unit

Event Timeline

rstone created this revision.Jan 25 2022, 9:36 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 25 2022, 9:36 PM

rstone requested review of this revision.Jan 25 2022, 9:36 PM

Harbormaster completed remote builds in B44093: Diff 101925.Jan 25 2022, 9:36 PM

btw, same comment for all, but this matches the suggestions that came out of the IRC discussions...

This revision is now accepted and ready to land.Jan 25 2022, 9:59 PM

vangyzen added inline comments.Jan 27 2022, 1:57 AM

sys/sys/systm.h
116	`_KERNEL \|\| _KERNEL_UT` Missed because it was wrong already? (I like vim's `%` command for finding the matching `#if / #endif`.)

glebius@ has requested a different approach where unit tests define _KERNEL and opt out of the KPI symbols that they don't want, rather than opting into the symbols that they do want.

Revision Contents
Changeset List

Path

Size

sys/

systm.h

6 lines

Diff 101925

View Options

Expose <sys/systm.h> kernel decls to sysunit testsAbandonedPublicActions