From b3c99d8ead1ab6230fbeb435f51e852fde2d741b Mon Sep 17 00:00:00 2001 From: Robert Schmidt Date: Mon, 24 Nov 2025 14:45:41 +0100 Subject: [PATCH] Avoid recreation of LCIDs with wrong priority In all cases (NSA, HO), we create LCIDs before the UE triggers random access. Hence, process_addmod_bearers_cellGroupConfig() is not necessary, as it just tries to recreate an LCID. Furthermore, it includes a bug, as this function does set a wrong priority (0), so the sorting of LCIDs is not correct. This in turn creates problems with reestablishment, as LCID 1/SRB1 is not in the first position, making reestablishment fail (where we assume LCID 1 be in first position). The actual bug of setting priorities is fixed in the next commit. --- openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c | 1 - 1 file changed, 1 deletion(-) diff --git a/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c b/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c index 3f546d41b4..291e4266cb 100644 --- a/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c +++ b/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c @@ -832,7 +832,6 @@ static void nr_rx_ra_sdu(const module_id_t mod_id, nr_mac_reset_ul_failure(UE_scheduling_control); reset_dl_harq_list(UE_scheduling_control); reset_ul_harq_list(UE_scheduling_control); - process_addmod_bearers_cellGroupConfig(&UE->UE_sched_ctrl, UE->CellGroup->rlc_BearerToAddModList); int ss_type; // we configure the UE using common search space with DCIX0 while waiting for a reconfiguration in SA // in NSA (or do-ra) there is no reconfiguration in NR