- User Since
- May 15 2018, 3:36 AM (264 w, 4 d)
Jan 14 2023
Changes based on Allan's comments:
- Moved a comment around
- Changed the name of a variable to have its purpose clearer
- Updated a comment to reflect the current prototype
Mar 11 2022
Nov 12 2021
Nov 11 2021
The code looks fine (small change, after all), other than my request for comments. :)
May 30 2021
May 18 2021
May 17 2021
May 13 2021
I still use an 80 character limit, and an 80 character-wide Terminal window.
Jul 11 2020
I think the best way forward is to start with -@ first and leave -l output as it is, at first.
Jul 10 2020
Jul 1 2020
Jun 21 2020
Long long long overdue.
Jun 20 2020
Jun 18 2020
May 27 2020
May 24 2020
May 15 2020
May 10 2020
Oh, an example output:
Mar 8 2020
I have not forgotten about this; I'm working on another aspect of this (dataset properties), and am holding off on committing this a bit.
Feb 24 2020
Feb 17 2020
New version coming now. Thanks!
Feb 16 2020
Nov 13 2019
Nov 10 2019
My only concern involves the /etc symlink possibly pointing at a non-existent file. This is a small concern not worth preventing this.
Nov 3 2019
Great, thanks! :) I'll go over it in more detail over the next few days.
A quick skim and the main question I had is: no acl check?
Oct 1 2019
Sep 28 2019
Sep 24 2019
There is no way to get a "default" implementation for this.
Perhaps I'm confused, but since you've added a new VFS operation, shouldn't there be a default implementation? You mention doing this for ZFS, but it's only there for tmpfs and UFS; does this need to be a VFS function, or should it be done as part of the mount structure and maintenance functions?
Sep 23 2019
Sep 21 2019
Remove thread argument to vfs_emptydir. Also remove a blank line. Per kib.
The td arg is still there.
Any further comments on this? I'd gotten an OOB message from Josh about this. If not, I'll make sure it's still functional and plan on committing this weekend or Monday.
Sep 18 2019
Sep 15 2019
I forgot to add the assertion kib suggested. (NB, still doesn't have the td changes we discussed.)
Sep 14 2019
More discussion about it than it's worth. Ah well.
BTW, you marked it as needing revision, but didn't say what revision.
Could you please tell more about this?
If you need more than one sync, then you've broken posix semantics.
If you think that it's useful and sync 3 would be an improvement, then could you please explain why?
Small change from kib feedback.
Sep 13 2019
More changes due to feedback from kib. Thanks!
New update coming in.
Feedback from kib.
Updated diff coming in a few seconds.
Sep 11 2019
Sep 6 2019
Feedback from Alan.
Sep 5 2019
Sep 3 2019
Sep 1 2019
Add support for having "cover" and "noemptydir" to the options parsing.
Aug 29 2019
Update the man page dates per reminder.
Aug 28 2019
Aug 13 2019
Jun 28 2019
Still no examples, but linking to the consumer helps.
Jun 25 2019
As for rounding - the bugs section, which mentioned them, is gone. The code is just a fixed-point library, so the rounding is what one would expect, I suppose: stuff gets truncated according to the radix point. Should I document it somehow?
Jun 22 2019
This much work in the kernel for what should be user-space makes me twitchy... but there is lots of precedent for it, so I obviously have to get with the times.
Jun 13 2019
I'm too old for the world.
The spilt-up man pages are much better. The examples definitely help a lot. But now I'm wondering about things like rounding -- the obvious usage that occurred to me here was, ahem, ZZZZZZ9.99, to borrow from a long-ago past :).
Jun 5 2019
This mostly seems ok -- it's pretty much nothing but a man page and header file, after all -- but my biggest complaint is that the man page is exceedingly complicated, and does not provide any real examples. The examples would also be a good way to show the justification for the code.
Jun 4 2019
Jun 3 2019
I'd like to see some example usages.
May 25 2019
May 24 2019
A bit late, but the updated diff promised. Thanks cem & jhb!
Doing testing with the changes, then I'll upload the diffs.
May 23 2019
I'll get to those tomorrow, thanks! (Various stuff going on today has thrown my schedule off.)
May 3 2019
The promised updated diffs.
I'm updating and build testing before uploading new diffs.
May 1 2019
Apr 24 2019
Something made me think that it was not acceptable, but now I can't remember what it was, so it's clearly wrong.
Apr 10 2019
I lost the length descriptor prefix in the last change. I've put it back, and run cryptocheck with -A lengths of 0, 13, 16, 32, 192102, and 127091. (User-space can't test more than 256k unfortunately.)
Review feedback incorporated.
New diff coming right after this.