search

filecoin-project / ref-fvm

Reference implementation of the Filecoin Virtual Machine
https://fvm.filecoin.io/
Other
376 stars 133 forks source link

Charge for wasm instrumentation code #727

Open Stebalien opened 2 years ago

Stebalien commented 2 years ago

Currently, we charge gas, but we don't charge for the logic that charges for gas. We should do that.

Basically, this means adding a small static charge per gas instrumentation point in wasm code.

maciejwitowski commented 1 year ago

@Stebalien do we need this for M2.1?

Stebalien commented 1 year ago

We can probably get away with punting on this until we've done the gas fuzzing. If this shows up as an issue, we can revisit it.

maciejwitowski commented 1 year ago

@aakoshh Just a heads up that it would be great if you helped us with this one 🙏