Skip to content

feat: count the base "block open" fee against memory retention - #1413

Closed
Stebalien wants to merge 1 commit into
masterfrom
feat/reduce-memory-retention-again
Closed

feat: count the base "block open" fee against memory retention#1413
Stebalien wants to merge 1 commit into
masterfrom
feat/reduce-memory-retention-again

Conversation

@Stebalien

Copy link
Copy Markdown
Member

This effectively removes the memory retention fee from newly opened blocks. The cross-over point is ~44MiB....

This will save about 4.4K gas per 1K block opened (about 2% of the total gas value). Not much, but it helps a bit.

Really, the main reason I made this PR is that it simplifies the FIP a bit. Now I can just talk about the total block open gas charge as:

min(min_gas_retention_per_byte * size, block_open_fee)

This effectively removes the memory retention fee from newly opened
blocks. The cross-over point is ~44MiB....

This will save about 4.4K gas per 1K block opened (about 2% of the total
gas value). Not much, but it helps a bit.
@Stebalien
Stebalien force-pushed the feat/reduce-memory-retention-again branch from 101dcd0 to 1b82836 Compare January 10, 2023 19:23
@Stebalien
Stebalien requested review from arajasek and raulk January 10, 2023 19:33
@Stebalien
Stebalien marked this pull request as draft January 12, 2023 22:00
@Stebalien Stebalien added this to the M2.2 milestone Jan 12, 2023
@Stebalien Stebalien closed this Jan 12, 2023
@Stebalien
Stebalien deleted the feat/reduce-memory-retention-again branch April 4, 2025 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant