Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Evaluate viability of using wasmi's out-of-box fuel metering #683

Closed
jayz22 opened this issue Feb 13, 2023 · 1 comment · Fixed by #815
Closed

Evaluate viability of using wasmi's out-of-box fuel metering #683

jayz22 opened this issue Feb 13, 2023 · 1 comment · Fixed by #815
Assignees
@graydon @jayz22

Comments

@jayz22
Copy link
Contributor

jayz22 commented Feb 13, 2023 •
edited
Loading

Wasmi recently added out-of-the-box support for gas metering. Issue and PR.

This issue tracks the work needed to evaluate its viability in our use case.
@jayz22 jayz22 self-assigned this Feb 13, 2023
@jayz22 jayz22 mentioned this issue Feb 24, 2023
Closed
@jayz22
Copy link
Contributor Author

jayz22 commented May 4, 2023

Want to make a note that the new metering charging model of giving the VM a preallocated budget (cpu and mem) and let it perform accounting and limit-checking aligns well with the requirements of account abstract and operation bundling, see this discord thread for details. Just a bit of extra motivation.

@jayz22 jayz22 mentioned this issue Jun 2, 2023
Merged
This was referenced Jun 5, 2023
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@graydon graydon

@jayz22 jayz22

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Switch to wasmi native fuel metering jayz22/rs-soroban-env
2 participants
@graydon @jayz22