Page MenuHomeFreeBSD

Expose ip6?_var.h kernel decls to sysunit tests
AbandonedPublic

Authored by rstone on Jan 25 2022, 9:36 PM.
Tags
None
Referenced Files
F104718483: D34038.diff
Mon, Dec 9, 2:07 AM
Unknown Object (File)
Mon, Dec 2, 8:33 PM
Unknown Object (File)
Oct 5 2024, 4:49 PM
Unknown Object (File)
Sep 13 2024, 6:37 AM
Unknown Object (File)
Sep 8 2024, 12:39 PM
Unknown Object (File)
Sep 4 2024, 4:26 PM
Unknown Object (File)
Sep 1 2024, 4:22 AM
Unknown Object (File)
Jul 26 2024, 8:44 PM

Details

Reviewers
vangyzen
imp

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44095
Build 40983: arc lint + arc unit

Event Timeline

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

Can you please provide more context? Why do we need this? Do we need to go around all of the includes and substitute _KERNEL to "_KERNEL || KERNEL_UT"? Why can't the unit tests just define _KERNEL instead?

Can you please provide more context? Why do we need this? Do we need to go around all of the includes and substitute _KERNEL to "_KERNEL || KERNEL_UT"? Why can't the unit tests just define _KERNEL instead?

Ryan gave more context in D34034, which is another in this series of similar reviews.

sys/netinet6/ip6_var.h
259–261

You could remove this #endif / #if pair.

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.