Skip to content

Otsca test #398

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Otsca test #398

wants to merge 1 commit into from

Conversation

siemen11
Copy link

Add scripts on how to chain commands for each test of the current testOS, add a Bazel framework which extends the opentitan repo's, add test scripts to test the testOS from the host scripts.

@siemen11
Copy link
Author

This can have lower priority to be merged.

What still needs to be done:

  • to see whether these scripts are compatible with FPGA testing (I don't have an FPGA available at the moment)
  • to make the tests and host scripts better via templates (meaning there is a lot of similarity between large parts of the code)
  • a nice to have is to further extend the bazel and test framework to include the other scripts in testing

Feel free to add more to-do's to improve this

@siemen11
Copy link
Author

FPGA part is done now

Copy link
Member

@nasahlpa nasahlpa left a comment

Choose a reason for hiding this comment

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

Thanks Siemen for working this, having a test framework for ot-sca <-> pentest framework does have a huge impact on the maintainability of the pentesting infrastructure :)

I've dropped a couple of comments. It seems that your changes broke the current SCA capture scripts (please see the CI). In order to merge this we would need to make sure that they still work.

@siemen11
Copy link
Author

Sadly I must admin defeat in that I couldn't get OpenTitan's lint targets work for this WORKSPACE so licenses and empty lines added manually for now

Copy link
Member

@nasahlpa nasahlpa left a comment

Choose a reason for hiding this comment

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

Thanks Siemen. I'd suggest a couple of points before we can merge this:

  • Can you please rebase the PR such that we see that CI passes?
  • Would it be possible to add a very brief documentation to test/penetrationtests, i.e., a brief how to run this?
  • Could you please squash some of the commits? E.g., creating a new communication folder and then moving it again at the end?
  • Would it be possible to move the default configs for the target/communication scripts outside of these functions?

@siemen11 siemen11 force-pushed the otsca_test branch 5 times, most recently from 68be907 to c87d1a8 Compare July 22, 2025 18:22
Make a bazel framework in ot-sca as extension of OpenTitan and add test
scripts (pytests) from the host side for the penetrationtests.

Signed-off-by: Siemen Dhooghe <[email protected]>
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