Skip to content

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

Open
SparkiDev wants to merge 1 commit into
wolfSSL:masterfrom
SparkiDev:ppc64_le_and_elf1_fix
Open

PPC64 Assembly: ELF V1 headers added. Little-endian asm added.#10871
SparkiDev wants to merge 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
* key-setup path before any AES_*_crypto use. */
static void Aes_SetCrypto(void)
{
if (aes_ppc64_use_crypto < 0) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed and for other platforms and SHA2 as well.

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
* key-setup path before any AES_*_crypto use. */
static void Aes_SetCrypto(void)
{
if (aes_ppc64_use_crypto < 0) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed and for other platforms and SHA2 as well.

@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

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