For reverted state modification operations, keep access cost, but refund modification cost.
Motivation
Reverting a transaction, or any of its sub-calls, drops any state modifications that happened inside.
But now, users are being charged for the dropped modifications as if they persisted.
Since EIP-3298, the gas refund mechanism works for storage restores only inside the same transaction. But on revert, the gas refund is not increased; it is completely erased.
It can even be cheaper to transfer tokens back at the end of a transaction instead of reverting, to keep the existing gas refund.
This should be changed.
Reverted SSTORE deserves to be repriced to SLOAD gas (100 gas)
Reverted LOG0, LOG1, LOG2, LOG3 and LOG4 deserve to be repriced to 100 gas
Reverted CALL with value (positive_value_cost = 9,000 gas) deserves to be repriced to 100 gas
Reverted CALL with value and account creation (value_to_empty_account_cost = 25,000 gas) deserves to be repriced to 100 gas
Reverted CREATE and CREATE2 (32,000 gas) deserve to be repriced to 100 gas
Reverted SELFDESTRUCT (5,000 or 25,000 gas) deserves to be repriced to 100 gas
Moreover, it seems fair to charge CREATE and CREATE2 operations 32,000 fix price conditionally only if returned bytecode is not empty.
Specification
For each callframe, track revert_gas_refund, initially 0.
The set of operations that modify revert_gas_refund are:
And in case of revert let’s use this value instead of just erasing gas_refund:
if(call.reverted){// existing behaviortx.gas_refund-=call.gas_refund;// New behavior added to existing according to the EIP-3978tx.gas_refund+=call.revert_gas_refund;}
Rationale
Gas should reflect the cost of use.
The revert cost reflects the cost of access during execution, but not the cost of modification.