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

NetKVM: Add a case for checking the keyword in the driver logs #4145

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

Conversation

heywji
Copy link
Contributor

@heywji heywji commented Aug 28, 2024

Configure the driver with TX capacity of 64 (instead of default of 1024) and check in the driver logs that the driver limit the SG (scatter-gather) area " Limit m_SGTableCapacity by 64"

ID: 2402

Signed-off-by: wji [email protected]

@heywji heywji force-pushed the add_traceview_case branch 3 times, most recently from a380f4a to d31d08d Compare August 29, 2024 07:57
@heywji
Copy link
Contributor Author

heywji commented Aug 29, 2024

Hi @leidwang, Could you help review this patch when you are available?

Test result

 (1/1) Host_RHEL.m9.u5.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.netkvm_log_check.q35: STARTED
 (1/1) Host_RHEL.m9.u5.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.netkvm_log_check.q35: PASS (337.54 s)

Thanks in advance,
Wecom

qemu/tests/cfg/netkvm_log_check.cfg Outdated Show resolved Hide resolved
qemu/tests/netkvm_log_check.py Outdated Show resolved Hide resolved
qemu/tests/netkvm_log_check.py Outdated Show resolved Hide resolved
@heywji heywji force-pushed the add_traceview_case branch 3 times, most recently from 5d0aadd to fa867fc Compare September 5, 2024 06:57
@heywji
Copy link
Contributor Author

heywji commented Sep 5, 2024

Hi @leidwang, Could you help review this again I have resolve all the problems.

 (1/1) Host_RHEL.m9.u5.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.netkvm_log_check.q35: PASS (482.94 s)
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /root/avocado/job-results/job-2024-09-05T16.04-80ddf85/results.html
JOB TIME   : 485.19 s

qemu/tests/cfg/netkvm_log_check.cfg Outdated Show resolved Hide resolved
qemu/tests/netkvm_log_check.py Outdated Show resolved Hide resolved
qemu/tests/cfg/netkvm_log_check.cfg Outdated Show resolved Hide resolved
qemu/tests/netkvm_log_check.py Outdated Show resolved Hide resolved
@heywji heywji force-pushed the add_traceview_case branch 2 times, most recently from 2a0ec95 to 949c533 Compare September 23, 2024 06:12
@heywji
Copy link
Contributor Author

heywji commented Sep 23, 2024

Test Result:

 (1/1) Host_RHEL.m9.u5.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.netkvm_log_check.q35: PASS (918.48 s)
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /root/avocado/job-results/job-2024-09-23T14.19-44b4fd5/results.html
JOB TIME   : 921.11 s

Hi @leidwang, Could you help review this patch again when you are free?

Copy link
Contributor

@leidwang leidwang left a comment

Choose a reason for hiding this comment

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

LGTM

@heywji
Copy link
Contributor Author

heywji commented Oct 9, 2024

Hi @vivianQizhu, Could you help review this as well? Thanks : )

@heywji heywji force-pushed the add_traceview_case branch 4 times, most recently from c3787ee to f494847 Compare October 30, 2024 03:45
Configure the driver with TX capacity of 64 (instead of default of 1024)
and check in the driver logs that the driver limit the SG
(scatter-gather) area " Limit m_SGTableCapacity by 64"

Signed-off-by: wji <[email protected]>
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