Skip to content

Quodss/urwasm stateful#785

Merged
pkova merged 25 commits intourbit:next/kelvin/409from
dozreg-toplud:quodss/urwasm-stateful
Jul 17, 2025
Merged

Quodss/urwasm stateful#785
pkova merged 25 commits intourbit:next/kelvin/409from
dozreg-toplud:quodss/urwasm-stateful

Conversation

@dozreg-toplud
Copy link
Copy Markdown
Contributor

@dozreg-toplud dozreg-toplud commented Mar 23, 2025

Depends on urbit/wasm3#1

Adds stateful jet for ++run:wasm-lia:

  • Defines two sets of allocators for m3 structs and codepages;
  • Updates _reduce_monad to enable computation suspension;
  • Adds _get_state to load wasm3 state from persistent cache table;
  • Adds _move_state to store wasm3 state in persistent cache table with the new key and remove an entry with the old key;
  • updates u3we_lia_run version due to return type change in Quodss/urwasm stateful urbit#7128;

Misc:
builtins for c3_d, motes

TODO:

  • optimizations:
    • arena on a slab to prevent copying data one more time;
    • smarter m3 struct realloc (currently wastes memory since wasm3 uses realloc to grow dynamic arrays);

@dozreg-toplud dozreg-toplud marked this pull request as ready for review April 25, 2025 14:52
@dozreg-toplud dozreg-toplud requested a review from a team as a code owner April 25, 2025 14:52
@dozreg-toplud dozreg-toplud force-pushed the quodss/urwasm-stateful branch from ccc7df8 to 2c96610 Compare May 19, 2025 15:45
@dozreg-toplud
Copy link
Copy Markdown
Contributor Author

incremented registration tag to reflect change in urbit/urbit side of the PR (new feature)

@dozreg-toplud dozreg-toplud changed the base branch from develop to next/kelvin/409 June 18, 2025 17:57
@dozreg-toplud
Copy link
Copy Markdown
Contributor Author

I changed the target from develop to 409 without rebasing, is it ok? this should be merged with 409

@dozreg-toplud dozreg-toplud force-pushed the quodss/urwasm-stateful branch from ab15fb0 to 28c2442 Compare July 7, 2025 15:48
@pkova pkova merged commit 690763c into urbit:next/kelvin/409 Jul 17, 2025
2 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