Skip to content

Adding the check for module to be the same as driver to be tested#3095

Open
PavamanSubramaniyam wants to merge 1 commit intoavocado-framework-tests:masterfrom
PavamanSubramaniyam:driver_check_with_module
Open

Adding the check for module to be the same as driver to be tested#3095
PavamanSubramaniyam wants to merge 1 commit intoavocado-framework-tests:masterfrom
PavamanSubramaniyam:driver_check_with_module

Conversation

@PavamanSubramaniyam
Copy link
Contributor

@PavamanSubramaniyam PavamanSubramaniyam commented Feb 25, 2026

Introducing the check for making sure if the network adapter is physically present in the machine Thereby if the driver of the network interface is the same as the kernel module to be tested

vaishnavibhat
vaishnavibhat suggested changes Feb 25, 2026
View reviewed changes
Copy link
Contributor

@vaishnavibhat vaishnavibhat left a comment

Choose a reason for hiding this comment

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

Hi Pavaman,
Do you think this is a good candidate to be moved to avocado utils ?

Thank you.

@Naresh-ibm
Copy link
Collaborator

Naresh-ibm commented Feb 25, 2026

@PavamanSubramaniyam please check the pylint issue and re-push the code.
also add the run logs

@Naresh-ibm Naresh-ibm self-assigned this Feb 25, 2026
@PavamanSubramaniyam PavamanSubramaniyam force-pushed the driver_check_with_module branch 2 times, most recently from 5aea6ec to beea27e Compare March 4, 2026 05:00
@PavamanSubramaniyam
Adding the check for module to be the same as driver to be tested
4632f8a 
Introducing the check for making sure if the network adapter is physically present in the machine
Thereby if the driver of the network interface is the same as the kernel module to be tested

Signed-off-by: Pavaman Subramaniyam <pavsubra@linux.vnet.ibm.com>
@PavamanSubramaniyam PavamanSubramaniyam force-pushed the driver_check_with_module branch from beea27e to 4632f8a Compare March 4, 2026 05:14
@PavamanSubramaniyam
Copy link
Contributor Author

PavamanSubramaniyam commented Mar 7, 2026 •
edited
Loading

avocado run --max-parallel-tasks=1 driver_parameter.py -m driver_parameter.py.data/driver_parameter_bnx2x.yaml

JOB ID : cdaa55950797b15af8a9e1b604d4fa27c09df400
JOB LOG : /root/avocado-fvt-wrapper/results/job-2026-03-07T13.30-cdaa559/job.log
(01/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_0-7e15: STARTED
(01/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_0-7e15: PASS (35.58 s)
(02/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_1-84dd: STARTED
(02/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_1-84dd: PASS (35.65 s)
(03/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_8-96c5: STARTED
(03/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_8-96c5: PASS (34.85 s)
(04/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_64-27c9: STARTED
(04/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_64-27c9: PASS (37.02 s)
(05/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_1024-617a: STARTED
(05/16) driver_parameter.py:Moduleparameter.test;run-Test-num_queue-value-num_que_1024-617a: PASS (36.56 s)
(06/16) driver_parameter.py:Moduleparameter.test;run-Test-disable_tpa-value-dis_tpa_1-ba47: STARTED
(06/16) driver_parameter.py:Moduleparameter.test;run-Test-disable_tpa-value-dis_tpa_1-ba47: PASS (35.84 s)
(07/16) driver_parameter.py:Moduleparameter.test;run-Test-disable_tpa-value-dis_tpa_0-ab30: STARTED
(07/16) driver_parameter.py:Moduleparameter.test;run-Test-disable_tpa-value-dis_tpa_0-ab30: PASS (34.55 s)
(08/16) driver_parameter.py:Moduleparameter.test;run-Test-int_mode-value-int_1-2be4: STARTED
(08/16) driver_parameter.py:Moduleparameter.test;run-Test-int_mode-value-int_1-2be4: PASS (34.15 s)
(09/16) driver_parameter.py:Moduleparameter.test;run-Test-int_mode-value-int_2-8176: STARTED
(09/16) driver_parameter.py:Moduleparameter.test;run-Test-int_mode-value-int_2-8176: PASS (35.41 s)
(10/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_0-8a6f: STARTED
(10/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_0-8a6f: PASS (35.68 s)
(11/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_1-ca70: STARTED
(11/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_1-ca70: PASS (35.64 s)
(12/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_2-ac2f: STARTED
(12/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_2-ac2f: PASS (35.79 s)
(13/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_3-f64c: STARTED
(13/16) driver_parameter.py:Moduleparameter.test;run-Test-mrrs-value-mrrs_3-f64c: PASS (35.81 s)
(14/16) driver_parameter.py:Moduleparameter.test;run-Test-debug-value-debug_0-1181: STARTED
(14/16) driver_parameter.py:Moduleparameter.test;run-Test-debug-value-debug_0-1181: PASS (35.66 s)
(15/16) driver_parameter.py:Moduleparameter.test;run-Test-debug-value-debug_1-3b18: STARTED
(15/16) driver_parameter.py:Moduleparameter.test;run-Test-debug-value-debug_1-3b18: PASS (35.99 s)
(16/16) driver_parameter.py:Moduleparameter.test;run-Test-dropless_fc-value-dropless_fc_1-6852: STARTED
(16/16) driver_parameter.py:Moduleparameter.test;run-Test-dropless_fc-value-dropless_fc_1-6852: PASS (36.03 s)
RESULTS : PASS 16 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML : /root/avocado-fvt-wrapper/results/job-2026-03-07T13.30-cdaa559/results.html
JOB TIME : 637.70 s

@PavamanSubramaniyam
Copy link
Contributor Author

PavamanSubramaniyam commented Mar 24, 2026

I have added the run logs and also corrected the indentation issues and repushed the code.

@Naresh-ibm
Copy link
Collaborator

Naresh-ibm commented Mar 24, 2026

@PavamanSubramaniyam it would be great if you can add the runs logs based on cfg runs, as this is more helpful in cfg based runs.
what you do following things for quick runs

  1. create a temp cfg with 4-5 different driver entries
  2. pass only one driver from the input file and run the cfg in cli mode
  3. it should run only for the driver passed and other driver should cancel without running the tests.

So please attach this logs, so that we are good to merge it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@vaishnavibhat vaishnavibhat vaishnavibhat requested changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@Naresh-ibm Naresh-ibm

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@PavamanSubramaniyam @Naresh-ibm @vaishnavibhat