Page MenuHomeFreeBSD

Add delete throttle functionality from upstream openzfs
ClosedPublic

Authored by jpaetzel on Dec 31 2016, 9:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 7:48 PM
Unknown Object (File)
Fri, Dec 13, 6:46 PM
Unknown Object (File)
Sep 28 2024, 3:27 AM
Unknown Object (File)
Sep 27 2024, 8:30 AM
Unknown Object (File)
Sep 23 2024, 4:43 AM
Unknown Object (File)
Sep 18 2024, 10:28 PM
Unknown Object (File)
Sep 18 2024, 1:23 AM
Unknown Object (File)
Sep 11 2024, 1:54 PM
Subscribers

Details

Summary

ZFS I/O is done in txg groups. If you are deleting a large amount of data txg groups will be immediately filled with deletes, starving out other write I/O. This is especially easy to see if you run a local delete on an NFS fileserver. NFS writes will stall until the delete is complete.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jpaetzel retitled this revision from to Add delete throttle functionality from upstream openzfs.
jpaetzel updated this object.
jpaetzel edited the test plan for this revision. (Show Details)
jpaetzel added reviewers: mav, avg, delphij.

The default value for this sysctl (30) is taken from upstream, but prior to this change the default is essentially 0 (this feature is disabled)

While this does change the behavior of ZFS it's mostly a positive thing. In my testing setting the sysctl to 60 essentially preserves the default behavior of ZFS if you are doing local I/O only. The upstream value of 30 slows down local deletes by ~ 15% if the delete I/O is the only load on the system.

Also, Nexenta has been running with this change for a year, but they don't deal so much with local filesystem I/O.

Josh, it seems that this change is mostly a merge of an upstream, illumos, change.
So, please make sure to import the upstream change (illumos commit ff5177ee8bf9a355131ce2cc61ae2da6a5a6fdd6) into the svn vendor area vendor-sys/illumos and merge the change from there.
Also, please follow the loosely defined convention on how commit messages for illumos imports should look like. There are examples in commit logs.

I've done the vendor import but keep getting tree conflicts when trying to merge it to HEAD

I've done the vendor import but keep getting tree conflicts when trying to merge it to HEAD

There are two possibilities. Either the conflicts are caused by a FreeBSD change and in that case you have to resolve them. Or the conflicts are because of earlier upstream changes that we have not imported yet, in that case it's best to import them.

Hmm, tree conflicts...
This is a command that I use to merge from the illumos system vendor area:

svn merge -c XXXXXX ^/vendor-sys/illumos/dist sys/cddl/contrib/opensolaris/
This revision was automatically updated to reflect the committed changes.
jpaetzel mentioned this in rS312535: MFV 312436.