Skip to content

PPC64 Assembly: ELF V1 headers added. Little-endian asm added.#10871

Merged
dgarske merged 1 commit into
wolfSSL:masterfrom
SparkiDev:ppc64_le_and_elf1_fix
Jul 10, 2026
Merged

PPC64 Assembly: ELF V1 headers added. Little-endian asm added.#10871
dgarske merged 1 commit into
wolfSSL:masterfrom
SparkiDev:ppc64_le_and_elf1_fix

Conversation

@SparkiDev

Copy link
Copy Markdown
Contributor

Description

ELF V1 headers are need for cross-compiles.
PPC64 is big or little endian so assembly code modified to be able to be built for both. Fix wiring of vector AES implementations in aes.c.

Testing

Tested big-/little-endian PPC64 with assembly code.

@SparkiDev SparkiDev self-assigned this Jul 9, 2026
@SparkiDev

Copy link
Copy Markdown
Contributor Author

Code generated by PR:
https://github.com/wolfSSL/scripts/pull/619

@SparkiDev

Copy link
Copy Markdown
Contributor Author

Jenkins: retest this please

@SparkiDev SparkiDev force-pushed the ppc64_le_and_elf1_fix branch from 810046d to 71ffa4e Compare July 9, 2026 11:50
@SparkiDev

SparkiDev commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Jenkins: retest this please

FIPS-ready (aborted) x2
generic failed (lost connection)

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #10871

Scan targets checked: wolfcrypt-bugs, wolfcrypt-port-bugs, wolfcrypt-rs-bugs, wolfcrypt-src, wolfssl-bugs, wolfssl-src

Findings: 1
1 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread wolfcrypt/src/aes.c Outdated
ELF V1 headers are need for cross-compiles.
PPC64 is big or little endian so assembly code modified to be able to be built for both.
Fix wiring of vector AES implementations in aes.c.
@SparkiDev SparkiDev force-pushed the ppc64_le_and_elf1_fix branch from 71ffa4e to 3f3ebca Compare July 10, 2026 09:52
Comment thread wolfcrypt/src/aes.c Outdated

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #10871

Scan targets checked: wolfcrypt-bugs, wolfcrypt-port-bugs, wolfcrypt-rs-bugs, wolfcrypt-src, wolfssl-bugs, wolfssl-src

No new issues found in the changed files. ✅

@SparkiDev

Copy link
Copy Markdown
Contributor Author

Jenkins: retest this please

@dgarske dgarske left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Algorithm C-only PR-asm Speedup
AES-128-CBC-enc 33.0 81.9 2.49x
AES-256-CBC-enc 24.4 59.3 2.43x
AES-128-CTR 33.1 80.7 2.44x
AES-128-GCM-enc 11.7 54.4 4.66x
AES-256-GCM-enc 10.3 43.2 4.18x
GMAC Table 18.2 171.6 9.43x
AES-XTS-enc 24.9 74.3 2.98x
SHA-256 55.2 67.1 1.21x
SHA-384 / SHA-512 30.2 103.6 3.44x
SHA3-256 28.9 69.0 2.38x
HMAC-SHA512 29.7 101.7 3.42x

@dgarske dgarske merged commit 5d0da8e into wolfSSL:master Jul 10, 2026
349 of 351 checks passed
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.

4 participants