- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Feb 21 2024
Feb 19 2024
Feb 1 2024
Jan 22 2024
Jan 20 2024
Jan 19 2024
In D43505#991773, @oshogbo wrote:What about the header fields we have discussed?
In D43505#991783, @imp wrote:Id also be tempted to always do the requests in host endian and do the happing before we submit. Likewise with the replies. Not sure how feasible that is though.
Jan 18 2024
Jan 17 2024
Jan 13 2024
In D43425#990237, @delphij wrote:In D43425#990231, @rew wrote:For ZFS, the argument passed to the shell script is the
pool name to import.Ah you are right, but the name is not guaranteed to be identical to the filename. Probably zpool import -a -d /dev/$(mdconfig $0).uzip instead? (and maybe -R $1)
incorporate feedback:
Jan 12 2024
Jan 10 2024
Jan 9 2024
Jan 8 2024
- drop vm_page_all_valid() change
- fill added leaf page with mappings
Jan 5 2024
Jan 1 2024
Dec 29 2023
In D43218#985292, @markj wrote:This seems fine as a stopgap. If you mail me git patches for this and the MTRR review, I'll apply them.
Dec 28 2023
This looks good - thanks for the fix.
Dec 19 2023
address feedback from jhb
Dec 14 2023
address feedback
Dec 12 2023
In D41605#980592, @markj wrote:Why exactly do we need special handling for FREVOKE?
Dec 11 2023
Do not clear knotes from the TTY until it gets dealloc'ed unless the TTY
is being revoked, in that case delete the knotes when closed is called
on the TTY.
Nov 21 2023
Nov 15 2023
Nov 12 2023
Tested and works as expected - this will be nice to have, thanks.
Nov 7 2023
Aug 28 2023
In D41605#948445, @markj wrote:How do they get removed? tty_dealloc() calls knlist_destroy(), but this presumes that knotes have already been dequeued.
In D41605#948294, @kib wrote:What would prevent this situation for other kinds of knotes?
In D41605#948243, @kib wrote:But then, what would guarantee liveness of the tty when its detached knote f_event method is called?
Aug 26 2023
In D41605#948130, @kib wrote:Can you show the backtrace where tty filter is called without tty lock, please?
update revision with missing chunk
Aug 23 2023
Jul 13 2023
Jul 7 2023
Jun 20 2023
In D34717#925263, @mgrooms_shrew.net wrote:I can't tell you how disappointing it is to see comments like this after a review sits in public review for months. You could have easily voiced any of these concerns during the time leading up to the commit instead of withholding them until after it was made. What exactly is your goal hear?
In D34717#925210, @afedorov wrote:@corvink , I think this commit is too early. Because we haven't finished supporting snapshots, but this commit already fixes the user interface for live migration, which is not finished at all.
Sorry for the late comment, but maybe this commit should be rolled back?
In D34718#925207, @afedorov wrote:I think we should bring support for snapshots first. And enable BHYVE_SNAPSHOT by default.
May 19 2023
looks like this is value is stated in RFC8907 section 6.1 under the authen_type header
May 16 2023
Thanks
In D40108#913426, @gusev.vitaliy_gmail.com wrote:At least it saves pdi.pi_cfgdata. Do you think it is waste doing and .pi_cfgdata can not be changed by guest OS or be changed between bhyve versions ?
May 15 2023
small nits - other than that, looks good
not sure that devices should define a snapshot handler when not supported by the device.
May 12 2023
May 5 2023
In D39979#910118, @corvink wrote:Do you think we should add this option now or should we wait until someone has some issues or a feature requires it?
It might be helpful for debugging purposes too.
Apr 25 2023
In D39805#906206, @markj wrote:Would anyone be willing to test this on an AMD system?
Mar 31 2023
so I assume some ACPI devices will have different emulation than others?
Mar 30 2023
Mar 29 2023
the comments from D39315 apply to this review as well
Mar 28 2023
Seems a bit odd making this public? as opposed to keeping a reference of the softc in gpu_passthru and passing that off to pci?
Mar 27 2023
Mar 15 2023
committed in rG2dfd9979d827b36a9ed6ef2d5911a755091216c7 or SVN r279342