Review at  https://gerrit.osmocom.org/6678

osmo-bts-virtual: Make sure PRIM_INFO_MEAS have non-zero frame number

Measurement reports fed into L1SAP so far had their frame number always
set to zero, resulting in higher-layer common code aboe L1SAP to never
detect the end of the measurement period, which in turn caused no RSL
MEAS REP to be sent.

Change-Id: I67837d19515ea335614928570c12dd5027104c6b
---
M src/osmo-bts-virtual/l1_if.c
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/78/6678/1

diff --git a/src/osmo-bts-virtual/l1_if.c b/src/osmo-bts-virtual/l1_if.c
index 01314fa..6904fe6 100644
--- a/src/osmo-bts-virtual/l1_if.c
+++ b/src/osmo-bts-virtual/l1_if.c
@@ -296,7 +296,7 @@
 
 
 static void l1if_fill_meas_res(struct osmo_phsap_prim *l1sap, uint8_t chan_nr, 
float ta,
-                               float ber, float rssi)
+                               float ber, float rssi, uint32_t fn)
 {
        memset(l1sap, 0, sizeof(*l1sap));
        osmo_prim_init(&l1sap->oph, SAP_GSM_PH, PRIM_MPH_INFO,
@@ -306,6 +306,7 @@
        l1sap->u.info.u.meas_ind.ta_offs_qbits = (int16_t)(ta*4);
        l1sap->u.info.u.meas_ind.ber10k = (unsigned int) (ber * 10000);
        l1sap->u.info.u.meas_ind.inv_rssi = (uint8_t) (rssi * -1);
+       l1sap->u.info.u.meas_ind.fn = fn;
 }
 
 static int l1if_process_meas_res(struct gsm_bts_trx *trx, uint8_t tn, uint32_t 
fn, uint8_t chan_nr,
@@ -321,7 +322,7 @@
                gsm_lchan_name(lchan), chan_nr, 
ms_pwr_dbm(lchan->ts->trx->bts->band, lchan->ms_power),
                rssi, ber*100, n_errors, n_bits_total, lchan->meas.l1_info[1], 
lchan->rqd_ta, toa);
 
-       l1if_fill_meas_res(&l1sap, chan_nr, lchan->rqd_ta + toa, ber, rssi);
+       l1if_fill_meas_res(&l1sap, chan_nr, lchan->rqd_ta + toa, ber, rssi, fn);
 
        return l1sap_up(trx, &l1sap);
 }

-- 
To view, visit https://gerrit.osmocom.org/6678
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I67837d19515ea335614928570c12dd5027104c6b
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>

Reply via email to