consensus-shipyard / ipc

🌳 Spawn multi-level trees of customized, scalable, EVM-compatible networks with IPC. L2++ powered by FVM, Wasm, libp2p, IPFS/IPLD, and CometBFT.
https://ipc.space
Apache License 2.0
42 stars 39 forks source link

Enforce new base fee in mempool after gas market base fee update #1183

Open cryptoAtwill opened 3 days ago

cryptoAtwill commented 3 days ago

With the introduction of gas market, block gas limit and base fee rotation were introduced. However, there is a missing piece when the base fee is updated, it should be enforced in the mempool for message selections: i.e. message in mempool below the new base fee, should be deprioritised for message selection.

cryptoAtwill commented 3 days ago

There are two main solutions proposed:

Method 2 is preferred at the moment.

sanderpick commented 3 days ago

when you say,

i.e. message in mempool below the new base fee, should be deprioritised for message selection.

do this imply that during time of zero network usage (zero block gas utilization), that a user could set gas_fee_cap to 0 and get free block space? this is an edge case but just curious what the intention is.

cryptoAtwill commented 3 days ago

@sanderpick zero network usage should not have cause base fee to 0, the current gas market actor has a minimal_base_fee setting (non-zero generally) that prevents this from happening. So even if a block is empty, there is no free ride.

sanderpick commented 12 hours ago

cool, makes sense. so instead of "message in mempool below the new base fee, should be deprioritised for message selection", it should just be rejected / dropped

cryptoAtwill commented 1 hour ago

@sanderpick Not sure if we should drop them, because base fee varies, so next block those txns might have reached the base fee requirement.