Bjoern A. Zeeb wrote:
Hi,

I have added some functionality to ng_ether / ng_pppoe that is needed
for running multiple pppoe sessions over one single NIC even if
the AC uses s.th. like 'pppoe duplicate-protection' [1].

I have seperated the patches for easier review and tried to do it in
a manner so that parts can be used for other things too (at least the
ng_ether one). I also updated the man pages to reflect these changes.

Here are the patches:


http://sources.zabbadoz.net/freebsd/patchset/10005-net-netgraph-ether.diff




this comment and the code disagree....
+       /* If "promisc" hook not connected, let packet continue */
+       if (priv->lower == NULL || priv->lowerType!=LOWER_TYPE_PROMISC) {
+               m_freem(m);
+               return;
+       }


if not connected.. discarded....


In the documentation:
+written to the hook. The interface has to be put into promiscuous before
+by using p.ex.
+.Dv NGM_ETHER_SET_PROMISC .
+Normal incoming traffic is unaffected.

what is the "p.ex" ?


! Description:  hook for ng_ether to receive unicast packets to a
!               different Ethernet address than the one of the interface.


http://sources.zabbadoz.net/freebsd/patchset/10006-netgraph-pppoe.diff


! Description:  2 extensions to ng_pppoe:
!               + in addition to AC NAME also report AC MAC to userspace
!               + add the ability to give a MAC as source MAC for the
!                 session when connecting




looks ok from a quick look...

what are you doing with these?
have you tested teh pppoe with:
1/ pppoed
2/ ppp  (obviously yes from your patch below)
3/ mpd

to check that it does not cause regressions?




Please review and commit if you like it.


I have tested both for some weeks now with following patch to ppp(8). It needs a lot more review but may be good enough for testing. Comments/diffs/improved versions on this one welcome.

http://sources.zabbadoz.net/freebsd/patchset/10008-usr-sbin-ppp-pppoe-smac.diff


hmmm, I see,, very interesting...

you should make an attempt to contact brian

! Description: extend ppp to make use of patch 10006 (set source MAC ! for pppoe session). ! ! Remarks: hopefully seperated patch correctly from 10007; ! I am almost sure this patch will break something else.

[it will most likely not apply cleanly without patch 10007 - see other
mail to current@ ]



[1] http://juniper.net/techpubs/software/erx/erx50x/swconfig-link/html/pppoe-config5.html#123643



--
+------------------------------------+       ______ _  __
|   __--_|\  Julian Elischer         |       \     U \/ / hard at work in
|  /       \ [EMAIL PROTECTED]     +------>x   USA    \ a very strange
| (   OZ    )                                \___   ___ | country !
+- X_.---._/    presently in San Francisco       \_/   \\
          v

_______________________________________________
[EMAIL PROTECTED] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to