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

AES GCM stream: arm asm fix when --enable-opensslextra #6589

Merged

Conversation

SparkiDev
Copy link
Contributor

Description

aadLen is now in gcm field of Aes.

Testing

./configure '--disable-shared' 'LDFLAGS=--static' '--host=aarch64' 'CC=aarch64-linux-gnu-gcc' '--enable-opensslextra' '--enable-armasm' '--enable-aesgcm-stream' '--enable-cryptonly'

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@SparkiDev SparkiDev force-pushed the aarch64_aes_opensslextra_fix branch from 5e92ccc to 360b61a Compare July 10, 2023 00:13
Copy link
Contributor

@douzzer douzzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[cross-aarch64-all-armasm-unittest-sanitizer] [1 of 1] [360b61aff2]
    using aarch64-unknown-linux-gnu-gcc 12.3.1
    autogen.sh 360b61aff2...   real 0m15.743s  user 0m13.928s  sys 0m0.932s
    configure...   real 0m16.282s  user 0m9.257s  sys 0m8.430s
    build...   real 2m55.615s  user 10m58.825s  sys 0m12.451s
    unit.test...   real 3m55.247s  user 3m53.084s  sys 0m0.948s
    cross-aarch64-all-armasm-unittest-sanitizer OK

Copy link
Contributor

@JacobBarthelmeh JacobBarthelmeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolves compile issues with --enable-afalg=xilinx, thanks Sean

@JacobBarthelmeh JacobBarthelmeh merged commit 9d18648 into wolfSSL:master Jul 10, 2023
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.

3 participants