Forming a Ring: ETH v64 Wire Protocol Ring

@matthalp here is the proposal I promised:

Great! I will review and comment there.

@mbaxer I think it makes a lot of sense to have explicit chain state request/responses instead of relying on receiving the NewBlock/NewBlockHashes methods from peers. It’s also worth noting that chain state updates changes from a pub-sub type model to a polling model. There are some slight trade-offs here but a client typically communicates with at most 25 peers at a time, so there probably isn’t the scale to make any of the trade-offs notable.

The total difficulty messages are also interesting, but I have to think that through more. This morning I was wondering why headers don’t just include total difficulty instead of just the difficulty at that block, which may also solve some of the issues here (and it’s sealed in the header).

Just adding this here to not forget: It is probably worth removing the bloom for each receipt in the Receipts message since that information can be reconstructed from the block. Downloading receipts in fast sync is already pretty bandwidth-intensive and receipts consist of the outstanding majority of data downloaded as compared to the actual account states.

1 Like

I run a LES lightserv geth node that typically handles ~500 peers. It used to do ~1500 at the height of the KryptoKitty boom.

Changing from PUB-SUB to polling will have a negative impact on a minority of nodes in the network; a minority I wouldn’t flinch calling vulnerable.

@veox Thank you for pointing this out. I was specifically talking about ETH peers but had not discussed LES peers.

In LES there is an Annouce message that does broadcast the head hash, number, and TD (and some other metadata), which solves some of the issues @mbaxter is alluding to. Perhaps it is worth adopting this message in ETH if we do consider merging ETH and LES. Pub-Sub would remain under this paradigm, which I think does make sense.

1 Like

Sorry to break in off-topic, but is there a document which describes the major advances in the Wire Protocol, or events which led to certain important upgrades?

I created a versions doc for the Protocol itself, will soon create one for the EVM. It would be great to be able to do this for the Wire Protocol.

@ferranbt Can you link to where the message sizes are specified in Parity and Geth? My understanding is that the maximum message size should be 16 MB (uncompressed).

I think we should try to get this standardized if there is ambiguity here or at least have a discussion about why they are different.

There is a limit in the number of objects (headers, receipts…) or bytes sent per request (soft limit) in Geth and Parity. However, those limits are different in both implementations: max number of objects (Geth, Parity) and soft limit (Geth, Parity)

Related to this is a new proposal which would facilitate this I think: A cross-protocol, cross-implementation standard for server capacity management and flow control

@ferranbt From an implementation perspective it seems like you could be conservative and allocate the maximum 16 GB that is agreed upon between the clients. Less conservative would be to take the upper bound between the two.

I’m not sure I can see doing better than that. The number of domain objects or the soft limit are really implementors’ choice. If you have strong feelings I’d reach out to the client developers either through issues on their repo or through one of the messaging systems they use.

I talked offline with @zsfelfoldi and it seems like folding in LES to ETH is going to too ambitious for ETH v64.

Generally speaking about generality with peer-to-peer networking: some of the proposals for generalizing concepts smells to me like something that should be modifications to the RLPx wire protocol itself and not sub-protocols. The specific cases here are the (1) request IDs, (2) client-side flow control, and the general key-value list. I think request IDs can be addressed this round, but client-side flow control and generalized key-value list may need to wait. Personally, I am not a fan of ambiguity in specifications as they can be abused. I like adding things with specific use cases as needed. Implementors willing to put in the work to make the changes demonstrates that it is worthwhile.

Thank you everyone for your contributions! There are a lot of great ideas here.

I want to take a moment to establish the scope of ETH v64:

  1. Protocol warts: remove glaring holes, ambiguity, technical in current ETH v64 protocol (possibly at the RLPx wire level)
  2. Synchronization: make it easier to write synchronization code
  3. State: make client state more apparent amongst peers (e.g. archive vs. full) and even reduce data transferred (receipt bloom filters I’m looking at you)

Over the next few days I’m going to draft “minimum viable EIPs” for the ideas that seem like can be candidates for ETH v64. Each of these EIPs will have proper attribution to those who proposed it. I will ping either author to drive it hence forward unless they are not interested. That is not to say the ideas I do not draft into EIPs are not worthwhile, I just do not see them within the immediate scope of the three goals for ETH v64 mentioned above.

Let mebe explicit that amongst my proposals on here I only consider myself an author of Making pruned state explicit as the others are previously established ideas from prior works.

The next step is then getting client implementor buy in. This seems to be the right place to get consensus because (1) they are the implementors and (2) this is so low-level that other stakeholders higher in the stack will not even notice.

Yeah I am not sure what the obstacle would be. IMO flow control and capacity management should not be part of the light protocol at all, but a cross cutting concern affecting all protocols and servers that require quota and rate limiting and eviction policies.

I don’t see any particular great need though to merge LES with ETH.

I don’t see any particular great need though to merge LES with ETH.

As mentioned above I think over time we will see the lines blur between an ETH and LES client. At the ETH level we already have full- and fast-synced nodes and hiding their differences does cause strain on being able to sync. My rationale was that if clients are already having to think about the full-/fast- distinction then light clients only extend the spectrum.

Wearing my go-ethereum hat, I just feel like there could be some code consolidation to make things more maintainable. But perhaps that is a nit.

Check out the linked proposal. I agree there are commonalities, but I think the same thing applies to Whisper, Swarm etc., and capacity management is a problem for the whole server. Why have rate and quota limits applied to specific clients alone, when the same logic can be set as a server limit too? So rather than focusing on merging LES into ETH to take advantage of rate limiting, I propose taking rate limiting out of LES and making it a general sub-protocol.

I’m with you on rate limiting. I was moreso speaking about the actual messages supported by LES and ETH since they are both handling blockchain and account state constructs (plus the additional cryptographic constructs in LES). Whisper and Swarm have fundamentally different messages.

I’ve written a paper about a new sync protocol and algorithm, Red Queen’s. See also my presentation at Ethereum Core Devs Eth1x/Istanbul Planning Meeting in Berlin. It’s essentially an attempt to formalise Leaf Sync.

We can discuss it here or in a dedicated thread,

Happy to join forces with anybody working on new sync protocols.

@yperbasis Let’s keep it in a separate thread but make sure that the two works complement each other. For example, we may not want to have reqId solely live on the new sync message instead of all ETH messages. We have to figure out what makes sense there.

1 Like

See https://github.com/ethereum/pm/issues/129#issuecomment-536700238 . We would like to release 64 as a minor change to 63, whereby a forkid is used in the place of genesis hash for handshake.
I personally don’t think it matter if all the cool stuff you’ve been discussing above comes in eth65 instead. Anyone disagree? We would like to move forward with @karalabe’s forkid idea

2 Likes