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

Implement '_mm_aesenc_si128' support. #6

Merged
merged 3 commits into from Sep 11, 2019
Merged

Conversation

mjmacleod
Copy link
Contributor

This pull request implements _mm_aesenc_si128 using the armv8 crypto extensions (where available) and a neon based vectorised aes round when crypto extensions are not available.

The vectorised neon version is based on an implementation by @ardbiesheuvel (which is used in the kernel for aegis and also talked about in several articles http://www.workofard.com/2017/01/accelerated-aes-for-the-arm64-linux-kernel/ http://www.workofard.com/2017/07/ghash-for-low-end-cores/

I've reproduced it with his permission and made slight modifications to make it fit in with sse2neon, along with implementing a basic test. I've tagged @ardbiesheuvel so that he can confirm that it is okay for it to enter this project under MIT license.

@ardbiesheuvel
Copy link

ardbiesheuvel commented Sep 10, 2019

Acked-by: Ard Biesheuvel <ard.biesheuvel@arm.com>

mjmacleod added a commit to muntorg/munt-official that referenced this pull request Sep 10, 2019
…on't have crypto extensions.

Based with permission on implementation by Ard Biesheuvel.
See pull request DLTcollab/sse2neon#6 for more information.
mjmacleod added a commit to muntorg/munt-official that referenced this pull request Sep 10, 2019
…on't have crypto extensions.

Based with permission on implementation by Ard Biesheuvel.
See pull request DLTcollab/sse2neon#6 for more information.
@jserv
Copy link
Member

jserv commented Sep 10, 2019

@mjmacleod, Thanks for contribution! Can you please run clang-format in order to enforce consistent style?

@jserv jserv merged commit 0351741 into DLTcollab:master Sep 11, 2019
@jserv
Copy link
Member

jserv commented Sep 11, 2019

Thank @mjmacleod and @ardbiesheuvel for the great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants