Skip to content

Refresh BIOS artifact for checked BLS pairing result - #643

Merged
huangminghuang merged 1 commit into
masterfrom
fix/wire-339-bios-cdt-artifact
Sep 25, 2026
Merged

huangminghuang merged 1 commit into
masterfrom
fix/wire-339-bios-cdt-artifact

Conversation

@huangminghuang

Copy link
Copy Markdown
Contributor

Summary

  • Rebuild sysio.bios.wasm with the CDT BLS pairing-result check, so finalizer proof-of-possession verification rejects failed pairing operations before comparing output.
  • Refresh only the Deep Mind log and four consensus blockchain reference files whose bootstrap identities change with the new BIOS bytes.

Why

CDT #121 fixes the inline BLS verification helper, but existing deployed artifacts retain the old behavior until rebuilt. This updates BIOS without changing its source or ABI. Historical snapshot compatibility fixtures remain unchanged.

Validation

  • Built BIOS with CDT a00bd7187322df68be6616adbe4c6049b93d97bf; inspected the WASM failure-status branch and unchanged import signatures.
  • BIOS finalizer suite: 13 cases passed under sys-vm-jit.
  • CDT crypto extension tests: five passed, including the pairing failure-return regression.
  • Regenerated Deep Mind and consensus references with the native save modes, then passed normal comparisons under sys-vm-jit.
  • Historical snapshot compatibility: three cases passed.
  • Fresh isolated ON/ON build on master 0b08fb13; all tracked contract artifacts matched their build outputs.
  • Current-base focused core selection: seven cases / 166 assertions passed, including the two upstream ABI enum tests.

Coordinated platform E2E is explicitly excluded for this standalone artifact refresh; no no-expiry changes are included.

Change-Id: If6ada6e43aeda3b137185bbfe47736b1637c4b8d
@huangminghuang
huangminghuang marked this pull request as ready for review September 25, 2026 17:41
@huangminghuang
huangminghuang requested a review from a team September 25, 2026 17:41

@heifner heifner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this was necessary, but doesn't hurt.

@huangminghuang
huangminghuang merged commit 0dd064c into master Sep 25, 2026
13 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.

2 participants