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

Software Testing Infrastructure #152

Merged
merged 10 commits into from
Aug 22, 2024
Merged

Software Testing Infrastructure #152

merged 10 commits into from
Aug 22, 2024

Conversation

HU90m
Copy link
Member

@HU90m HU90m commented Aug 15, 2024

This moves the testing infrastructure from sonata-software (lowRISC/sonata-software#26) to this repository. The tests have been adapted to run in the minimal bare metal environment.

What was in the tests directory is now in a directory called 'checks'. The distinction is talked about in sw/cheri/README.md.

@HU90m HU90m requested review from GregAC, marnovandermaas and nbdd0121 and removed request for GregAC and marnovandermaas August 15, 2024 07:00
@HU90m HU90m mentioned this pull request Aug 19, 2024
Copy link
Contributor

@engdoreis engdoreis left a comment

Choose a reason for hiding this comment

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

LGTM. But it would be nice if Gary or Harry could review the Nix code.

util/test_runner.py Outdated Show resolved Hide resolved
util/test_runner.py Outdated Show resolved Hide resolved
cspecialr ca0, mtdc

// No bounds on stack, grows down from the end
li sp, 0x0010E000
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this constant comes from the linker script?

Copy link
Member Author

Choose a reason for hiding this comment

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

That's a nice idea

sw/cheri/tests/xmake.lua Outdated Show resolved Hide resolved
The cheriot test suite has stopped passing on the sonata system.
I'm sorry this commit exists.
@HU90m HU90m merged commit 4171621 into lowRISC:main Aug 22, 2024
2 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.

2 participants