Page MenuHomeFreeBSD

Wire up OpenZFS with VOP_DEALLOCATE(9)
AbandonedPublic

Authored by khng on Feb 21 2021, 8:38 AM.
Tags
None
Referenced Files
F81550761: D28834.id84405.diff
Wed, Apr 17, 10:05 PM
Unknown Object (File)
Thu, Apr 11, 7:39 PM
Unknown Object (File)
Sat, Apr 6, 12:59 PM
Unknown Object (File)
Sat, Apr 6, 10:56 AM
Unknown Object (File)
Fri, Mar 29, 12:35 PM
Unknown Object (File)
Mar 18 2024, 5:35 AM
Unknown Object (File)
Feb 15 2024, 12:19 PM
Unknown Object (File)
Feb 10 2024, 6:03 PM

Details

Reviewers
markj
allanjude
Summary

Wire up OpenZFS with VOP_DEALLOCATE(9)

This will go to upstream actually. However a differential is created for
easier tools/tools/git/git-arc.sh workflow.

Submitted by: Ka Ho Ng <khng@freebsdfoundation.org>
Sponsored by: The FreeBSD Foundation

Diff Detail

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

Event Timeline

khng requested review of this revision.Feb 21 2021, 8:38 AM

Rebased on top of the newest D27194.

khng edited the summary of this revision. (Show Details)
sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c
3832

Does this comment need updating to instead say what FreeBSD does in this case?

sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c
3832

That seems necessary. I don't recall we have the same limitation like Solaris that VOP can only issued on opened files.

khng marked an inline comment as not done.

Removed SPACECTL_F_CANEXPAND support on SPACECTL_DEALLOC

Can't think of any use of it right now.

  • New zfs_deallocate and new DMU api instead of calling into zfs_freesp

_PC_FDEALLOC_PRESENT -> _PC_DEALLOC_PRESENT