Skip to content

Add tuning parameters to stressapttest to replace blank commandline in profile#673

Merged
AlexW-F merged 3 commits intomainfrom
users/alexwill/stressaptestcommandline
Apr 3, 2026
Merged

Add tuning parameters to stressapttest to replace blank commandline in profile#673
AlexW-F merged 3 commits intomainfrom
users/alexwill/stressaptestcommandline

Conversation

@AlexW-F
Copy link
Copy Markdown
Contributor

@AlexW-F AlexW-F commented Apr 2, 2026

Summary

Replaces the unused free-form CommandLine parameter in the StressAppTest executor with explicit, typed parameters for memory and thread tuning. The default workload behavior is unchanged.

Changes

  • Removed the CommandLine property (was always set to "" in the profile)
  • Added three new optional parameters:
    • MemoryInMB — RAM to test in MB (-M flag); defaults to stressapptest auto-detect (~94% of system memory)
    • ThreadCount — number of memory copy threads (-m flag); defaults to stressapptest auto-detect (2 per CPU)
    • CpuStressThreadCount — number of CPU stress threads (-C flag); defaults to none
  • Added input validation for the new parameters
  • Updated the PERF-MEM-STRESSAPPTEST.json profile to expose the new parameters
  • Updated website documentation with parameter descriptions and usage examples

Default behavior

When no new parameters are specified, the generated command line is identical to before:

./stressapptest -s <duration> -l <logfile>

Testing

  • Added unit tests for validation of invalid parameter values (zero/negative)
  • Added unit tests verifying each new flag appears in the command line when specified
  • Added unit test confirming optional flags are omitted when parameters are not set

@AlexW-F AlexW-F enabled auto-merge (squash) April 3, 2026 00:01
@AlexW-F AlexW-F merged commit 3ffe2e9 into main Apr 3, 2026
5 checks passed
@AlexW-F AlexW-F deleted the users/alexwill/stressaptestcommandline branch April 3, 2026 00:13
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