Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vIOMMU: Add a case of migration #5217

Merged
merged 1 commit into from
Oct 13, 2023
Merged

Conversation

Yingshun
Copy link
Contributor

@Yingshun Yingshun commented Oct 8, 2023

This PR adds:
VIRT-294806: [vIOMMU] Migrate vm with iommu device

Test results: A failed case is due to a known issue.

 (1/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.virtio_muti_devices.vhost_on.virtio: PASS (207.00 s)
 (2/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.virtio_muti_devices.vhost_on.intel: ERROR: Login timeout expired    (output: 'exceeded 240 s timeout, last failure: Could not verify DHCP lease: 52:54:00:0d:38:53 --> 192.168.122.146') (594.96 s)
 (3/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.virtio_muti_devices.vhost_off.virtio: PASS (209.01 s)
 (4/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.virtio_muti_devices.vhost_off.intel: PASS (351.59 s)
 (5/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.scsi_controller.virtio: PASS (210.06 s)
 (6/6) type_specific.io-github-autotest-libvirt.vIOMMU.migration.iommu_device.scsi_controller.intel: PASS (353.35 s)

@Yingshun Yingshun force-pushed the viommu_mig branch 2 times, most recently from 939729d to 3b8a800 Compare October 8, 2023 09:01
@Yingshun
Copy link
Contributor Author

Yingshun commented Oct 8, 2023

@cliping can you plz help review this pr? Thanks!

Copy link
Contributor

@cliping cliping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Others LGTM

image_convert = 'no'
virsh_migrate_dest_state = "running"
virsh_migrate_src_state = "shut off"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe remove this blank line

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated.

virsh_migrate_options = "--p2p --live --verbose --persistent"
virsh_migrate_desturi = "qemu+ssh://${migrate_dest_host}/system"
virsh_migrate_connect_uri = "qemu:///system"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated, thx!

This PR adds:
    VIRT-294806: [vIOMMU] Migrate vm with iommu device

Signed-off-by: Yingshun Cui <[email protected]>
@chloerh chloerh merged commit a2a9856 into autotest:master Oct 13, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants