Various work on OpenZFS and ZFS/FreeBSD.
Mon, Jun 18
I have come to realise that there is another issue related to this, the default arc_max being wired ram that is not counted in max_wired means a default setup is allowed to wire more than the physical ram installed.
See my comment here for more explanation.
I have applied this patch to stable/11 r334820 and have been running without issue for 10 days.
Sat, Jun 16
Thu, Jun 14
Checked all 'Done' boxes, hope this is the way to use Phabricator ¯\_(ツ)_/¯
Apply suggestions from review round 0.
Tue, Jun 12
Thanks for the screenshot comparing the two man page renderings. I'm good with both ways.
Addressed all complaints marked 'Done' in my private tree, will post a revision once @bcr responds to how I should handle the newline style vs. whitespace issue.
Mon, Jun 11
Man page nit.
Adding range support for holds and bookmarks has been on my todo list for a while now.
Sun, Jun 10
Fri, Jun 8
Alexander pointed out that what I'd undone a change (range_tree_create()) because I'd ported from our 11-base system. Fixed that, and scrubbed a system a few times.
Sat, May 26
I don't think it is a very good idea to merge two pretty much independent ZoL commits into one FreeBSD commit. It could have sense if the first one would be significantly broken and second fixed the issue, but priority upgrade implemented in second commit is only a performance optimization, mostly unrelated to scrub even, in which case having it separate would make commit history cleaner for later comparison.
A late sequential-scan-related patch was mentioned when I created the PR for OpenZFS, it's at https://github.com/zfsonlinux/zfs/commit/a8b2e30685c9214c
Fri, May 25
Just a quick question, is this the same change that was recently presented at OpenZFS summit (http://open-zfs.org/w/images/a/a0/Saso_-_resilver_update.pdf)?
Or an alternative to it?
Thu, May 24
Tue, May 22
Unified for -stable and current now
It is only solution to live-lock problem I encounter on my server when there are massive-parallel fast download.
May 15 2018
May 11 2018
Update to latest -STABLE changes
Apr 27 2018
Mar 14 2018
Mar 12 2018
Jan 22 2018
@allanjude / @mahrens , it's worth pointing out that we eventually abandoned this change and went back to a 4K ABD chunk size. So while1K may have been more memory efficient in the short term, it ended up being suboptimal in the long run. I'm abandoning this issue and hoping no one repeats our lessons.
Dec 19 2017
just for whom it may concern: I've rolled back r326619 and reapplied slw patch on top - again, on "works for me" basics.
Dec 17 2017
Dec 7 2017
Nov 19 2017
I'll try to get some time allocated to that - right now the machine that
I use for this effort is busy doing other things but I may be able to
free up some time on it in the next week or two.
Well that's a bitch.
Oct 4 2017
"works for me"
(survived make buildworld two times in a row ;-) kernel from clean sources hangs, as usual, somewhere in stage3/clang)
Oct 1 2017
Confirmed that the latest patch Karl uploaded on Bugzilla bug 187594  (merged his patch with the one here) is...
Sep 30 2017
Sep 29 2017
Being new to phabricator.... how do I upload a diff that is off a (probably) different base rev and has my changes in it (I don't really want to "update" the existing diff, or do I?)
Definitely; I have the patch here in along with additional changes derived from my previous work running a soak test; should have some commentary and perhaps additional suggestions in a few days.
Sep 28 2017
As some of you probably know I've been chasing this same general issue here: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=187594
Sep 20 2017
@seanc OpenZFS/illumos has been at 4K chunk size since ABD was introduced. I'm not aware of any discussion around changing that.
avg@ added the loader tunable to make this adjustable in https://svnweb.freebsd.org/base?view=revision&revision=323797 however I have every reason to believe (and measurements from production) that 4K is a going to result in more waste than a 1K default. A 1K default may result in fragmentation with 4K slabs but there is less waste for everyone with 1K than 4K.