Skip to content

20250916-linuxkm-module-update-fips-hash#9206

Merged
philljj merged 1 commit intowolfSSL:masterfrom
douzzer:20250916-linuxkm-module-update-fips-hash
Sep 17, 2025
Merged

20250916-linuxkm-module-update-fips-hash#9206
philljj merged 1 commit intowolfSSL:masterfrom
douzzer:20250916-linuxkm-module-update-fips-hash

Conversation

@douzzer
Copy link
Contributor

@douzzer douzzer commented Sep 16, 2025

linuxkm/Makefile:

  • add module-update-fips-hash rule, for in-place FIPS hash update without rebuild;
  • improve PIE sequence in module build rule to double-check stability of the relocation table after final rebuild;

Makefile.am: add a module-update-fips-hash passthrough target.

tested with

wolfssl-multi-test.sh ...
check-source-text-fips-dev
linuxkm-fips-dev-static-hash-intelasm-LKCAPI-insmod
linuxkm-fips-dev-module-update-fips-hash-intelasm-LKCAPI-insmod
linuxkm-fips-v5-vanilla-insmod-wolfguard-cust3

after adding support for the module-update-fips-hash method.

Note the new logic in this PR is purely additive. The old methods of updating the FIPS hash (editing fips_test.c or passing in WOLFCRYPT_FIPS_CORE_HASH_VALUE for rebuild) still work as before, and are still the default method in testing, preserving all existing coverage of module build reproducibility.

The new method simply overcomes non-reproducibility for those that need it.

* add module-update-fips-hash rule, for in-place FIPS hash update without rebuild;
* improve PIE sequence in module build rule to double-check stability of the relocation table after final rebuild;

Makefile.am: add a module-update-fips-hash passthrough target.
@douzzer
Copy link
Contributor Author

douzzer commented Sep 16, 2025

retest this please

@philljj philljj merged commit 5cb2103 into wolfSSL:master Sep 17, 2025
324 of 325 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

Comments