Change in ...osmo-bsc[master]: doc/manuals, vty doc: more handover doc clarifications

neels gerrit-no-reply at lists.osmocom.org
Mon Jun 24 11:50:49 UTC 2019


neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/14582


Change subject: doc/manuals, vty doc: more handover doc clarifications
......................................................................

doc/manuals, vty doc: more handover doc clarifications

Related: OS#3487
Change-Id: I1639efb2dbcca4f0e9c33a74f3067606ce5f4209
---
M doc/manuals/chapters/handover.adoc
M include/osmocom/bsc/handover_cfg.h
M src/osmo-bsc/handover_vty.c
3 files changed, 25 insertions(+), 19 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/82/14582/1

diff --git a/doc/manuals/chapters/handover.adoc b/doc/manuals/chapters/handover.adoc
index 2f9d598..bb99751 100644
--- a/doc/manuals/chapters/handover.adoc
+++ b/doc/manuals/chapters/handover.adoc
@@ -434,7 +434,8 @@
 
 Configuration settings relevant for algorithm 1 start with `handover1`. For
 further details, please refer to the OsmoBSC VTY Reference
-(<<vty-ref-osmobsc>>) or the telnet VTY online documentation.
+(<<vty-ref-osmobsc>>) or the telnet VTY online documentation. See the
+`handover1` settings on the `config-net` and `config-net-bts` nodes.
 
 ==== Handover Algorithm 2
 
@@ -451,7 +452,8 @@
 
 Configuration settings relevant for algorithm 2 start with `handover2`. For
 further details, please refer to the OsmoBSC VTY Reference
-<<vty-ref-osmobsc>> or the telnet VTY online documentation.
+<<vty-ref-osmobsc>> or the telnet VTY online documentation. See the `handover2`
+settings on the `config-net` and `config-net-bts` nodes.
 
 ===== Load Distribution
 
@@ -472,8 +474,8 @@
 adhere to minimum reception levels and distance, see `min rxlev` and `max
 distance`.
 
-Load distribution will take effect only for already established voice channels.
-An MS will always first establish a voice call with its current cell choice; in
+Load distribution will take effect only for already established channels.
+For example, an MS will always first establish a voice call with its current cell choice; in
 load situations, it might be moved to another cell shortly after that.
 Considering the best neighbor _before_ starting a new voice call might be
 desirable, but is currently not implemented. Consider that RXLEV/RXQUAL ratings
diff --git a/include/osmocom/bsc/handover_cfg.h b/include/osmocom/bsc/handover_cfg.h
index 92b5cd4..f174aad 100644
--- a/include/osmocom/bsc/handover_cfg.h
+++ b/include/osmocom/bsc/handover_cfg.h
@@ -96,22 +96,22 @@
 		"handover1 ", "window rxlev averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER1 \
 		HO_CFG_STR_WIN_RXLEV \
-		"How many RxLev measurements are used for averaging\n" \
+		"How many RxLev measurements to use for averaging\n" \
 		"RxLev averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec1_rxqual_avg_win, 1, \
 		"handover1 ", "window rxqual averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER1 \
 		HO_CFG_STR_WIN_RXQUAL \
-		"How many RxQual measurements are used for averaging\n" \
+		"How many RxQual measurements to use for averaging\n" \
 		"RxQual averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec1_rxlev_neigh_avg_win, 10, \
 		"handover1 ", "window rxlev neighbor averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER1 \
 		HO_CFG_STR_WIN_RXLEV \
-		"How many Neighbor RxLev measurements are used for averaging\n" \
-		"How many Neighbor RxLev measurements are used for averaging\n" \
+		"How many Neighbor RxLev measurements to use for averaging\n" \
+		"How many Neighbor RxLev measurements to use for averaging\n" \
 		"Neighbor RxLev averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec1_pwr_interval, 6, \
@@ -142,22 +142,22 @@
 		"handover2 ", "window rxlev averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_WIN_RXLEV \
-		"How many RxLev measurements are used for averaging\n" \
+		"How many RxLev measurements to use for averaging\n" \
 		"RxLev averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec2_rxqual_avg_win, 1, \
 		"handover2 ", "window rxqual averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_WIN_RXQUAL \
-		"How many RxQual measurements are used for averaging\n" \
+		"How many RxQual measurements to use for averaging\n" \
 		"RxQual averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec2_rxlev_neigh_avg_win, 10, \
 		"handover2 ", "window rxlev neighbor averaging", "<1-10>", atoi, "%u", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_WIN_RXLEV \
