Page MenuHomeFreeBSD

zfs: Increase compatibility for different environments
ClosedPublic

Authored by imp on Jul 24 2022, 5:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 17, 4:09 PM
Unknown Object (File)
Oct 1 2024, 7:19 PM
Unknown Object (File)
Sep 23 2024, 8:36 PM
Unknown Object (File)
Sep 21 2024, 6:34 AM
Unknown Object (File)
Sep 17 2024, 4:54 PM
Unknown Object (File)
Sep 17 2024, 11:40 AM
Unknown Object (File)
Sep 16 2024, 4:32 PM
Unknown Object (File)
Sep 16 2024, 5:23 AM
Subscribers

Details

Summary

libsa uses the full OpenZFS compilation environment when we build this
included in zfs.c there. Other parts of the tree have not been adapted
to the full OpenZFS environment yet and need these ASSERT* defines to
build properly. Since the ASSERT* macros are normally defined in
sys/debug.h in the OpenZFS compatibility spl, only define them when the
guard for it (_SPL_DEBUG_H) is not defined. This should cover the
non-libsa boot loaders (gptzfsboot in particular) that need to use
zfssubr.c.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Jul 24 2022, 5:45 PM

I am wondering, if we should use #ifndef ASSERT etc instead.

This revision is now accepted and ready to land.Jul 24 2022, 7:37 PM

Move to using #ifndef ASSERT3S
Commit message updated locally.

This revision now requires review to proceed.Jul 24 2022, 8:06 PM
This revision is now accepted and ready to land.Jul 24 2022, 8:15 PM