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.

Sign up for GitHub

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

Jump to bottom

block_with_iommu.cfg: limit iommu case for windows on Intel CPU only #4222

Merged
merged 1 commit into from
Dec 4, 2024
Merged

block_with_iommu.cfg: limit iommu case for windows on Intel CPU only #4222

merged 1 commit into from
Dec 4, 2024

Conversation

menli820
Copy link
Contributor

For currently we use the wrong limit parameter in the cfg. So we can not filter case on the intel machine. Update to the right parameter.

ID: 3144

@menli820
block_with_iommu.cfg: limit iommu case for windows on Intel CPU only
cb8b3aa 
For currently we use the wrong limit parameter in the cfg.
So we can not filter case on the intel machine. Update to
the right parameter.

Signed-off-by: menli <[email protected]>
@menli820
Copy link
Contributor Author

@qingwangrh @XueqiangWei please help to review this patch, thanks

@qingwangrh
Copy link
Contributor

@menli820, could you please post your test result here?

@menli820
Copy link
Contributor Author

menli820 commented Dec 4, 2024 •
edited
Loading

AMD host (not filter this case):

python3 ConfigTest.py --testcase=block_with_iommu --guestname=Win2019 --driveformat=virtio_blk --platform=x86_64 --machines=q35 --customsparams="cdrom_virtio = isos/windows/virtio-win-prewhql-0.1-266.iso" --firmware=default_bios --clone=yes --dryrun

(1/1) Host_RHEL.m9.u5.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: STARTED
(1/1) Host_RHEL.m9.u5.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: CANCEL: Test cancelled due to --dry-run (0.00 s)
RESULTS : PASS 0 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
JOB HTML : /var/tmp/avocado-dry-run-fwtn0xzw/job-2024-12-04T00.32-5da1f97/results.html
JOB TIME : 1.60 s

Intel host (can filter the case):
(1/2) Host_RHEL.m9.u6.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: STARTED
(1/2) Host_RHEL.m9.u6.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: CANCEL: Test cancelled due to --dry-run (0.00 s)
(2/2) Host_RHEL.m9.u6.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.block_with_iommu.with_installation.cdrom.extra_cdrom_ks.q35: STARTED
(2/2) Host_RHEL.m9.u6.qcow2.virtio_blk.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.block_with_iommu.with_installation.cdrom.extra_cdrom_ks.q35: CANCEL: Test cancelled due to --dry-run (0.00 s)
RESULTS : PASS 0 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 2
JOB HTML : /var/tmp/avocado-dry-run-f19jg727/job-2024-12-04T00.34-251345a/results.html
JOB TIME : 1.38 s

@qingwangrh
Copy link
Contributor

LGTM

XueqiangWei
XueqiangWei approved these changes Dec 4, 2024
View reviewed changes
Copy link
Contributor

@XueqiangWei XueqiangWei left a comment

Choose a reason for hiding this comment

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

LGTM,
Acked-by: [email protected]

@XueqiangWei XueqiangWei merged commit e7bdede into autotest:master Dec 4, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@XueqiangWei XueqiangWei XueqiangWei approved these changes

@qingwangrh qingwangrh qingwangrh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@menli820 @qingwangrh @XueqiangWei