diff options
| author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-05-22 18:02:51 +0200 |
|---|---|---|
| committer | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-05-28 13:59:25 +0200 |
| commit | 1db67e0a35a479ef20b871c459ea3f3e0155b620 (patch) | |
| tree | 7c15b7dbd27c6f181dcb9fd827d18627f76e4213 /src/bts.cpp | |
| parent | 9200ce60196a289968144582f1acfac25e17eed5 (diff) | |
tbf: Remove TimingAdvance storage
Currently the TA storage stores up to 30 TLLI->TA mappings, if more
entries are created the oldest one is dropped. In theory this can
lead to missing TA information if many MS are present.
This commit removes the TimingAdvance class completely, since the TA
value is now stored in the GprsMs objects.
Note that the GprsMs objects are currently not kept after the TBFs
have detached from them, so the TA values are now kept for a shorter
time than before.
Ticket: #1674
Sponsored-by: On-Waves ehf
Diffstat (limited to 'src/bts.cpp')
| -rw-r--r-- | src/bts.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/bts.cpp b/src/bts.cpp index 010b8e88..7225a78c 100644 --- a/src/bts.cpp +++ b/src/bts.cpp @@ -945,7 +945,13 @@ void gprs_rlcmac_pdch::rcv_measurement_report(Packet_Measurement_Report_t *repor "block, but there is no resource request " "scheduled! TLLI=0x%08x\n", report->TLLI); } else { - bts()->timing_advance()->remember(report->TLLI, sba->ta); + GprsMs *ms = bts()->ms_store().get_ms(report->TLLI); + if (!ms) + LOGP(DRLCMAC, LOGL_NOTICE, "MS send measurement " + "but TLLI 0x%08x is unknown\n", report->TLLI); + else + ms->set_ta(sba->ta); + bts()->sba()->free_sba(sba); } gprs_rlcmac_meas_rep(report); |