Page MenuHomeFreeBSD

Fix MPD5 PPPoE support on vlans and add basic parsing of PADM messages

Authored by ale on Mar 1 2017, 10:49 PM.
Referenced Files
Unknown Object (File)
Fri, Nov 17, 9:22 PM
Unknown Object (File)
Wed, Nov 8, 12:25 PM
Unknown Object (File)
Wed, Nov 8, 12:20 PM
Unknown Object (File)
Nov 6 2023, 4:54 PM
Unknown Object (File)
Nov 5 2023, 5:09 AM
Unknown Object (File)
Oct 21 2023, 4:05 PM
Unknown Object (File)
Oct 20 2023, 11:34 AM
Unknown Object (File)
Oct 18 2023, 6:10 AM



The netgraph node paths cannot contain '.' and ':' characters, so the ng_ether module sanitizes interface names containing them (like the new format for vlans, eg. fxp0.1) by replacing illegal characters with underscores. This patch keeps interface name and netgraph path separated in the pppoe layer so that mpd can be used successfully on vlan interfaces.

iface = fxp0.1
path = fxp0_1:

Moreover I've added basic support to display PADM messages, introduced by the following patch:

Test Plan

I've setup pppoe connections on vlans and used ngctl to test the PADM messages.

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

ale retitled this revision from to Fix MPD5 PPPoE support on vlans and add basic parsing of PADM messages.
ale updated this object.
ale edited the test plan for this revision. (Show Details)
ale added a reviewer: mav.
ale set the repository for this revision to rP FreeBSD ports repository.
ale added a subscriber: julian.
mav edited edge metadata.

I have no objections.

This revision is now accepted and ready to land.Mar 2 2017, 5:50 AM

Do you plan the apply the patch upstream and release a new version or may I apply it directly to the ports tree?

looks ok.
Who;s going to commit?
any doc change required?

No need to change documentation IMO, the manual already states that you have to specify the ethernet interface name:

set pppoe iface interface-name

Set the name of the Ethernet interface to use. PPPoE does not affect normal use of the interface.

Hey since you are the new maintainer of the mpd5 port can you commit also this patch, please? Thanks

This revision was automatically updated to reflect the committed changes.