Log (M)CS UL update errors
Previously some of the errors in update_cs_ul() call were silently
ignored. Let's log all those as errors with appropriate message.
Note: test output needs updating because we do not (yet) set proper meas
struct in TBF tests. That's likely wrong but it's better to update tests
in a separate commit.
Change-Id: I4084fb281dd9dad04a2a3a68cac2a8f7b462548e
diff --git a/src/gprs_ms.cpp b/src/gprs_ms.cpp
index 87a35d1..8855053 100644
--- a/src/gprs_ms.cpp
+++ b/src/gprs_ms.cpp
@@ -633,11 +633,19 @@
OSMO_ASSERT(current_cs_num > 0);
- if (!m_current_cs_ul)
+ if (!m_current_cs_ul) {
+ LOGP(DRLCMACMEAS, LOGL_ERROR,
+ "Unable to update UL (M)CS because it's not set: %s\n",
+ m_current_cs_ul.name());
return;
+ }
- if (!meas->have_link_qual)
+ if (!meas->have_link_qual) {
+ LOGP(DRLCMACMEAS, LOGL_ERROR,
+ "Unable to update UL (M)CS %s because we don't have link quality measurements.\n",
+ m_current_cs_ul.name());
return;
+ }
old_link_qual = meas->link_qual;
@@ -652,6 +660,9 @@
low = bts_data->mcs_lqual_ranges[current_cs_num-1].low;
high = bts_data->mcs_lqual_ranges[current_cs_num-1].high;
} else {
+ LOGP(DRLCMACMEAS, LOGL_ERROR,
+ "Unable to update UL (M)CS because it's neither GPRS nor EDGE: %s\n",
+ m_current_cs_ul.name());
return;
}