In D20324#443703, @trasz wrote:I'll elaborate more soon, but basically, this is in the context of https://reviews.freebsd.org/D20477. In short: this is less about using it "when you need a tree", and more about having a tree that you can easily transfer between domains, in this case between the kernel and userspace.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Jun 7 2019
Jun 7 2019
So like, what's the motivation of this change? What are the goals? How was this design chosen? There's 6000 lines of diff present as a fait accompli without any kind of socialization on arch@ or anywhere else. Code is great, but the high level stuff is missing.
The concept is cool but it's impossible to closely review 2000 lines of novel content. On the plus side, half of that is manual page and 1/4 is test cases, so that increases confidence somewhat. Still 500 lines of dense header macros with approximately zero comments (just API specification).
Adjust test to pass bytes, rather than block count.
cem added a reviewer for D20392: random(4): Add regression tests for uint128 implementation, Chacha CTR: secteam.
cem updated the diff for D20392: random(4): Add regression tests for uint128 implementation, Chacha CTR.
Rebase onto SVN head
- keystream API takes blocks, not bytes still
- Drop uint128_add64 API tests, which does not exist
Jun 6 2019
Jun 6 2019
In D20529#443815, @vangyzen wrote:In my opinion, no. To my knowledge, I'm the only person to report this, and I only hit it by running OneFS under bhyve, which uses a larger MAXPHYS.
In D20529#443683, @vangyzen wrote:Fix it in bhyve instead
In D20529#443608, @bryanv wrote:What is the panic message?
Restore previous PAGE_SIZE lock yielding behavior.
Jun 5 2019
Jun 5 2019
In D20324#443471, @markj wrote:I think I understand this a bit better than I did yesterday. The idea is that you allocate your array of objects, and initialize the ARB with the array size, so there's an upper limit on the number of entries in the tree.
In D20324#443241, @markj wrote:I'm interested in hearing more about the motivation too.
cem added a comment to D20458: first step towards enforcing must-succeed semantics for bus accessors.
In D20458#443358, @avg wrote:I would like to commit this change as is now. And then change printf to KASSERT in a week rather than in a month.
Hope that no one would object.
Jun 4 2019
Jun 4 2019
What is the benefit of this new data structure over existing data structures? Do you have empirical results?
In D20508#443091, @jhb wrote:I find that less readable to add a cast for one of the 4 calls to this function in an expression. This route seems to avoid the need to do similar hacks when decoding other multi-byte values in the future as well.
daemon(8): Don't block SIGTERM during restart delay
virtio(4): Add PNP match metadata for virtio devices
virtio(4): Expose PNP metadata through newbus
@bryanv , do you have any remaining concerns with this one? Thanks!
@bryanv, do you have any remaining concerns about this one or could it go in?
virtio_random(4): Fix random(4) integration
Jun 3 2019
Jun 3 2019
style.9: Codify tolerance for eliding blank lines
I'd cast the left shift instead (leaving the return type matching the type and range of the decoded value), but don't feel strongly about it. This works.
Jun 2 2019
Jun 2 2019
- Drop unneeded headers
- Only allow first virtio_random device to attach
Jun 1 2019
Jun 1 2019
In D20419#442441, @bryanv wrote:Other drivers following the callout pattern seem to be: bcm2835_rng.c, octeon_rnd.c, hifn7751.c, glxsb.c, tpm20.c although perhaps there is a desired functional difference that I am not aware of.
random(4): Fix RANDOM_LOADABLE build
May 30 2019
May 30 2019
cem accepted D20317: Netdump: coalesce writes less than buffer size for more efficient use of network.
LGTM!
cem added a comment to D20317: Netdump: coalesce writes less than buffer size for more efficient use of network.
In D20317#441987, @sam_samgwydir.com wrote:I added (offset != 0) to the predicate to ignore the final len=0, offset=0 write.
cem retitled D20466: random(4): Fix RANDOM_LOADABLE build from random(4): Fix RANDOM_LOADABLE option build to random(4): Fix RANDOM_LOADABLE build.
- Fix linking errors introduced inadvertently in r338324
- Correct MD HWRNG drivers in files.* to specify !random_loadable
May 29 2019
May 29 2019
Is it worth a ratelimit or do none of these show up often in practice? panic is inherently ratelimiting 😁
Consensus seems to be empty (blank) lines for functions with no locals should
be fully optional. Update the text to match.
If there is a cheap way to do it, it might be nice to stop counting at the first (latest) SVN revision. That way git starts over with lower "revision" numbers once an actual switch is made. It's also a shorter walk through git's object graph.
May 28 2019
May 28 2019
- Handle short reads.
Hey Bryan,
style.9: Correct usage's definition to match its declaration
In D20448#441414, @rgrimes wrote:I would even be fine with "If the function has no local variables",
- Use atomic_thread_fence_rel instead of wmb
- Fix lock leak
In D20419#441239, @bryanv wrote:It has been a long while since I wrote this driver but the callout pattern was certainly cribbed from another driver in the tree. Has this API changed over the years?
May 27 2019
May 27 2019
- Document in module.9
kldxref(8): Sort MDT_MODULE info first in linker.hints output
- Drop modevent handler changes.
In D19298#440806, @ngie wrote:
- This broke the build with gcc:
virtio_random(4): Remove unneeded reference to device
virtio.4: Add missing devices and Xr
aesni(4): Fix trivial type typo
sys/bufobj.h: Avoid using C++ reserved keyword 'private'
May 26 2019
May 26 2019
- Extract child_pnp_info routine to virtio.c using generic ivar routines.
- Add missing IVAR accessors to virtio_mmio(4).
May 25 2019
May 25 2019
In D20406#440587, @markj wrote:Looks ok to me, but I don't understand the tradeoffs involved in sharing the pnpinfo format among multiple bus types. Is it theoretically possible for there to be a collision, e.g., where two different drivers, one for mmio and one for pci, match on the same string?
Rework comment wording describing the change.
x11/terminology: Update to 1.4.1
virtio_pci(4): Fix typo in read_ivar method
May 24 2019
May 24 2019
Disable intr_storm_threshold mechanism by default