Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
…nto dev
  • Loading branch information
Giulio2002 committed Dec 6, 2024
2 parents a6b22e5 + a4c7589 commit ff67ed1
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion configs/mainnet.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ DEPOSIT_CONTRACT_ADDRESS: 0x00000000219ab540356cBB839Cbe05303d7705Fa

# Networking
# ---------------------------------------------------------------
# `10 * 2**20` (= 15728640, 15 MiB)
# `15 * 2**20` (= 15728640, 15 MiB)
GOSSIP_MAX_SIZE: 15728640
# `2**10` (= 1024)
MAX_REQUEST_BLOCKS: 1024
Expand Down
2 changes: 1 addition & 1 deletion configs/minimal.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ DEPOSIT_CONTRACT_ADDRESS: 0x1234567890123456789012345678901234567890

# Networking
# ---------------------------------------------------------------
# `10 * 2**20` (= 10485760, 15 MiB)
# `15 * 2**20` (= 15728640, 15 MiB)
GOSSIP_MAX_SIZE: 15728640
# `2**10` (= 1024)
MAX_REQUEST_BLOCKS: 1024
Expand Down
3 changes: 2 additions & 1 deletion specs/bellatrix/p2p-interface.md
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,8 @@ Per `context = compute_fork_digest(fork_version, genesis_validators_root)`:

With the addition of `ExecutionPayload` to `BeaconBlock`s, there is a dynamic
field -- `transactions` -- which can validly exceed the `GOSSIP_MAX_SIZE` limit (1 MiB) put in
place at Phase 0, so GOSSIP_MAX_SIZE has increased to 10 Mib on the network, then in prospect to a gas limit increase it was increased again to 15 Mib.
place at Phase 0, so GOSSIP_MAX_SIZE has increased to 10 MiB on the network, then
in prospect to a gas limit increase it was increased again to 15 MiB.
At the `GAS_LIMIT` (~30M) currently seen on mainnet in 2021, a single transaction
filled entirely with data at a cost of 16 gas per byte can create a valid
`ExecutionPayload` of ~2 MiB. Thus we need a size limit to at least account for
Expand Down

0 comments on commit ff67ed1

Please sign in to comment.