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

add module testing #520

Merged
merged 1 commit into from
Jul 8, 2024
Merged

add module testing #520

merged 1 commit into from
Jul 8, 2024

Conversation

roxell
Copy link
Collaborator

@roxell roxell commented Jul 8, 2024

Test to load/unload modules multiple times to see if something bad happens, which it shouldn't.

The majority of drivers registers themselves to an API that calls back into the driver when the hardware side (or network protocol, file system, etc) is getting detected or requested.

Loading and unloading modules that the DUT don't have hardware for is likely to find more of the obvious bugs.

scope:
- functional
devices:
- juno
Copy link
Member

Choose a reason for hiding this comment

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

Maybe rockpi instead?

format: Lava-Test Test Definition 1.0
name: module-tests
description: |
Load and unload kernle modules.
Copy link
Member

Choose a reason for hiding this comment

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

typo

@roxell roxell force-pushed the modules branch 4 times, most recently from 119a2a9 to 7a64f27 Compare July 8, 2024 09:16
Test to load/unload modules multiple times to see if something bad
happens, which it shouldn't.

The majority of drivers registers themselves to an API that calls back
into the driver when the hardware side (or network protocol, file
system, etc) is getting detected or requested.

Loading and unloading modules that the DUT don't have hardware for is
likely to find more of the obvious bugs.

Signed-off-by: Anders Roxell <[email protected]>
@nareshkamboju
Copy link
Collaborator

@roxell Thanks for addressing all the review comments.

@nareshkamboju nareshkamboju merged commit e3f51b8 into Linaro:master Jul 8, 2024
3 checks passed
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.

3 participants