-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ensure determinism #94
Comments
So this seems to have been done with https://github.com/massalabs/massa-sc-runtime/blob/main/src/tunable_memory.rs. Correct? @adrien-zinger |
And this is done at massa-sc-runtime/src/execution_impl.rs Line 29 in 6e722ee
|
Are these two fixed by using canonical nans? Source: https://github.com/WebAssembly/design/blob/037c6fe94151eb13e30d174f5f7ce851be0a573e/Nondeterminism.md |
should be, yes |
The only remaining potential source of nondeterminism is the stack limit. |
This seems like a good starting point: wasmerio/wasmer#2807 |
124: Remove several sources of non-determinism r=damip a=damip For determinism: #94 Co-authored-by: damip <[email protected]> Co-authored-by: gterzian <[email protected]>
Based on the discussion with Wasmer people, https://github.com/massalabs/massa-sc-runtime/pull/127/files adds stack checks. This should be enough to ensure determinism |
Discussed in #73
Originally posted by damip January 9, 2022
To make sure execution is deterministic we need to:
The text was updated successfully, but these errors were encountered: