Page MenuHomeFreeBSD
Differential D50720

openzfs: Add a _WANT_ZNODE that defines struct znode / znode_t for userspace
ClosedPublic
Actions

Authored by jrtc27 on Jun 6 2025, 2:15 PM.
Tags
None
Referenced Files
F126346717: D50720.id156637.diff
Mon, Aug 18, 6:38 AM
Unknown Object (File)
Fri, Aug 15, 12:17 AM
Unknown Object (File)
Mon, Aug 11, 3:27 AM
Unknown Object (File)
Mon, Aug 11, 1:11 AM
Unknown Object (File)
Sun, Aug 10, 4:50 AM
Unknown Object (File)
Mon, Aug 4, 8:17 AM
Unknown Object (File)
Sat, Jul 26, 12:39 PM
Unknown Object (File)
Sat, Jul 26, 12:38 PM
View All 45 Files
Subscribers
delphij

Details

Reviewers
imp
kib
markj
mm
Commits
rGb4f9be7fb371: openzfs: Add a _WANT_ZNODE that defines struct znode / znode_t for userspace
Summary

Note that ZNODE_OS_FIELDS needs to change to using struct vnode over
vnode_t (matching struct zfsvfs rather than vnode_t) since vnode_t is
only defined in the kernel SPL, not the userspace SPL (libspl).

Diff Detail

Repository
rG FreeBSD src repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 64677
Build 61561: arc lint + arc unit

Event Timeline

jrtc27 created this revision.Jun 6 2025, 2:15 PM
Herald added a subscriber: delphij. · View Herald TranscriptJun 6 2025, 2:15 PM
jrtc27 requested review of this revision.Jun 6 2025, 2:15 PM
Harbormaster completed remote builds in B64677: Diff 156637.Jun 6 2025, 2:15 PM
jrtc27 added a child revision: D50721: libprocstat: Drop zfs_defs.c hack, including its _KERNEL define.Jun 6 2025, 2:16 PM
imp accepted this revision.Jun 6 2025, 2:50 PM
This revision is now accepted and ready to land.Jun 6 2025, 2:50 PM
markj added inline comments.Jun 6 2025, 5:02 PM
sys/contrib/openzfs/include/os/freebsd/zfs/sys/zfs_znode_impl.h
194

Doesn't this need to be before the __cplusplus closing brace?

jrtc27 added inline comments.Jun 6 2025, 5:03 PM
sys/contrib/openzfs/include/os/freebsd/zfs/sys/zfs_znode_impl.h
194

Yes, thanks

jrtc27 updated this revision to Diff 156645.Jun 6 2025, 5:18 PM

Move endif

This revision now requires review to proceed.Jun 6 2025, 5:18 PM
Harbormaster completed remote builds in B64684: Diff 156645.Jun 6 2025, 5:18 PM
jrtc27 marked an inline comment as done.Jun 6 2025, 5:19 PM
markj accepted this revision.Jun 6 2025, 5:52 PM
This revision is now accepted and ready to land.Jun 6 2025, 5:52 PM
Closed by commit rGb4f9be7fb371: openzfs: Add a _WANT_ZNODE that defines struct znode / znode_t for userspace (authored by jrtc27). · Explain WhyJun 16 2025, 9:37 PM
This revision was automatically updated to reflect the committed changes.
jrtc27 added a commit: rGb4f9be7fb371: openzfs: Add a _WANT_ZNODE that defines struct znode / znode_t for userspace.

Revision Contents
Changeset List

PathSize
sys/
contrib/
openzfs/
include/
os/
freebsd/
zfs/
sys/
8 lines
sys/
2 lines
5 lines

Diff 156637

View Options

sys/contrib/openzfs/include/os/freebsd/zfs/sys/zfs_znode_impl.h

Loading...
View Options

sys/contrib/openzfs/include/sys/zfs_vfsops.h

Loading...
View Options

sys/contrib/openzfs/include/sys/zfs_znode.h

Loading...