-		"How many Neighbor RxLev measurements are used for averaging\n" \
-		"How many Neighbor RxLev measurements are used for averaging\n" \
+		"How many Neighbor RxLev measurements to use for averaging\n" \
+		"How many Neighbor RxLev measurements to use for averaging\n" \
 		"Neighbor RxLev averaging: " HO_CFG_STR_AVG_COUNT) \
 	\
 	HO_CFG_ONE_MEMBER(unsigned int, hodec2_pwr_interval, 6, \
@@ -184,7 +184,7 @@
 	HO_CFG_ONE_MEMBER(bool, hodec2_as_active, 0, \
 		"handover2 ", "assignment", "0|1", a2bool, "%d", bool2i, \
 		HO_CFG_STR_HANDOVER2 \
-		"Enable or disable in-call channel re-assignment\n" \
+		"Enable or disable in-call channel re-assignment within the same cell\n" \
 		"Disable in-call assignment\n" \
 		"Enable in-call assignment\n") \
 	\
@@ -200,7 +200,7 @@
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_MIN \
 		"How weak may RxLev of an MS become before triggering HO\n" \
-		"minimum RxLev (dBm)\n") \
+		"minimum RxLev (dBm; note: negative values)\n") \
 	\
 	HO_CFG_ONE_MEMBER(int, hodec2_min_rxqual, 5, \
 		"handover2 ", "min rxqual", "<0-7>", atoi, "%d", as_is, \
@@ -247,27 +247,30 @@
 		"handover2 ", "penalty-time max-distance", "<0-99999>", atoi, "%d", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_PENALTY_TIME \
-		"Time to suspend handovers after leaving this cell due to exceeding max distance\n" \
+		"Time to suspend handover for a subscriber after leaving this cell due to exceeding max distance;" \
+		" see also 'handover2 retries'\n" \
 		"Seconds\n") \
 	\
 	HO_CFG_ONE_MEMBER(int, hodec2_penalty_failed_ho, 60, \
 		"handover2 ", "penalty-time failed-ho", "<0-99999>", atoi, "%d", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_PENALTY_TIME \
-		"Time to suspend handovers after handover failure to this cell\n" \
+		"Time to suspend handover for a subscriber after a failed handover into this cell;" \
+		" see also 'handover2 retries'\n" \
 		"Seconds\n") \
 	\
 	HO_CFG_ONE_MEMBER(int, hodec2_penalty_failed_as, 60, \
 		"handover2 ", "penalty-time failed-assignment", "<0-99999>", atoi, "%d", as_is, \
 		HO_CFG_STR_HANDOVER2 \
 		HO_CFG_STR_PENALTY_TIME \
-		"Time to suspend handovers after assignment failure in this cell\n" \
+		"Time to suspend handover for a subscriber after a failed re-assignment within this cell;" \
+		" see also 'handover2 retries'\n" \
 		"Seconds\n") \
 	\
 	HO_CFG_ONE_MEMBER(int, hodec2_retries, 0, \
 		"handover2 ", "retries", "<0-9>", atoi, "%d", as_is, \
 		HO_CFG_STR_HANDOVER2 \
-		"Immediately retry on handover/assignment failure\n" \
+		"Number of times to immediately retry a failed handover/assignment, before a penalty time is applied\n" \
 		"Number of retries\n") \
 
 #define HO_CFG_ALL_MEMBERS \
diff --git a/src/osmo-bsc/handover_vty.c b/src/osmo-bsc/handover_vty.c
index 7979511..e9333ec 100644
--- a/src/osmo-bsc/handover_vty.c
+++ b/src/osmo-bsc/handover_vty.c
@@ -107,7 +107,8 @@
       "handover2 congestion-check (disabled|<1-999>|now)",
       HO_CFG_STR_HANDOVER2
       "Configure congestion check interval\n"
-      "Disable congestion checking, do not handover based on cell overload\n"
+      "Disable congestion checking, do not handover based on cell load. Note: there is one global congestion check"
+      " interval, i.e. contrary to other handover2 settings, this is not configurable per individual cell.\n"
       "Congestion check interval in seconds (default "
       OSMO_STRINGIFY_VAL(HO_CFG_CONGESTION_CHECK_DEFAULT) ")\n"
       "Manually trigger a congestion check to run right now\n")

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/14582
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1639efb2dbcca4f0e9c33a74f3067606ce5f4209
Gerrit-Change-Number: 14582
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190624/49bc4d56/attachment.html>


More information about the gerrit-log mailing list