Skip to content

fix(identity): guard the DID cache TTLs with its lock, not atomics - #191

Merged
ewanc26 merged 1 commit into
mainfrom
fix/wiiu-atomics
Oct 7, 2026
Merged

ewanc26 merged 1 commit into
mainfrom
fix/wiiu-atomics

Conversation

@ewanc26

@ewanc26 ewanc26 commented Oct 7, 2026

Copy link
Copy Markdown
Owner

What this changes

The DID document cache read its two TTLs through _Atomic time_t. On 32-bit PowerPC (the Wii U) a 64-bit time_t has no lock-free atomic load, so the library needed __atomic_load_8, which the devkitPPC link does not provide. Cobalt's .wuhb build failed to link once its sign-in began resolving DIDs (cobalt#202). The TTLs are now plain values guarded by the cache's existing lock.

Verification

  • cmake --build build-wiiu --target wolfram (the Wii U library) builds with no undefined references.
  • Host ctest: 152 of 153 pass. The one not run is moderation_demo, an example binary this build directory does not build; it is not touched by this change.

Docs

Docs: none (an internal synchronisation change; no documented behaviour changed). CHANGELOG entry under Fixed.

Changelog: CHANGELOG.md

Checklist

  • CI is green (CI gate and flow / conventions)
  • No secrets, tokens or signing keys in the diff
  • Tests added or changed with the behaviour; none skipped, disabled or deleted

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@ewanc26
ewanc26 merged commit 5ee79da into main Oct 7, 2026
17 checks passed
@ewanc26
ewanc26 deleted the fix/wiiu-atomics branch October 7, 2026 23:02
@ewanc26 ewanc26 mentioned this pull request Oct 7, 2026
2 of 3 tasks
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.

1 participant