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

move statistics counter rlc_sent() to gprs_rlcmac_sched

The counter rlc_sent has nothing to do with the TBF.
The RLC packet got sent in the gprs_rlcmac_sched().

Change-Id: I5d2b910ea7cc250f17530406eda3be9b29b051fd
---
M src/gprs_rlcmac_sched.cpp
M src/tbf_dl.cpp
2 files changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/84/84/1

diff --git a/src/gprs_rlcmac_sched.cpp b/src/gprs_rlcmac_sched.cpp
index 313e23f..8e05623 100644
--- a/src/gprs_rlcmac_sched.cpp
+++ b/src/gprs_rlcmac_sched.cpp
@@ -341,8 +341,11 @@
                dl_ass_tbf, ul_ack_tbf);

        /* Prio 2: select data message for downlink */
-       if (!msg)
+       if (!msg) {
                msg = sched_select_downlink(bts, trx, ts, fn, block_nr, pdch);
+               if (msg)
+                       bts->bts->rlc_sent();
+       }

        /* Prio 3: send dummy contol message */
        if (!msg)
diff --git a/src/tbf_dl.cpp b/src/tbf_dl.cpp
index 9e4d078..62d378f 100644
--- a/src/tbf_dl.cpp
+++ b/src/tbf_dl.cpp
@@ -762,8 +762,6 @@
        /* Increment TX-counter */
        m_tx_counter++;

-       bts->rlc_sent();
-
        return dl_msg;
 }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5d2b910ea7cc250f17530406eda3be9b29b051fd
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus <[email protected]>

Reply via email to