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)
Sat, Feb 15, 7:08 PM
Unknown Object (File)
Feb 4 2025, 2:19 AM
Unknown Object (File)
Jan 25 2025, 4:47 AM
Unknown Object (File)
Jan 17 2025, 7:07 PM
Unknown Object (File)
Jan 14 2025, 4:11 AM
Unknown Object (File)
Jan 11 2025, 7:20 PM
Unknown Object (File)
Jan 10 2025, 9:27 AM
Unknown Object (File)
Dec 22 2024, 7:22 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 45656
Build 42544: arc lint + arc unit

Event Timeline

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

Could use some verbose message here:)

1916

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

2066

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