Page MenuHomeFreeBSD
Differential D50720

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

Authored by jrtc27 on Fri, Jun 6, 2:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 5, 12:58 AM
Unknown Object (File)
Thu, Jul 3, 4:38 PM
Unknown Object (File)
Wed, Jul 2, 11:47 AM
Unknown Object (File)
Tue, Jul 1, 4:12 PM
Unknown Object (File)
Tue, Jul 1, 11:06 AM
Unknown Object (File)
Mon, Jun 30, 2:35 PM
Unknown Object (File)
Mon, Jun 30, 1:08 PM
Unknown Object (File)
Mon, Jun 30, 11:09 AM
View All 28 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 64684
Build 61568: arc lint + arc unit

Event Timeline

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

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

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

Yes, thanks

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

Move endif

This revision now requires review to proceed.Fri, Jun 6, 5:18 PM
Harbormaster completed remote builds in B64684: Diff 156645.Fri, Jun 6, 5:18 PM
jrtc27 marked an inline comment as done.Fri, Jun 6, 5:19 PM
markj accepted this revision.Fri, Jun 6, 5:52 PM
This revision is now accepted and ready to land.Fri, Jun 6, 5:52 PM
Closed by commit rGb4f9be7fb371: openzfs: Add a _WANT_ZNODE that defines struct znode / znode_t for userspace (authored by jrtc27). · Explain WhyMon, Jun 16, 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/
9 lines
sys/
2 lines
5 lines

Diff 156645

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...