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

SRIOV: fix "Login timeout expired" because of 256 vcpus #6084

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

meinaLi
Copy link
Contributor

@meinaLi meinaLi commented Dec 20, 2024

The numbers of guest vcpu is better little than the host vcpus' number. But we don't have this host to match 256 cpus. So after discussing we decided to use the same host cpus' number to test guest vcpus.

The numbers of guest vcpu is better little than the host vcpus' number.
But we don't have this host to match 256 cpus. So after discussing we
decided to use the same host cpus' number to test guest vcpus.

Signed-off-by: Meina Li <[email protected]>
@meinaLi
Copy link
Contributor Author

meinaLi commented Dec 20, 2024

# avocado run --vt-type libvirt --vt-omit-data-loss --vt-machine-type q35 vIOMMU.intel_iommu_with_dma_translation.disable_dma_translation_with_more_vcpus
JOB ID     : 030d3d202e322d40c44251ae4d32ea05be5d5fb9
JOB LOG    : /var/log/avocado/job-results/job-2024-12-20T02.14-030d3d2/job.log
 (1/1) type_specific.io-github-autotest-libvirt.vIOMMU.intel_iommu_with_dma_translation.disable_dma_translation_with_more_vcpus: STARTED
 (1/1) type_specific.io-github-autotest-libvirt.vIOMMU.intel_iommu_with_dma_translation.disable_dma_translation_with_more_vcpus: PASS (160.60 s)
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /var/log/avocado/job-results/job-2024-12-20T02.14-030d3d2/results.html
JOB TIME   : 162.76 s

Comment on lines +27 to +28
cpu_cmd = "lscpu | grep '^CPU(s):'"
vcpus_output = process.run(cpu_cmd, shell=True, ignore_status=False).stdout_text
Copy link
Contributor

Choose a reason for hiding this comment

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

What about using avocado.utils.cpu.online_count()? or avocado.utils.cpu.total_count()?

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.

2 participants