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

[pentest] Ibex SCA tests #22133

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Conversation

nasahlpa
Copy link
Member

This commit adds the following penetration tests:

  • ibex.sca.register_file_read
  • ibex.sca.register_file_write
  • ibex.sca.tl_read
  • ibex.sca.tl_write

This code is used to check whether data written/read to/from the tegister file or transmitted over TL-UL leaks.

This commit adds the following penetration tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

This code is used to check whether data written/read to/from the
tegister file or transmitted over TL-UL leaks.

Signed-off-by: Pascal Nasahl <[email protected]>
@nasahlpa nasahlpa requested a review from a team as a code owner March 20, 2024 12:54
@nasahlpa nasahlpa requested review from engdoreis, vogelpi and johannheyszl and removed request for a team March 20, 2024 12:54
nasahlpa added a commit to nasahlpa/ot-sca that referenced this pull request Mar 20, 2024
This commit adds the following SCA Ibex tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

The device code is located in lowRISC/opentitan#22133 and the binary
was created from this PR.

Signed-off-by: Pascal Nasahl <[email protected]>
nasahlpa added a commit to nasahlpa/ot-sca that referenced this pull request Mar 20, 2024
This commit adds the following SCA Ibex tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

The device code is located in lowRISC/opentitan#22133 and the binary
was created from this PR.

Signed-off-by: Pascal Nasahl <[email protected]>
@nasahlpa nasahlpa added the CherryPick:master This PR should be cherry-picked to master label Mar 20, 2024
nasahlpa added a commit to nasahlpa/ot-sca that referenced this pull request Mar 20, 2024
This commit adds the following SCA Ibex tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

The device code is located in lowRISC/opentitan#22133 and the binary
was created from this PR.

Signed-off-by: Pascal Nasahl <[email protected]>
Copy link
Contributor

@johannheyszl johannheyszl left a comment

Choose a reason for hiding this comment

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

thanks @nasahlpa

@moidx moidx merged commit f411684 into lowRISC:earlgrey_es_sival Mar 21, 2024
30 checks passed
Copy link

Successfully created backport PR for master:

nasahlpa added a commit to nasahlpa/ot-sca that referenced this pull request Mar 21, 2024
This commit adds the following SCA Ibex tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

The device code is located in lowRISC/opentitan#22133 and the binary
was created from lowRISC/opentitan@55091e7 with:
./bazelisk.sh build //sw/device/tests/crypto/cryptotest/firmware:firmware_fpga_cw310_test_rom

Signed-off-by: Pascal Nasahl <[email protected]>
nasahlpa added a commit to lowRISC/ot-sca that referenced this pull request Mar 21, 2024
This commit adds the following SCA Ibex tests:
- ibex.sca.register_file_read
- ibex.sca.register_file_write
- ibex.sca.tl_read
- ibex.sca.tl_write

The device code is located in lowRISC/opentitan#22133 and the binary
was created from lowRISC/opentitan@55091e7 with:
./bazelisk.sh build //sw/device/tests/crypto/cryptotest/firmware:firmware_fpga_cw310_test_rom

Signed-off-by: Pascal Nasahl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPick:master This PR should be cherry-picked to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants