rp-pppoe: add support for logical interface names and fix interface triggers

Signed-off-by: Alin Nastac <alin.nastac@gmail.com>

diff --git a/net/rp-pppoe/Makefile b/net/rp-pppoe/Makefile
index 89ee79930c7bdb66b06a17c3200db1a6f403b7ce..49a2c742a39dcc0c1e0496f3bb7f560588eb73f1 100644 (file)
--- a/net/rp-pppoe/Makefile
+++ b/net/rp-pppoe/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=rp-pppoe
 PKG_VERSION:=3.12
-PKG_RELEASE:=3
+PKG_RELEASE:=4
 PKG_MAINTAINER:=Daniel Dickinson <lede@cshore.thecshore.com>
 PKG_LICENSE:=LGPL-2.0+
 

diff --git a/net/rp-pppoe/files/pppoe-relay.init b/net/rp-pppoe/files/pppoe-relay.init
index 4cc034e38bc6fe0db6848e53d3c866e6e942551a..259d9a7894293224d9d442655bd1ae50032b424e 100755 (executable)
--- a/net/rp-pppoe/files/pppoe-relay.init
+++ b/net/rp-pppoe/files/pppoe-relay.init
@@ -13,20 +13,14 @@ pppoe_triggers() {
     config_get server_interfaces "$cfg" server_interface
     config_get client_interfaces "$cfg" client_interface
     config_get both_interfaces "$cfg" both_interfaces
-    for interface in $server_interfaces; do
-       append interfaces "$interface" "|"
-    done
-    for interface in $client_interfaces; do
-       append interfaces "$interface" "|"
-    done
-    for interface in $both_interfaces; do
-       append interfaces "$interface" "|"
+    for interface in $server_interfaces $client_interfaces $both_interfaces; do
+       procd_add_reload_interface_trigger $interface
     done
 }
 
 pppoe_relay_instance() {
     local cfg="$1"
-    local enabled interface server_interfaces client_interfaces both_interfaces maxsessions timeout OPTIONS
+    local enabled interface device server_interfaces client_interfaces both_interfaces maxsessions timeout OPTIONS
     config_get_bool enabled "$cfg" enabled 1
     [ "$enabled" -gt 0 ] || return 0
     config_get server_interfaces "$cfg" server_interface
@@ -39,23 +33,34 @@ pppoe_relay_instance() {
     if [ "$use_non_uci_config" -gt 0 ]; then
        . /etc/default/pppoe-relay
     else
-       [ -z "${server_interfaces}${client_interfaces}${both_interfaces}" ] && return 1
+       local NEED_INTFS=SC
+       . /lib/functions/network.sh
        for interface in $server_interfaces; do
-           append OPTIONS "-S $interface"
+           if network_get_physdev device $interface; then
+               append OPTIONS "-S $device"
+               NEED_INTFS=${NEED_INTFS/S/}
+           fi
        done
        for interface in $client_interfaces; do
-           append OPTIONS "-C $interface"
+           if network_get_physdev device $interface; then
+               append OPTIONS "-C $device"
+               NEED_INTFS=${NEED_INTFS/C/}
+           fi
         done
        for interface in $both_interfaces; do
-           append OPTIONS "-B $interface"
+           if network_get_physdev device $interface; then
+               append OPTIONS "-B $device"
+               NEED_INTFS=${NEED_INTFS/?/}
+           fi
        done
+       [ -z "${NEED_INTFS}" ] || return 1 # need at least 2 interfaces, one for server(s) and one for client(s)
        [ -n "$maxsessions" ] && append OPTIONS "-n $maxsessions"
        [ -n "$timeout" ] && append OPTIONS "-i $timeout"
     fi
 
     procd_open_instance
-    procd_set_param command /usr/sbin/pppoe-relay -F
-    procd_append_param command $OPTIONS
+    procd_set_param command /usr/sbin/pppoe-relay -F $OPTIONS
+    procd_set_param respawn
     procd_close_instance
 }
 
@@ -66,12 +71,9 @@ start_service() {
     config_foreach pppoe_relay_instance pppoe_relay
 }
 
-reload_triggers() {
-    local interfaces
+service_triggers() {
+    procd_add_reload_trigger "pppoe"
 
     config_load pppoe
     config_foreach pppoe_triggers pppoe_relay
-
-    procd_add_reload_trigger "pppoe"
-    procd_add_interface_trigger "$interfaces"
 }

diff --git a/net/rp-pppoe/files/pppoe-server.init b/net/rp-pppoe/files/pppoe-server.init
index f0555ebd30da9d3b0eee93a96c7468b59a22a072..8ec0acc394a53d037e95886a5e7fe86d5cc6eb1d 100755 (executable)
--- a/net/rp-pppoe/files/pppoe-server.init
+++ b/net/rp-pppoe/files/pppoe-server.init
@@ -11,11 +11,12 @@ pppoe_triggers() {
     config_get_bool enabled "$cfg" enabled 1
     [ "$enabled" -gt 0 ] || return 0
     config_get interface "$cfg" interface
+    procd_add_reload_interface_trigger $interface
 }
 
 pppoe_instance() {
     local cfg="$1"
-    local enabled interface ac_name service_names service_name maxsessionsperpeer localip firstremoteip maxsessions optionsfiles randomsession unit offset timeout mss sync OPTIONS
+    local enabled interface device ac_name service_names service_name maxsessionsperpeer localip firstremoteip maxsessions optionsfiles randomsession unit offset timeout mss sync OPTIONS
     config_get_bool enabled "$cfg" enabled 1
     [ "$enabled" -gt 0 ] || return 0
     config_get interface "$cfg" interface
@@ -37,12 +38,13 @@ pppoe_instance() {
     if [ "$use_non_uci_config" -gt 0 ]; then
        . /etc/default/pppoe-server
     else
-       [ -z "$interface" ] && return 1
+       . /lib/functions/network.sh
+       network_get_physdev device $interface || return 1
        [ -n "$ac_name" ] && append OPTIONS "-C $ac_name"
        for service_name in $service_names; do
            append OPTIONS "-S $service_name"
        done
-       append OPTIONS "-I $interface"
+       append OPTIONS "-I $device"
        [ -n "$maxsessionsperpeer" ] && append OPTIONS "-x $maxsessionsperpeer"
        [ -n "$localip" ] && append OPTIONS "-L $localip"
        [ -n "$firstremoteip" ] && append OPTIONS "-R $firstremoteip"
@@ -57,8 +59,8 @@ pppoe_instance() {
     fi
 
     procd_open_instance
-    procd_set_param command /usr/sbin/pppoe-server -F
-    procd_append_param command -k $OPTIONS
+    procd_set_param command /usr/sbin/pppoe-server -F -k $OPTIONS
+    procd_set_param respawn
     procd_set_param file /etc/ppp/options
     procd_append_param file /etc/ppp/pppoe-server-options
     procd_close_instance
@@ -70,10 +72,8 @@ start_service() {
 }
 
 service_triggers() {
-    local interface
+    procd_add_reload_trigger "pppoe"
+
     config_load pppoe
     config_foreach pppoe_triggers pppoe_server
-
-    procd_add_reload_trigger "pppoe"
-    procd_add_interface_trigger "$interface"
 }