Refactor process_addmod_bearers_cellGroupConfig() and set priority

Move the file process_addmod_bearers_cellGroupConfig() to the file where
it is used. Correctly set priorities as requested in RLC bearer
configuration.

openair2/LAYER2/NR_MAC_gNB/config.c

@@ -1099,6 +1099,13 @@ void nr_mac_configure_sib1(gNB_MAC_INST *nrmac, const plmn_id_t *plmn, uint64_t
   AssertFatal(cc->sib1_bcch_length > 0, "could not encode SIB1\n");
 }
 
+static bool process_addmod_bearers_cellGroupConfig(NR_UE_sched_ctrl_t *sched_ctrl, const NR_RLC_BearerConfig_t *conf)
+{
+  int priority = conf->mac_LogicalChannelConfig->ul_SpecificParameters->priority;
+  nr_lc_config_t c = {.lcid = conf->logicalChannelIdentity, .priority = priority};
+  return nr_mac_add_lcid(sched_ctrl, &c);
+}
+
 bool nr_mac_add_test_ue(gNB_MAC_INST *nrmac, uint32_t rnti, NR_CellGroupConfig_t *CellGroup)
 {
   /* ideally, instead of this function, "users" of this function should call
@@ -1120,7 +1127,9 @@ bool nr_mac_add_test_ue(gNB_MAC_INST *nrmac, uint32_t rnti, NR_CellGroupConfig_t
   }
   int ss_type = NR_SearchSpace__searchSpaceType_PR_ue_Specific;
   configure_UE_BWP(nrmac, nrmac->common_channels[0].ServingCellConfigCommon, UE, false, ss_type, -1, -1);
-  process_addmod_bearers_cellGroupConfig(&UE->UE_sched_ctrl, CellGroup->rlc_BearerToAddModList);
+  const struct NR_CellGroupConfig__rlc_BearerToAddModList *l = CellGroup->rlc_BearerToAddModList;
+  for (int i = 0; l != NULL && i < l->list.count; ++i)
+    process_addmod_bearers_cellGroupConfig(&UE->UE_sched_ctrl, CellGroup->rlc_BearerToAddModList->list.array[i]);
   AssertFatal(CellGroup->rlc_BearerToReleaseList == NULL, "cannot release bearers while adding new UEs\n");
   NR_SCHED_UNLOCK(&nrmac->sched_lock);
   LOG_I(NR_MAC, "Added new UE %x with initial CellGroup\n", rnti);

openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_primitives.c

@@ -3963,23 +3963,6 @@ void nr_mac_trigger_reconfiguration(const gNB_MAC_INST *nrmac, const NR_UE_info_
     free_cellGroupConfig(cellGroup_for_UE);
 }
 
-/* \brief add bearers from CellGroupConfig.
- *
- * This is a kind of hack, as this should be processed through a F1 UE Context
- * setup request, but some modes do not use that (NSA/do-ra/phy_test).  */
-void process_addmod_bearers_cellGroupConfig(NR_UE_sched_ctrl_t *sched_ctrl, const struct NR_CellGroupConfig__rlc_BearerToAddModList *addmod)
-{
-  if (addmod == NULL)
-    return; /* nothing to do */
-
-  for (int i = 0; i < addmod->list.count; ++i) {
-    const NR_RLC_BearerConfig_t *conf = addmod->list.array[i];
-    int lcid = conf->logicalChannelIdentity;
-    nr_lc_config_t c = {.lcid = lcid};
-    nr_mac_add_lcid(sched_ctrl, &c);
-  }
-}
-
 long get_lcid_from_drbid(int drb_id)
 {
   return drb_id + 3; /* LCID is DRB + 3 */

openair2/LAYER2/NR_MAC_gNB/mac_proto.h

@@ -480,9 +480,6 @@ bool nr_mac_check_ul_failure(gNB_MAC_INST *nrmac, int rnti, NR_UE_sched_ctrl_t *
 
 void nr_mac_trigger_reconfiguration(const gNB_MAC_INST *nrmac, const NR_UE_info_t *UE, int new_bwp_id);
 
-void process_addmod_bearers_cellGroupConfig(NR_UE_sched_ctrl_t *sched_ctrl,
-                                            const struct NR_CellGroupConfig__rlc_BearerToAddModList *addmod);
-
 bool nr_mac_add_lcid(NR_UE_sched_ctrl_t *sched_ctrl, const nr_lc_config_t *c);
 nr_lc_config_t *nr_mac_get_lc_config(NR_UE_sched_ctrl_t* sched_ctrl, int lcid);
 bool nr_mac_remove_lcid(NR_UE_sched_ctrl_t *sched_ctrl, long lcid);