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.

Sign up for GitHub

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

Jump to bottom

Drop VIA Padlock support #5903

Closed
gilles-peskine-arm opened this issue Jun 9, 2022 · 1 comment · Fixed by #9258
Closed

Drop VIA Padlock support #5903

gilles-peskine-arm opened this issue Jun 9, 2022 · 1 comment · Fixed by #9258
Assignees
@tom-daubney-arm
Labels
api-break This issue/PR breaks the API and must wait for a new major version component-crypto Crypto primitives and low-level interfaces deprecation needs deprecation announcement size-xs Estimated task size: extra small (a few hours at most)

Comments

@gilles-peskine-arm
Copy link
Contributor

gilles-peskine-arm commented Jun 9, 2022 •
edited by tom-daubney-arm
Loading

Mbed TLS supports AES acceleration on 32-bit x86 chips with VIA Padlock (MBEDTLS_PADLOCK_C).

This processor feature has lost a lot of its popularity now that x86 chips are mostly 64-bit and have Intel's AES acceleration. The Arm maintainer team doesn't have the knowledge or hardware to maintain this feature. As a consequence, unless there is significant demonstrated interest, we intend to remove MBEDTLS_PADLOCK_C and library/padlock.c in Mbed TLS 4.0. It will still be possible to use VIA padlock via a PSA accelerator driver (which will not be part of Mbed TLS).
@gilles-peskine-arm gilles-peskine-arm added component-crypto Crypto primitives and low-level interfaces api-break This issue/PR breaks the API and must wait for a new major version labels Jun 9, 2022
@gilles-peskine-arm gilles-peskine-arm mentioned this issue Jun 9, 2022
Closed
@mpg mpg mentioned this issue Dec 15, 2022
Closed
@daverodgman daverodgman added deprecation needs deprecation announcement size-xs Estimated task size: extra small (a few hours at most) labels Aug 7, 2023
@ronald-cron-arm
Copy link
Contributor

Not supported in psa-crypto and not planned to be supported thus this should probably be a MUST rather than a SHOULD.

@gilles-peskine-arm gilles-peskine-arm moved this to Implementation in progress in Mbed TLS 4.0 planning Jun 25, 2024
@gilles-peskine-arm gilles-peskine-arm mentioned this issue Jun 25, 2024
Merged
4 tasks
@github-project-automation github-project-automation bot moved this from Implementation in progress to Done in Mbed TLS 4.0 planning Jun 26, 2024
@minosgalanakis minosgalanakis moved this to Configuration split in Mbed TLS Epics Aug 2, 2024
@github-project-automation github-project-automation bot moved this to Mbed TLS 4.0 SHOULD in Backlog for Mbed TLS Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tom-daubney-arm tom-daubney-arm

Labels
api-break This issue/PR breaks the API and must wait for a new major version component-crypto Crypto primitives and low-level interfaces deprecation needs deprecation announcement size-xs Estimated task size: extra small (a few hours at most)
Projects
Backlog for Mbed TLS
Status: Mbed TLS 4.0 SHOULD
Mbed TLS 4.0 planning
Status: Done
Mbed TLS Epics
Status: Configuration split
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Drop support for VIA Padlock tom-daubney-arm/mbedtls
4 participants
@daverodgman @gilles-peskine-arm @ronald-cron-arm @tom-daubney-arm