User Details
- User Since
- Oct 24 2014, 7:17 PM (395 w, 1 d)
Tue, May 17
Still need to track down problem cited by Peter Holm.
Changes suggested by kib and refinements suggested by Robert Morris
Mon, May 16
Sun, May 15
I concur that this note should be added.
Sat, May 14
Calculations and comment are accurate.
This is an accurate calculation the the extra needed space.
Tue, May 10
Thu, May 5
Sun, Apr 24
This seems like a useful change.
Apr 15 2022
I like the approach suggested by kib that ufs_open() not set VIRF_PGREAD when the page size is larger than the fragment size.
Apr 6 2022
I concur with the analysis. Waiting for trims in the current location is clearly too late. Waiting in the new location will ensure that all the dependencies get flushed. And once flushed it should not be possible to add any new ones thus no new trims should be able to appear.
Mar 17 2022
Looks good. Short and to the point.
Mar 16 2022
The main clients of making snapshots are dump(8), fsck_ffs(8), and mksnap_ffs(8) and none of them want to change any other attributes. Generally speaking, administrators do not use the mount(8) command directly to take snapshots. The one other change I would recommend is to document in the mount.8 manual page that when the "snapshot" option is specified, any other options will be ignored.
Mar 13 2022
I picked those names when ufs was the only filesystem in BSD. I should have renamed them when I added the VFS interface. Better late than never!
This change looks reasonable to me.
Mar 1 2022
Feb 28 2022
Feb 27 2022
Feb 23 2022
Feb 21 2022
minor format question. Change looks correct.
Feb 20 2022
This update reflects some minor cleanups that were reported by Peter Holm during his testing.
Feb 16 2022
Feb 15 2022
These changes address comments from all three reviewers.
Feb 6 2022
This patch eliminates the LOR in my test case.
Feb 4 2022
Feb 1 2022
I agree with this change.
Jan 31 2022
Eivind Eklund was the person that first added KASSERT which was the start of what probably should have been called kassert.h. Many others added to it over time probably putting more effort into than he did. But I am inclined to give him credit for starting the ball rolling. He was an active committer from February 1997 until June 2005 (461 base, 69 ports, 26 doc). I note that he still has a freefall account.
Jan 30 2022
Useful cleanup.
Once passing Peter Holm's testing, this looks good to go.
Jan 28 2022
Based on our previous discussions in D33921 these changes improve readability and consistency.
Jan 27 2022
I think that this change is an improvement.
This looks good to me.
Jan 20 2022
Changes to avoid needing to free newb look good. Really glad to not have to figure out how to do that. Other changes look right. Awaiting Peter's tests to see if you have found all the needed changes.
Jan 19 2022
Generally looks good. Definitely want to see if Peter turns up anything. If Peter clears it, freeing newb will be necessary. Unfortunately freeing newb is going to require writing a bunch of code to unwind a lot of soft update stuff. It will be a lot easier if we reorder the code to get the buffer first and then only allocate the block once we know we will have a buffer for it. Since the buffer is hashed on lbn rather than bn we do not need to have a block allocated before we get its buffer.
This patch eliminates my bufwait <-> snaplk LOR warning when doing background fsck.
Jan 18 2022
Jan 10 2022
Jan 6 2022
Jan 5 2022
Latest updates all look good.
Jan 3 2022
Jan 2 2022
Looks good to me. If inspired a comment over msdosfs_remount_ro() would be nice, though the name pretty much says what it does.
Overall looks good. Glad to see it done.
Your comment wrap rework looks good. As you note, it is consistent with the way di_shortlink is commented.
I have made the commit to head and have scheduled MFC in two weeks.
I will certainly MFC it to 13. I have not set up an environment for MFC'ing to 12, but will do so if time permits.
Please send me your email address to mckusick@mckusick.com.
Dec 29 2021
Since the change is such a minor edit, I will make it without starting a new Phabricator review.
Thanks for pointing out the need for clarification.
Dec 28 2021
This looks like a useful change.
Dec 26 2021
See suggested clarification.
This looks good.
The idea and implementation look good. I would like my noted style comments handled.
Dec 16 2021
Dec 15 2021
Comment added. The function has always scanned the entire string and used the last instance of the option. Changing it now may break existing and working installations so I left it as is. Note that if it gets changed here it should be changed in the other six places to be consistent.
Add comment describing getfsopt(). Historical footnote: this code originated in 4.4BSD.
Incorporate feedback from kib.
I changed the name from hasopt() to getfsopt() in keeping with the other names in getfsent(3) so that it can be added more easily in the future if desired. At the moment there are variations of getfsopt() used in seven places in the base system lib/libutil/quotafile.c, libexec/rpc.rquotad/rquotad.c, sbin/fsck/fsck.c, sbin/mount/mount.c, sbin/swapon/swapon.c, usr.sbin/bsdinstall/partedit/gpart_ops.c, and usr.sbin/bsdinstall/partedit/partedit.c). Updated diff to follow shortly.
Dec 14 2021
Nov 23 2021
Nov 16 2021
Nov 15 2021
Nov 11 2021
Nov 10 2021
Sorry for my long delay.
Nov 9 2021
I have added comments / questions to your three latest (helpful) comments. I would also like your feedback on my question to your earlier comment at line 340 of g_union.c.