Merge pull request #45 from openNDS/v3.1.0beta

V3.1.0beta

Author: bluewavenet

ChangeLog

@@ -1,4 +1,5 @@
 mesh11sd (3.1.0)
+  * Add - send meaningful multipath warnings to syslog [bluewavenet]
   * Fix - diplay warning message only if path_change_count changes [bluewavenet]
   * Add - adjust bridge ruleset and do checks to ensure bridge ports all have different mac addresses [bluewavenet]
   * Fix - remove ruleset on daemon stop [bluewavenet]
@@ -19,7 +20,7 @@ mesh11sd (3.1.0)
   * Fix - set procd parameters to respawn zombie nodes [bluewavenet]
   * Add - README examples of using copy/connect [bluewavenet]
 
- -- Rob White <dot@blue-wave.net>  Sat, 20 Mar 2024 11:38:51 +0000
+ -- Rob White <dot@blue-wave.net>  Thu, 21 Mar 2024 07:55:51 +0000
 
 mesh11sd (3.0.0)
   * Add - startup check for ath10k-ct drivers and terminate if found [bluewavenet]

README.md

@@ -184,7 +184,7 @@ config mesh11sd 'setup'
 	#
 	# All mesh peer and mesh gate nodes will autonomously track the mesh portal channel
 	# regardless of the configured auto_mesh_band
-	# 
+	#
 	#option portal_channel 'auto'
 	# or
 	#option portal_channel '4'
@@ -292,7 +292,7 @@ Access to the remote meshnode peers will not be possible using the ipv4 address
 
 * interface_timeout - the time in seconds that mesh11sd will wait for a mesh interface to establish before continuing. Default 10 seconds
 
-* mesh_path_cost - sets the STP cost of the mesh network. Default: 0. Can be set to any value from 0 to 65534. Setting to 0 disables STP
+* mesh_path_cost - sets the STP cost of the mesh network. Default: 10. Can be set to any value from 0 to 65534. Setting to 0 disables STP
 
 * auto_config - (optional) - autonomously configures the mesh network. Default 1 (enabled). Set to 0 to disable
 

src/mesh11sd

@@ -1493,25 +1493,30 @@ check_path_changes() {
 
 							if [ "$is_connected" -eq 0 ]; then
 								debugtype="warn"
-								syslogmessage="[ $hop_count ] hops to station [ $peermac ], [ $path_change_count ] path_change(s) for station detected"
+								syslogmessage="Warning! Station [ $peermac ] is [ $hop_count ] hops away and [ $path_change_count ] path_change(s) for station have been detected"
 								write_to_syslog
-								debugtype="warn"
-								syslogmessage="There are multiple hops to station [ $peermac ], consider its location or adjust txpower and/or rssi_threshold"
+								syslogmessage="Warning! Station [ $peermac ], consider its location or adjust txpower and/or rssi_threshold"
 								write_to_syslog
 								# If we want to make reactive fixups, do them here, for example:
 								#iw dev $iface station del $peermac
 								#iw dev $iface mpath del $peermac
 							fi
 
 						elif [ "$hop_count" -eq 1 ] && [ "$path_change_count" -gt 1 ]; then
-							# If we want to make reactive fixups, do them here, for example:
-							#iw dev $iface mpath del $peermac
 							debugtype="warn"
-							syslogmessage="Station [ $peermac ] has [ $path_change_count ] path changes, consider its location or adjust txpower and/or rssi_threshold"
+							syslogmessage="Warning! Station [ $peermac ] is an immediate neighbour, but has had [ $path_change_count ] path_change(s) detected"
+							write_to_syslog
+							syslogmessage="Warning! Station [ $peermac ], consider its location or adjust txpower and/or rssi_threshold"
 							write_to_syslog
+							# If we want to make reactive fixups, do them here, for example:
+							#iw dev $iface mpath del $peermac
 						fi
 
 						echo "previous_path_change_count=$path_change_count" > /tmp/$peermac_id
+					else
+						debugtype="info"
+						syslogmessage="Path to station [ $peermac ] is stable"
+						write_to_syslog
 					fi
 				done
 			fi