In D38858#885508, @rew wrote:! In D38858#884878, @gusev.vitaliy_gmail.com wrote:
Did you mean about format of the image files or something another? If format, I am going to resolve it with coming nvlist changes.you plan on converting the snapshot file format to an nvlist?
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Mar 3 2023
Mar 3 2023
Moved casper dependency to lib9p.
Added revert commit 966026246e62769f3bcd8247a47fe0f4f0433aba
gusev.vitaliy_gmail.com requested review of D38889: bhyvectl: Correct socket_fd closing in send_message().
Fixed @markj notes.
gusev.vitaliy_gmail.com added inline comments to D38872: bhyve: Exit with EX_OSERR if init checkpoint or restore time failed.
gusev.vitaliy_gmail.com updated the diff for D38872: bhyve: Exit with EX_OSERR if init checkpoint or restore time failed.
Fix note, use init_checkpoint_thread(ctx) != 0
gusev.vitaliy_gmail.com added inline comments to D38856: bhyve: [snapshot] Add cap limits for ipc socket.
gusev.vitaliy_gmail.com updated the diff for D38856: bhyve: [snapshot] Add cap limits for ipc socket.
Add empty line after #endif
gusev.vitaliy_gmail.com added inline comments to D38855: bhyve: [snapshot] Do not flush readonly device at blockif_pause().
Mar 2 2023
Mar 2 2023
Corrected according with @rew notes.
In D38858#884892, @rew wrote:In D38858#884878, @gusev.vitaliy_gmail.com wrote:Did you mean about format of the image files or something another? If format, I am going to resolve it with coming nvlist changes.
Have you sought out jhb's advice on this yet? I know he has expressed his opinion on it in the past.
In D38858#884827, @rew wrote:I still don't understand why you haven't addressed my comments that I raised in the original review. Do you feel it's not worth addressing or is that I'm out of line?
@corvink As you suggested I split this patch to the several reviews:
Rebase on to review https://reviews.freebsd.org/D38872
gusev.vitaliy_gmail.com edited reviewers for D38857: bhyve: Init checkpoint before caph_enter(), added: jhb; removed: • tonym_aai-corp.com, bhyve.
gusev.vitaliy_gmail.com edited reviewers for D38866: libvmm: Add missed ioctl to vm_ioctl_cmds, added: jhb; removed: • tonym_aai-corp.com, bhyve.
gusev.vitaliy_gmail.com requested review of D38856: bhyve: [snapshot] Add cap limits for ipc socket.
Mar 1 2023
Mar 1 2023
Correct 'fd' check:
@rew Could you take a look?
Update according to https://reviews.freebsd.org/D38836
Feb 28 2023
Feb 28 2023
Updated to the latest main branch. Open directory at bhyvectl side and pass fd to bhyve so that it can use openat() under capsicum.
gusev.vitaliy_gmail.com added a comment to D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
up?
Feb 22 2023
Feb 22 2023
gusev.vitaliy_gmail.com added a comment to D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
@jhb Could you look at this review ?
gusev.vitaliy_gmail.com added a comment to D38474: vmm: TSC offset is broken for resume after a7db532e3a6f 'vmm: Simplify saving ...'.
Could anyone commit it ?
gusev.vitaliy_gmail.com added a comment to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
Could anyone commit it ?
gusev.vitaliy_gmail.com added a comment to D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
Could anyone commit it ?
Feb 17 2023
Feb 17 2023
gusev.vitaliy_gmail.com added inline comments to D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
gusev.vitaliy_gmail.com updated the diff for D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
spinup_vcpu all guest-cpus with suspended state and later resume it. Resume only BSP in case of startup.
Feb 16 2023
Feb 16 2023
gusev.vitaliy_gmail.com added a comment to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
@jhb Updated diff accordingly with your note.
gusev.vitaliy_gmail.com updated the diff for D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
Removed "vmmops_snapshot" and related code.
Feb 14 2023
Feb 14 2023
gusev.vitaliy_gmail.com added a comment to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
@jhb Current things are good enough to proceed?
Feb 11 2023
Feb 11 2023
Feb 10 2023
Feb 10 2023
gusev.vitaliy_gmail.com updated subscribers of D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
gusev.vitaliy_gmail.com added a comment to D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
@jhb Could you review this patch, please? If possible it should be committed, since w/o patch, suspend/resume doesn't work with multi devices.
gusev.vitaliy_gmail.com updated the diff for D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
gusev.vitaliy_gmail.com added inline comments to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
gusev.vitaliy_gmail.com added inline comments to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
gusev.vitaliy_gmail.com added a comment to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
In D38476#875774, @markj wrote:The commit hash in the review title doesn't match anything in the src repo.
gusev.vitaliy_gmail.com added a comment to D38474: vmm: TSC offset is broken for resume after a7db532e3a6f 'vmm: Simplify saving ...'.
In D38474#875991, @jhb wrote:A more concise description for the log message might be "actually restore vcpu->tsc_offset".
gusev.vitaliy_gmail.com added inline comments to D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
gusev.vitaliy_gmail.com added inline comments to D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
gusev.vitaliy_gmail.com added inline comments to D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
gusev.vitaliy_gmail.com added inline comments to D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
gusev.vitaliy_gmail.com updated the test plan for D38476: bhyve: Resume is broken after 39ec056e6dbd 'vmm: Rework snapshotting ...'.
gusev.vitaliy_gmail.com updated the test plan for D38477: bhyve: Resume fails due to invalid guest state if guest_ncpus > 1.
Feb 9 2023
Feb 9 2023
Nov 21 2022
Nov 21 2022
Nov 18 2022
Nov 18 2022
gusev.vitaliy_gmail.com added inline comments to D37417: vmm: remove unneccessary rendezvous assertion.
gusev.vitaliy_gmail.com added inline comments to D37417: vmm: remove unneccessary rendezvous assertion.
Nov 15 2022
Nov 15 2022
Oct 21 2022
Oct 21 2022
gusev.vitaliy_gmail.com added a comment to D36244: bhyve: [snapshot] virtio-net features could be not negotiated.
@jhb Could you commit it?
Oct 19 2022
Oct 19 2022
gusev.vitaliy_gmail.com added a comment to D36244: bhyve: [snapshot] virtio-net features could be not negotiated.
@markj Could you commit it?
Aug 17 2022
Aug 17 2022
Aug 12 2022
Aug 12 2022
gusev.vitaliy_gmail.com added a comment to D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
@jhb up?
gusev.vitaliy_gmail.com added a comment to D36119: bhyve: Fix uses of uninitialized variables in pci_nvme.c.
In D36119#820585, @markj wrote:In D36119#820583, @gusev.vitaliy_gmail.com wrote:I don't see any changes in Makefile or rated. What warnings are you fixing?
In this review, a couple of warnings about uninitialized variables I found by removing the WARNS?=2 line in the Makefile and recompiling pci_nvme.c. D36118 contains fixes for most of the other warnings in that file. Everything else I did so far is not in phabricator yet.
Aug 11 2022
Aug 11 2022
gusev.vitaliy_gmail.com added a comment to D36119: bhyve: Fix uses of uninitialized variables in pci_nvme.c.
I don't see any changes in Makefile or rated. What warnings are you fixing?
Aug 10 2022
Aug 10 2022
gusev.vitaliy_gmail.com added a comment to D36119: bhyve: Fix uses of uninitialized variables in pci_nvme.c.
what if enable extra warning by default? It would help avoid adding new bugs.
Aug 1 2022
Aug 1 2022
gusev.vitaliy_gmail.com added a comment to D29262: Bhyve - Using JSON format for saving and restoring the state.
In D29262#817674, @mgrooms_shrew.net wrote:In D29262#817628, @gusev.vitaliy_gmail.com wrote:In D29262#815770, @ionut.mihalache1506_gmail.com wrote:What is the real purpose for this review? If debugging, I think this is not right way by adding huge amount of new code. I would suggest to remove all json things, remove all SNAPSHOT_VAR_OR_LEAVE and its variations and move basis to nvlist.
Using nvlist is more elegant and simpler, brings ability to extract (dump) all information for debugging, and gives to snapshot/resume dealing with optional parameters, removing some parameters, adding new parameters without breaking "resume" with previously created snapshots.
I've been funding work in collaboration with Mihai Carabas at UPB since 2016. The goal was to attract more developers to the bhyve project and implement suspend/resume with live migration. After the initial patch landed thanks to JHB, we spoke to project leaders about what needed to be done to move the suspend/resume code out from under #ifdef so it could become a generally useful feature. Mihai and I prioritized which projects should receive scholarship funding based on that feedback and engaged students that were willing to further these goals. One of the blocking items was to improve the state file format. This patch was the result of a student project to work on that item. Sadly, there has been little to no review or feedback. As a result, these considerable efforts are wasted, students eventually get frustrated and loose interest in working on FreeBSD projects.
We had a few discussions related to what file format to use. One of the options discussed was to use a format similar to the one used for the basic VM config file. Maybe Mihai can recall more specific details. I just remember that using json was the result of a consensus. If there was a new discussion to reach a new consensus, we'd be thrilled. That would at least be progress.
gusev.vitaliy_gmail.com added a comment to D29262: Bhyve - Using JSON format for saving and restoring the state.
In D29262#815770, @ionut.mihalache1506_gmail.com wrote:
Jul 21 2022
Jul 21 2022
gusev.vitaliy_gmail.com added a comment to D35447: vmm: save/restore pir_desc is required for posted-interrupts and VID.
@jhb up?
Jul 12 2022
Jul 12 2022
In D35328#811976, @andy_omniosce.org wrote:This change looks good to me. Is there anything stopping this being merged?
(I'm doing another sync to illumos at the end of the month and I'd like to include this if possible)
Jul 5 2022
Jul 5 2022
gusev.vitaliy_gmail.com added a comment to D35719: libvmm: add __BEGIN_DECLS/__END_DECLS for linking with c++ binaries.
@markj Could you approve again and commit?
gusev.vitaliy_gmail.com updated the diff for D35719: libvmm: add __BEGIN_DECLS/__END_DECLS for linking with c++ binaries.
Corrected include order according to Mark's comment.
Jul 1 2022
Jul 1 2022
gusev.vitaliy_gmail.com added a comment to D35447: vmm: save/restore pir_desc is required for posted-interrupts and VID.
In D35447#809008, @jhb wrote:Ok. Long term Patrick is more correct and we should not be storing this state but instead only trying to store "architectural" state like the APIC page, not the PIR, but I understand why for now we might need to do this.
Jun 29 2022
Jun 29 2022
gusev.vitaliy_gmail.com added a comment to D35447: vmm: save/restore pir_desc is required for posted-interrupts and VID.
@jhb ping?
Jun 24 2022
Jun 24 2022
gusev.vitaliy_gmail.com updated the summary of D35590: bhyve: multiple devices support for suspend/checkpoint/resume.
Jun 23 2022
Jun 23 2022
gusev.vitaliy_gmail.com added inline comments to D35575: bhyve: Support Hyper-V (base) and hyperv clock.