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

[ci] Add more TVLA tests #287

Open
nasahlpa opened this issue Jan 12, 2024 · 1 comment
Open

[ci] Add more TVLA tests #287

nasahlpa opened this issue Jan 12, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request Priority:P1

Comments

@nasahlpa
Copy link
Member

Currently, CI only runs a basic AES test for TVLA. We should extend CI to also test other features.
However, as I am not aware of which features are actually used and should be supported, could somebody (@vogelpi @vrozic @johannheyszl) please help me creating a list of testvectors we would like to have in CI? For example, as the --tttest-step-file feature (mentioned in #284 ) was not mentioned anywhere (documentation or getting started), I was not aware that this is something we would like to have.

I am happy to add all necessary TVLA features to CI and debug them if necessary.

@vogelpi
Copy link
Collaborator

vogelpi commented Jan 12, 2024

Thanks for opening the issue @nasahlpa . What I would like to cover are the following features:

  • Select 2 bytes in 2 rounds, e.g. Bytes 0 and 1 in Rounds 0 and 1. This will dramatically reduce the memory footprint and computation time. Note, we need a small change in the argument passing. Shouldn't be tricky though. Currently if selecting bytes and rounds, individual bytes and rounds are supported only. However this is not sufficient as we've just seen.
  • Compute and memory could further be reduced by choosing fewer samples (not to be confused with fewer traces please).
  • Plotting figures after computation.
  • Recomputing the figures after loading ttest-step-file (this is currently broken).
  • For now let's use the CW database for this. It's fine to switch once ceca.py is ported over to the new format.

I can work on this as well when I have some spare cycles.

vogelpi added a commit to vogelpi/ot-sca that referenced this issue Jan 12, 2024
vogelpi added a commit to vogelpi/ot-sca that referenced this issue Jan 16, 2024
vogelpi added a commit to vogelpi/ot-sca that referenced this issue Jan 16, 2024
vogelpi added a commit that referenced this issue Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority:P1
Projects
None yet
Development

No branches or pull requests

2 participants