Skip to content

[WIP] Test with various crypto engines#16749

Closed
bernd-edlinger wants to merge 1 commit intoopenssl:masterfrom
bernd-edlinger:enable_crypto_engines_test
Closed

[WIP] Test with various crypto engines#16749
bernd-edlinger wants to merge 1 commit intoopenssl:masterfrom
bernd-edlinger:enable_crypto_engines_test

Conversation

@bernd-edlinger
Copy link
Member

Enable these crypto engines always when available:

  • afalg
  • devcrypto
  • padlock
  • dasync

Where dasync will have lowest precedence.

@bernd-edlinger bernd-edlinger added the branch: master Applies to master branch label Oct 5, 2021
@bernd-edlinger
Copy link
Member Author

Hmm, also with #16734 applied there are tons of test failures.

@beldmit
Copy link
Member

beldmit commented Oct 6, 2021

Are they related to #16738?

@bernd-edlinger
Copy link
Member Author

yes, but there are obviously many more "features" in the new design.

@bernd-edlinger bernd-edlinger force-pushed the enable_crypto_engines_test branch from 0cef3b9 to 6d446ea Compare October 6, 2021 17:33
@t8m t8m added the triaged: feature The issue/pr requests/adds a feature label Oct 21, 2021
@t8m t8m closed this Oct 21, 2021
@t8m t8m reopened this Oct 21, 2021
@bernd-edlinger bernd-edlinger force-pushed the enable_crypto_engines_test branch 3 times, most recently from 367c970 to 50ac5c6 Compare November 23, 2021 18:33
@bernd-edlinger bernd-edlinger force-pushed the enable_crypto_engines_test branch from 50ac5c6 to ef82b86 Compare December 5, 2021 13:03
@t8m t8m closed this Jul 3, 2024
@t8m t8m reopened this Jul 3, 2024
@t8m
Copy link
Member

t8m commented Jul 3, 2024

Is this still relevant?

Marking as inactive to be closed at the end of 3.4 dev barring further input.

@t8m t8m added the inactive This label should not be applied to open issues anymore. label Jul 3, 2024
@bernd-edlinger
Copy link
Member Author

The point is that, the engines don't work, therefore are all those tests are failed.
While with the equivalent PR for 1.1.1 (see #16750) all tests were passed.

Enable these crypto engines always when available:

- padlock
- devcrypto
- afalg
- dasync

Where dasync will have lowest precedence.
@t8m
Copy link
Member

t8m commented Aug 28, 2025

Engines are to be dropped in 4.0. Closing.

@t8m t8m closed this Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch: master Applies to master branch inactive This label should not be applied to open issues anymore. triaged: feature The issue/pr requests/adds a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants