Skip to content

Add support for PowerPC 32-bit SHA256 assembly speedups#576

Merged
danielinux merged 5 commits intowolfSSL:masterfrom
dgarske:ppc32_asm
Jun 24, 2025
Merged

Add support for PowerPC 32-bit SHA256 assembly speedups#576
danielinux merged 5 commits intowolfSSL:masterfrom
dgarske:ppc32_asm

Conversation

@dgarske
Copy link
Copy Markdown
Contributor

@dgarske dgarske commented Jun 20, 2025

Add support for PowerPC 32-bit SHA256 assembly speedups.

  • Minor fix for ELF loading to set entry point before copying sections (resolves issue with loading on top of current location).
  • Rename elf_load_image_mmu argument entry to pentry to avoid confusion with GET_H64 macro's use of header member "entry".
  • Added some useful PowerPC timer code and examples in hal/nxp_p1021.c

…r ELF loading to set entry point before copying sections (resolves issue with loading on top of current location). Rename elf_load_image_mmu argument `entry` to `pentry` to avoid confusion with GET_H64 macro's use of header member "entry".
@dgarske dgarske requested a review from danielinux June 20, 2025 21:46
@dgarske dgarske self-assigned this Jun 20, 2025
… cert gen, crypto callback missing max digest size).
rizlik
rizlik previously approved these changes Jun 24, 2025
Copy link
Copy Markdown
Contributor

@rizlik rizlik left a comment

Choose a reason for hiding this comment

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

It looks good, my only comment is that moving the parsing of the entry point doesn't solve the general problem of memory overwrite when loading the ELF.

In other archs application and wolfBoot agreed on separating completely the areas where can ELF can be loaded from the memory used by wolfBoot. This way, as the ELF is verified, we can put the responsibility of not using any memory area used by wolfBoot on the ELF.

@rizlik rizlik assigned dgarske and unassigned rizlik Jun 24, 2025
@dgarske
Copy link
Copy Markdown
Contributor Author

dgarske commented Jun 24, 2025

The wolfHSM tests are having issues (hanging). @bigbrett is looking into and will followup with a separate PR.

@dgarske dgarske removed their assignment Jun 24, 2025
@danielinux danielinux merged commit 499c081 into wolfSSL:master Jun 24, 2025
258 of 261 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.

3 participants