Page MenuHomeFreeBSD

stand/libsa: zfs use standard ZFS_EARLY stuff
ClosedPublic

Authored by imp on Jul 24 2022, 5:45 PM.
Tags
None
Referenced Files
F102981252: D35894.diff
Tue, Nov 19, 11:00 AM
Unknown Object (File)
Oct 18 2024, 11:10 PM
Unknown Object (File)
Oct 10 2024, 5:03 AM
Unknown Object (File)
Oct 1 2024, 10:36 PM
Unknown Object (File)
Sep 25 2024, 6:34 AM
Unknown Object (File)
Sep 24 2024, 11:33 PM
Unknown Object (File)
Sep 24 2024, 8:27 PM
Unknown Object (File)
Sep 23 2024, 5:12 PM
Subscribers

Details

Summary

Now that the minor issues preventing zfs.c from using CFLAGS_EARLY have
been fixed, use that mechanism like everything else that needs the
OpenZFS spl headers. This simplifies things somewhat. Update comments to
document why zfs.c is still special, though in different ways.

Note: We also use the fact that NEED_SOLARIS_BOOLEAN is only defined in
an environment where the solaris compat boolean stuff will be defined
prior to this point (eg, when we're building zfs.c in libsa), but not in
other environments (like when we're building mkimage and various
stand-alone boot loaders that don't use libsa for space reasons (though
they should be changed to use the same ZFS compile env in the fullness
of time). This has to be done in the same change as using the ZFS_EARLY
stuff for zfs.c to not break zfs.c building for one commit.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Jul 24 2022, 5:45 PM
This revision is now accepted and ready to land.Jul 24 2022, 7:27 PM

We have to do the boolean_t and ZFS_EARLY change at the same time
commit message updated locally.

This revision now requires review to proceed.Jul 24 2022, 8:07 PM
This revision is now accepted and ready to land.Jul 24 2022, 8:20 PM
This revision was automatically updated to reflect the committed changes.