On 2011-07-22 7:58 AM, Philip Prindeville wrote:
On 7/6/11 1:41 AM, Philip Prindeville wrote:
 Bump to the current version of PPP.

 A few of the patches are either in 2.4.5 or something functionally equivalent 
is, so those patches were removed.

 1 patch was added to accommodate pppol2pv3_addr which is referenced in a linux 
kernel header file, but isn't defined in the kernel header files packaged with 
PPP.

 Signed-off-by: Philip Prindeville<[email protected]>


Can I please get a code review of this?
Here's a few things spotted during review:

- You forgot to refresh the patches, there's lots of fuzz/offsets when applying them - The change that removes the exponential backoff during pppoe discovery was dropped, please add it back - Did you test whether pppd still properly survives errors during pppoe initialization (e.g. when receiving broken data from the AC)?

- Felix
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to