From bc429eae94e033125266bc9f9c544cad94ee4bf6 Mon Sep 17 00:00:00 2001 From: Francisco Paisana Date: Fri, 9 Oct 2020 17:21:47 +0100 Subject: [PATCH] add comment explaining why BSR has to be accounting in the computation of the number of bytes to allocate in UL --- srsenb/src/stack/mac/scheduler_ue.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/srsenb/src/stack/mac/scheduler_ue.cc b/srsenb/src/stack/mac/scheduler_ue.cc index 88ff85cdc..b68596061 100644 --- a/srsenb/src/stack/mac/scheduler_ue.cc +++ b/srsenb/src/stack/mac/scheduler_ue.cc @@ -998,7 +998,9 @@ uint32_t sched_ue::get_pending_ul_new_data(uint32_t tti, int this_ue_cc_idx) } } if (pending_data > 0) { - // BSR is expected + // The scheduler needs to account for the possibility of BSRs being allocated in the UL grant. + // Otherwise, the UL grants allocated for very small RRC messages (e.g. rrcReconfigurationComplete) + // may be fully occupied by a BSR, and RRC the message transmission needs to be postponed. pending_data += (pending_lcgs <= 1) ? sbsr_size : lbsr_size; } else { if (is_sr_triggered() and active_lcgs > 0 and this_ue_cc_idx >= 0) {