- User Since
- May 15 2018, 3:36 AM (98 w, 6 h)
Sun, Mar 8
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.
Apr 8 2019
Apr 6 2019
(I'll go through the rest of the comments this weekend.)
Apr 4 2019
Per feedback from cem, remove aes_block_t. Note that I did use a uint8_t pointer for some of the work, rather than having annoying typecasts in function calls.
Changed the GMAC_DIGEST_LEN uses to be sizeof(tag).
Responses to cem; a new patch incoming.
Mar 28 2019
Implement some (but not all, due to asking some questions) of the feedback from cem@.
Respond to cem. Updated diff to follow.
Mar 27 2019
Mar 14 2019
Mar 11 2019
Mar 1 2019
Feb 27 2019
Feedback from jhb. (Just uploaded for completeness sake; I'll be checking it in after I update my svn source, apply the patch, and do a buildworld just to make sure I didn't do anything stupid again.)
Move the #includes around per cem.