Various work on OpenZFS and ZFS/FreeBSD.
Jun 7 2019
Jun 6 2019
Sorry, I myself went missing for a long while.
Yes, I can commit this change.
May 11 2019
Apr 9 2019
Mar 11 2019
Mar 7 2019
Feb 22 2019
Feb 21 2019
Feb 19 2019
Feb 14 2019
Let me try to contact George again.
Feb 13 2019
Feb 12 2019
While I see the problem you are fixing, the fix looks ugly to me, that is why I would look for something nicer. I agree that according to logic of remove_reference() dropping last reference for header in ghost state is a failure, but how can remove_reference() be called before the arc_access() just on following line? I would guess from description telling about the case of prefetch read it should happen no sooner then we actually initiate the I/O, which is done much later then those two lines. So while I agree it is somewhat odd to have buffer for header in ghost state, is that a criminal.
Feb 6 2019
Jan 24 2019
Thank you for the additions! Just my $0.05 worth of nit picks post commit.
Jan 17 2019
Update the diff based on Allan's comments. I've decided to remove the newsyslog compression example. This might be something better suited for the handbook. In it's place, I've added Allan's zfs snapshot range deletion examples. I've also changed the date in the custom property example to a more prominent date. ;-)
Jan 15 2019
Thanks for your feedback, @allanjude. I'll work on an updated patch that incorporates it.
Dec 23 2018
Adding more reviewers for more pairs of eyes for eventual approval.
Dec 14 2018
Update with suggestions by Dru.
These look great!
Dec 13 2018
Nov 9 2018
Oct 12 2018
Oct 11 2018
Aug 29 2018
- ARC don't rised, memory pressure does not arise, page daemon not activated.
The ARC is not growing after 8, but the ARC hit rate is too low. Why is it not growing? Is it because the free_memory < (arc_c >> arc_no_grow_shift) condition is true, or is there some other reason?
Aug 28 2018
Aug 23 2018
To be clear, I'm just stating that r332365 changed zfs_arc_free_target to be equal vm_cnt.v_free_target. It looks to me that this is equivalent to the change you made to arc_available_memory(EXCLUDE_ZONE_CACHE), where v_free_target is referenced directly.
Aug 22 2018
Sorry that this review has stalled lately. I would like to compare this patch to what's in -CURRENT, which has evolved a fair bit since the patch was updated. Once that picture is more clear, we can focus on stable/11.
Aug 7 2018
I can confirm that the patch solves the issues as described in the "lightly loaded system eats swap space" thread on freebsd-stable mailing list.
On one of my desktop systems the issue is particularly noticeable as the disk subsystem is rather slow, the system became unusable after a
weekend sitting idle. I have not experienced the issues anymore after applying the patch
Jul 9 2018
Jul 7 2018
What else is preventing this from getting merged?