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

[DI] Move scripts not used by the CI #53

Open
mpg opened this issue Oct 3, 2024 · 0 comments
Open

[DI] Move scripts not used by the CI #53

mpg opened this issue Oct 3, 2024 · 0 comments
Assignees
Labels
size-m Estimated task size: medium (~1w)

Comments

@mpg
Copy link
Contributor

mpg commented Oct 3, 2024

[Moved and re-organized from #12 and #16.]

Note: the scope (as a first step) is now only to move to the framework, not to adapt to tf-psa-crypto. Some comments and size estimates are still from the time where the scope included adapting to tf-psa-crypto.

The lists below are temporary and might still evolve.

Table legend:
I: Currently identical between 3.6 and development?
S: Work size for the move and potentially changes needed for TF-PSA-Crypto

Moving to the framework - used by tf-psa-crypto

name I S Issue/PR Comment
ecc-heap.sh y S depends on benchmark.c, adapt to tf_psa_crypto_config.h
gen_ctr_drbg.pl y XS
gen_gcm_decrypt.pl y -
gen_gcm_encrypt.pl y -
gen_pkcs1_v21_sign_verify.pl y -
generate-afl-tests.sh y -
prepare_release.sh y ? do we keep it?
test_config_script.py y XS changes for crypto as part of work in crypto_config.py
tmp_ignore_makefiles.sh y S do we keep it? if yes, adapt to TF-PSA-Crypto tree
bump_version.sh n S adapt to TF-PSA-Crypto
code_size_compare.py n M adapt to mbedtls_config.h split, adapt to tf_psa_crypto_config.h

Moving to the framework - not used by tf-psa-crypto

name I S Issue/PR Comment
audit-validity-dates.py y XS
footprint.sh y do we keep it? broken, redundant with code_size_compare.py?
generate_server9_bad_saltlen.py y -
set_psa_test_dependencies.py y -

Not moving to the framework

name I S Issue/PR Comment
ecp_comb_table.py y - Utility to add a new curve in ecp_curves.c, tied to ecp_curves.c code / Manuel has plan to complete rewrite this, see Mbed-TLS/mbedtls#7623
memory.sh + massif_max.pl n do we keep it? if yes, adapt to mbedtls_config.h split
@mpg mpg added the size-m Estimated task size: medium (~1w) label Oct 3, 2024
@mpg mpg self-assigned this Oct 3, 2024
@mpg mpg moved this to Framework 3/3 (remaining scripts) in Mbed TLS Epics Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-m Estimated task size: medium (~1w)
Projects
Status: Framework 3/3 (remaining scripts)
Development

No branches or pull requests

1 participant