AllCoreDevs - Testing CL Breakout #094
Transcript
- Justin Traglia
https://github.com/ethereum/pm/issues/2204#issuecomment-5459533350
- Justin Traglia
- Nico Flaig
you don't miss the proposal though, you'd just fall back to local block
- Jihoon
maybe we should go with nico's suggestion for simplicity?
- Jihoon
i think it's good to specify this in the spec and let clients to have some freedom
- Nico Flaig
seems fine to me
- Mehdi Aouadi
already in Teku’s devnet 0 branch
- Justin Traglia
Reacted to "maybe we should go with nico's suggestion for simplicity?" with 👍
- Justin Traglia
Regarding KZG taking up a bunch of time, I’ve been using vkzg (Vulcan KZG) for a few weeks on my node at home. This moves KZG cell recovery & proof computations to the GPU. This is just a little side project at the moment. https://github.com/jtraglia/vkzg
- Enrico Del Fante (tbenr)
Reacted to "Regarding KZG taking up a bunch of time, I’ve been using vkzg (Vulcan KZG) for a few weeks on my node at home. This moves KZG cell recovery & proof computations to the GPU. This is just a little side project at the moment. https://github.com/jtraglia/vkzg" with 🔥
- Potuz
Reacted to "Regarding KZG taking..." with 🔥
- Luca | Vero
Reacted to "Regarding KZG taking up a bunch of time, I’ve been using vkzg (Vulcan KZG) for a few weeks on my node at home. This moves KZG cell recovery & proof computations to the GPU. This is just a little side project at the moment. https://github.com/jtraglia/vkzg" with 🔥
- Jihoon
Reacted to "Regarding KZG taking up a bunch of time, I’ve been using vkzg (Vulcan KZG) for a few weeks on my node at home. This moves KZG cell recovery & proof computations to the GPU. This is just a little side project at the moment. https://github.com/jtraglia/vkzg" with 🔥
- Nico Flaig
Reacted to "Regarding KZG taking..." with 🔥
Call summary
Targets
- •FOCIL DevNet 0 — discuss devnet results in tomorrow's FOCIL breakout - 00:16:38
Decisions
- •ePBS bid gossip fix: ignore bids from builders with an exit in the previous full payload (address map check, no state mutation) - 00:07:21
- •Add non-empty transaction assertion to FOCIL gossip rule; start with 4,112 inclusion list size limit - 00:15:50
Highlights
- Epbs Devnet Testing:
- ·Plan to test payloads with >8,192 deposits on DevNet 8; pk910 has tooling (batching contract) ready - 00:10:24
- ·Justin needs >8,000 ETH for top-ups; pk910 will supply testnet ETH - 00:11:06
- Focil Gossip Checks:
- ·Proposal to add gossip check rejecting empty transactions in inclusion lists (FOCIL) - 00:11:33
- ·Teku already has this in their DevNet 0 branch (per chat) - 00:15:38
- ·Starting with 4,112 max inclusion list size; Jihoon to clean up consensus-specs PR and open new engine API PR - 00:16:03
- Epbs Bid Gossip Validation:
- ·Bug: builder with exit in parent payload can submit invalid bid, blocking other builders - 00:01:06
- ·Agreed fix: ignore bids from builders with an exit in the previous full payload (Nico's suggestion) - 00:07:21
- ·Implementation: keep a map of exited builder addresses from previous payload; check pub key + source address match before rejecting - 00:08:27
- Ptc Attestation Performance:
- ·Potuz: PTC attestation validation is a surprising CPU bottleneck alongside KZG data column checks - 00:17:34
- ·Root cause: no subnets for PTC attestations, so all messages are received by all nodes - 00:21:49
- ·Mitigation: move PTC validation to background thread; BLS and KZG offloaded to background improves performance significantly - 00:19:39
- ·Justin shared vkzg side project: offloads KZG cell recovery and proof computation to GPU via Vulkan - 00:23:22
Action Items
- •Jihoon - Open new engine API PR for FOCIL empty-transaction gossip check and 4,112 max size limit - 00:16:09
- •All CL teams - Include FOCIL devnet 0 changes in client DevNet 0 branches - 00:16:29
Key decisions
ePBS bid gossip fix: ignore bids from builders with an exit in the previous full payload (address map check, no state mutation)
Fixes a bug where a builder with an exit in the parent payload could submit an invalid bid and block other builders.Add non-empty transaction assertion to FOCIL gossip rule; start with 4,112 inclusion list size limit
Establishes gossip validation rules for Forward Inclusion Lists (FOCIL) and sets the initial maximum size limit.Plan to test payloads with >8,192 deposits on DevNet 8
DevNet 8Testing scalability of deposits using a batching contract and tooling provided by pk910.Include FOCIL devnet 0 changes in client DevNet 0 branches
DevNet 0Consensus Layer teams agreed to incorporate FOCIL gossip checks and size limits into their respective DevNet 0 branches.
AI Disclaimer: Some content or metadata on EIPsInsight may be AI-inferred or automatically compiled. If you find any discrepancy, please contact us at dev@avarch.org.