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

Performance tests #840

Closed
wants to merge 38 commits into from
Closed

Conversation

mikaelsky
Copy link
Collaborator

Added a benchmark test bench that is being used for the performance tests. The benchmark testbench has almost everything in the core disabled. It has 128kb of IRAM to ensure we have enough space for the test suites and U mode disabled.

Added a test finish process allowing the main.c to signal the testbench to stop running.

Updated common.mk to support using something other than the simple testbench using makefile variables.

Added a run_performance.sh script that runs through all the tests with the _all setting. This will basically plot all the performance numbers measured.

Fixed a few bugs in the I performance test causing test failures.

TODO: Fix the mret test as its still failing. Added support for C and B extensions.

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Added Benchmark testbench

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Added an option to the makefile to overwrite the path to the sim folder. This allows us to point to a different test environment than just simple

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Added the path to the benchmark testbench for benchmark runs.

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Added support for the benchmark testbench

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Added support for the benchmark testbench

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Increase memory size to 128kb to be able to fit full benchmark test code

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
disabled the mret test from _all until it gets debugged.
Changed the store test to use SP+4 as the target.

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
removed U mode to allow mret testing

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
fixed silent mode spelling

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
fixed bug in blt backwards

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Fixed bltu backwards not working

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
@mikaelsky
Copy link
Collaborator Author

Note: Software pull request fails as its going to the performance_test area and trying to build, which right now doesn't work.

Added default makefile that will call into the I directory to make checkin and regressions pass

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Updated makefile to support clean_all

Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
Signed-off-by: Mikael Mortensen <119539842+mikaelsky@users.noreply.github.com>
@mikaelsky mikaelsky closed this Mar 10, 2024
@stnolting
Copy link
Owner

FTR

Update of this PR in #844

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.

None yet

2 participants