[c-lightning] Splitting gossipd
decker.christian at gmail.com
Mon Jul 2 02:01:02 AEST 2018
Thanks Rusty for taking the lead here, this is a very much needed
simplification of how we handle peer connections. Splitting out a
`connectd` is definitely a good idea, also because it moves any code
that is directly in contact with the peers into a separate binary and
away from our global daemons.
Maybe we could even go a step further and merge all the daemons
(`handshaked`, `openingd`, `channeld`, `closingd`) that handle the peer
connection, into `channeld`. We've discussed this a few times before,
but I think this would be a major simplification (there is only ever one
owner, no hand-over between daemons of the peer fd, no need to drain
message queues, ...), and it's much more in line with other projects
that have separate workers, e.g., apache2, at a reasonable cost IMHO.
This way we could move the listening socket back to the main daemon,
which would then just take the peer fd, and fork off the `channeld` for
that, using fd inheritance (which also simplifies alternative
implementations since we don't need to use fd-passing as much.
We'd lose the simple synchronous nature of `openingd` and `closingd`,
but I don't think that that's a major loss. Also I don't think that the
flexibility argument to keep separate daemon applies here, since I don't
really see us switching out `openingd` with a custom implementation for
Looking forward to hearing what you guys think about this.
Rusty Russell <rusty at rustcorp.com.au> writes:
> Hi all,
> Now we're post 0.6, I want to address some technical debt.
> Much of that means enhancing unit tests and writing some canned protocol
> tests (which will let us do fuzz testing), but we need to make sure our
> architecture is correct otherwise we'll just end up writing tests twice.
> The current gossipd maintains the graph topology, but *also* connects
> and chats to all peers which are not part of an active channel. These
> two roles are quite separate, though it's slightly more efficient than
> having a separate daemon because we don't need to pass messages back and
> forth between them.
> Unfortunately, the transition between gossipd and the other daemons has
> been a source of bugs: it'd be nice to isolate this. Thus I've begun
> splitting the "connect and handshake" role of gossipd into a new
> "connectd". That's turned out pretty clean.
> connectd itself has two parts: firstly it is responsible for answering
> incoming connections, and trying to connect out to peers we want to talk
> to, then handing them to the master. Secondly, it takes peers from the
> master which don't have a channel, and sits there forwarding gossip.
> This "peer in gossip mode" handling is completely separate from the
> "establish connection" part of connectd; worse, if the master decides it
> *does* want to talk to a channel, it has to sync any partially read and
> written gossip messages.
> If, instead, we have a per-peer daemon which handles "just gossiping",
> this is not a problem, as each daemon can be synchronous (thus
> simpler). It consumes a bit more resources per peer, but it now means
> that *all peers* post-init are isolated into their own daemons.
> I've yet to complete my implementation, but I thougth I'd share the
> design so the patch doesn't come as a complete shock.
> c-lightning mailing list
> c-lightning at lists.ozlabs.org
More information about the c-lightning