kernel decls to sysunit tests
AbandonedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 6, 1:45 AM

	Unknown Object (File)
	Mon, Nov 25, 11:18 AM

	Unknown Object (File)
	Mon, Nov 25, 4:56 AM

	Unknown Object (File)
	Fri, Nov 22, 8:33 AM

	Unknown Object (File)
	Fri, Nov 22, 6:17 AM

	Unknown Object (File)
	Fri, Nov 15, 12:57 PM

	Unknown Object (File)
	Mon, Nov 11, 4:02 PM

	Unknown Object (File)
	Mon, Nov 11, 9:02 AM

View All 32 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