<p dir="ltr"><br>
On May 19, 2015 8:49 PM, "Yousong Zhou" <<a href="mailto:yszhou4tech@gmail.com">yszhou4tech@gmail.com</a>> wrote:<br>
><br>
> The first four patches are minor ones formed while makeing this series.  The<br>
> last two patches are required for L2TP protocol with xl2tpd to be working and<br>
> will also allow netifd to bring up interface on network device link lost in the<br>
> following scenarios.</p>
<p dir="ltr">Well, currently the patch misses a magic timeout callback to check if the link is up in the first place.  It intends to depend upon xl2tpd to redial with an interval longer than netifd's time of teardown and setup of the interface.</p>
<p dir="ltr">                yousong</p>
<p dir="ltr">><br>
>  - Quit of pppd without calling ppp-down script to explicity update link state<br>
>    in which case netifd would wrongly think the corresponding interface was<br>
>    still up.<br>
>  - With dummy `sleep 3600` as the proto_task running then issue `xl2tpd-control<br>
>    remove l2tp-xxx` to trigger the link lost.  Netifd got the Interface xxx has<br>
>    lost the connection information but did not try to teardown and bring up the<br>
>    interface as the proto_task was still running.<br>
>  - Crash xl2tpd to trigger a link lost and the result is the same as above.<br>
><br>
> Protocols PPPoE and L2TP were tested against this patch series on QEMU Malta<br>
> target and appeared to be working fine.<br>
><br>
> Yousong Zhou (6):<br>
>   Changes for more readability.<br>
>   system: fix typo in returning address length.<br>
>   interface: remove redundant iface_attr_info.<br>
>   interface: minor fix for unnecessary ++ operation.<br>
>   proto-shell: allow running protocols without proto_task.<br>
>   interface: teardown on l3_dev link lost.<br>
><br>
>  device.c                |    4 ++--<br>
>  device.h                |    3 +++<br>
>  interface.c             |   40 ++++++++++++++++++++++++++--------------<br>
>  interface.h             |    1 +<br>
>  proto-shell.c           |   20 ++++++++++++++------<br>
>  scripts/netifd-proto.sh |    2 ++<br>
>  system.h                |    2 +-<br>
>  ubus.c                  |    2 +-<br>
>  8 files changed, 50 insertions(+), 24 deletions(-)<br>
><br>
> --<br>
> 1.7.10.4<br>
><br>
</p>