On Tue, Apr 18, 2023 at 16:18:20 +0200, Samuel Thibault wrote: > Guillaume Nault, le mar. 18 avril 2023 15:38:00 +0200, a ecrit: > > On Tue, Apr 18, 2023 at 01:54:09PM +0200, Samuel Thibault wrote: > > > Guillaume Nault, le mar. 18 avril 2023 13:25:38 +0200, a ecrit: > > > > As I said in my previous reply, a simple L2TP example that goes until PPP > > > > channel and unit creation is fine. But any more advanced use of the PPP > > > > API should be documented in the PPP documentation. > > > > > > When it's really advanced, yes. But here it's just about tunnel > > > bridging, which is a very common L2TP thing to do. > > > > I can't undestand why you absolutely want this covered in l2tp.rst. > > Because that's where people working on L2TP software will look for it. Sorry to have not commented earlier, and thank you Samuel for working on improving the L2TP documentation. I think documentation like l2tp.rst is best when it provides a high level overview of how things fit together. When it comes to actually implementing a userspace L2TP/PPP daemon, I feel that at a certain point you're better off referring to existing userspace code alongside the kernel sources themselves, as any summary is inevitably going to leave gaps. From that perspective I'd almost sooner we didn't have the code snippet in l2tp.rst. That said, I can't see the harm in improving the code snippet, given that we have it already. Having no mention of PPPIOCBRIDGECHAN given that it can be used to implement tunnel switching is an oversight really. FWIW I agree the term "tunnel switching" is a bit misleading, and of course the PPP ioctl supports bridging any flavour of channel, not just PPPoL2TP. However from the L2TP perspective people perhaps have something along the lines of this IETF draft in mind: https://datatracker.ietf.org/doc/html/draft-ietf-l2tpext-tunnel-switching-08 ...which we could perhaps link to to clarify the intent in the context of the L2TP codebase? > > Also, it's probably a desirable feature, but certainly not a common > > thing on Linux. This interface was added a bit more than 2 years ago, > > which is really recent considering the age of the code. > > Yes, and in ISPs we have been in need for it for something like > decades. I can find RFC drafts around 2000. > > Or IPs have just baked their own kernel implementation (xl2tpd, > accel-ppp, etc.) Yes. It's sad that support wasn't available sooner in the kernel, but I'm not sure that's indicative of lack of desire for the feature necessarily. > > Appart from maybe go-l2tp, I don't know of any user. > I confirm that go-l2tp does use it :-) -- Tom Parkin Katalix Systems Ltd https://katalix.com Catalysts for your Embedded Linux software development
Attachment:
signature.asc
Description: PGP signature
