Page MenuHomeFreeBSD

boot/zfs: Extend zfsimpl.h and make it easier to use
ClosedPublic

Authored by markj on May 20 2022, 11:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 23, 12:12 AM
Unknown Object (File)
May 27 2026, 12:36 PM
Unknown Object (File)
May 22 2026, 7:56 PM
Unknown Object (File)
May 15 2026, 4:11 PM
Unknown Object (File)
May 15 2026, 3:00 PM
Unknown Object (File)
May 15 2026, 2:33 PM
Unknown Object (File)
May 15 2026, 10:25 AM
Unknown Object (File)
May 14 2026, 4:00 PM
Subscribers

Details

Summary

Some makefs(8) patches make use of zfsimpl.h (not zfsimpl.c though) to
provide definitions for various on-disk structure. Most of this diff
simply adds new definitions that are useful.

Also reduce dependencies of the header:

  • remove an unused list_node_t field to drop the sys/list.h dependency
  • replace CTASSERT with _Static_assert

And fix the declaration of decode_embedded_bp_compressed().

Diff Detail

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

Event Timeline

sys/cddl/boot/zfs/zfsimpl.h
560–561

Could use some verbose message here:)

1913–1914

we are actually not using this structure at all, I think we can just drop it from here.

2062

extern maybe?

markj marked 3 inline comments as done.

Address review feedback

This revision is now accepted and ready to land.May 21 2022, 6:00 PM