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 multithread option , add option to only run stuck_address test #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wsmlby
Copy link

@wsmlby wsmlby commented Aug 2, 2023

No description provided.

@lucaceresoli
Copy link
Contributor

About "add option to only run stuck_address test", I think adding an option for a specific test is not a good idea. Somebody would like to run only Random Value + Bit Flip + 18-bit Writes we cannot realistically have a configuration flag for every possible combination. Even one flag per each specific test would bee too much.

Rather we should have a single way to select the desired set of tests for each execution, for example:

# Enable only selected tests
memtester -e stuck-addr,bit-flip,16bit-writes 100M 5
# Enable all tests except selected ones
memtester -d stuck-addr,bit-flip,16bit-writes 100M 5

Does it look reasonable?

@jnavila
Copy link
Owner

jnavila commented Aug 3, 2023

The selection of test can already be performed via the environment variable MEMTESTER_TEST_MASK. This feature may not be clear in the manpage, but have a look at the source code to get it correctly.

@jnavila
Copy link
Owner

jnavila commented Aug 3, 2023

Also remember that I do not intend to maintain memtester, because that would be a fork. If you are interested in helping on the development, please send your patches to Charles Cazabon, who is the original author of this tool.

see https://pyropus.ca./software/memtester/ for more details.

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