-
Notifications
You must be signed in to change notification settings - Fork 182
netkvm: add netkvm_poll_mode test case #4353
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
base: master
Are you sure you want to change the base?
Conversation
Test result:
|
3a430f1
to
653d703
Compare
Poll mode is a must-have feature for Windows Server 2025 and Windows 11. Signed-off-by: Wenkang Ji <[email protected]>
653d703
to
9b32bdb
Compare
@leidwang Hi Leidong, please help review this poll mode patch. I have completed it now. |
|
||
:param test: QEMU test object | ||
:param params: Dictionary with the test parameters | ||
:param env: Dictionary with test environmen. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:param env: Dictionary with test environmen. | |
:param env: Dictionary with test environment. |
mapping_output = re.findall(keyword, result) | ||
if not mapping_output: | ||
test.error("Can't get %s from traceview", keyword) | ||
return mapping_output |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to return any value for run()
setup_rss_queues = '"*NumRssQueues" %s' | ||
check_rss_queues = "*NumRssQueues" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rss_queue_name = "*NumRssQueues"
Please update these lines as we discussed.
queues = ${smp} | ||
variants: | ||
- rss_16: | ||
rss_queues = 16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rss_queue_value = 1
restore_image_after_testing = yes | ||
# this case only tested on prewhql255 or higher version | ||
cdroms += " virtio" | ||
required_virtio_win_prewhql = [0.1.255, ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we also need add virtio-win version here
Poll mode is a must-have feature for Windows Server 2025 and Windows 11.
ID: 2267
Signed-off-by: Wenkang Ji [email protected]