EIP-4844 Implementers' Call #5

[timbeiko]

Meeting Info

• Nov 22, 2022, 15:30 UTC
• Duration: 60 minutes
• Recording: https://youtu.be/tTF2VHcAMzE
• Notes & action items

📆 Subscribe to the Ethereum Protocol Call calendar for calendar invites

Agenda

• Spec updates
  • Sidecar optional if no blob? (context)
  • EIP-4844: Ancestor blob availability check consensus-specs#3099
  • [EIP-4844] Set A Minimum Data Gasprice EIPs#5862
  • [EIP-4844] Return Modulus From Precompile EIPs#5864
  • Update EIP-4844: Disable Blob Transaction Broadcast EIPs#5930
  • Update EIP-4844: Clarify Fork Block Behavior EIPs#6020
• Large Block Spam Test
• Devnet 3 updates
• Readiness checklist
  • Testing
  • kzg libraries
• Shanghai CFI consideration

[gakonst]

Code here https://github.com/gakonst/stress4844, tested for 128KB, blocked on finding an MEV Boost builder for 512KB/1024KB etc.

I've been spamming https://goerli.etherscan.io/address/0xaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa from https://goerli.etherscan.io/address/0xf5f02208279382cb8e6af943dafa8eac418bbdd3.

[asn-d6]

It might be fruitful to spend a few moments discussing whether we want to make sidecars optional when a block has no blob transactions. We have a cryptography PR which fixes an issue presented when this happens: ethereum/consensus-specs#3093

However, if other subsystems would benefit from sidecars being optional, we might be able to set stricter invariants in the cryptography library as well.

[adietrichs]

One more (hopefully last) spec PR update:

• [EIP-4844] Set A Minimum Data Gasprice EIPs#5862
  Given that this remains controversial and that we will start with a lower throughput anyway, my recommendation would be to stick with a min data gasprice of 1 and close this PR.
• [EIP-4844] Return Modulus From Precompile EIPs#5864
  Merged. Ended up going with padded return data, to match the style of existing precompiles.
• Update EIP-4844: Disable Blob Transaction Broadcast EIPs#5930
  Waiting for a final review, should hopefully be merged by the time of the call. Blob transactions are only announced, not broadcast.
• Update EIP-4844: Clarify Fork Block Behavior EIPs#6020
  For completeness: Specifies that excess_data_gas starts at 0 at the fork.

[protolambda]

After 5930 I think we can update the EIP to "review" status. Any objections?

[holgerd77]

Code here https://github.com/gakonst/stress4844, tested for 128KB, blocked on finding an MEV Boost builder for 512KB/1024KB etc.

I've been spamming https://goerli.etherscan.io/address/0xaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa from https://goerli.etherscan.io/address/0xf5f02208279382cb8e6af943dafa8eac418bbdd3.

That is super interesting, could you expand on this a bit what this stress test actually does? Does it simulate a (modified) devp2p network? Locally? Distributed? Or does it somehow interact with an existing network simulating an increased load with some tricks?

Anyhow: pretty cool! 🙂 💯

[adietrichs]

• [EIP-4844] Return Modulus From Precompile EIPs#5864
  Merged. Ended up going with padded return data, to match the style of existing precompiles.

For completeness: Just merged a tiny additional PR to clarify that the return data follows usual EVM big endianness:

• Update EIP-4844: Clarify Precompile Return Data Endianness EIPs#6026

[timbeiko]

Closed in favour of #679