Fix: remove useless mutexlock.

openair2/LAYER2/MAC/eNB_scheduler_dlsch.c

@@ -1009,9 +1009,9 @@ schedule_ue_spec(module_id_t module_idP,
                                               , 0
 #endif
                                              );
-            pthread_mutex_lock(&rrc_release_freelist);
 
             if((rrc_release_info.num_UEs > 0) && (rlc_am_mui.rrc_mui_num > 0)) {
+              pthread_mutex_lock(&rrc_release_freelist);
               uint16_t release_total = 0;
 
               for (release_num = 0, release_ctrl = &rrc_release_info.RRC_release_ctrl[0];
@@ -1058,9 +1058,9 @@ schedule_ue_spec(module_id_t module_idP,
                 if(release_total >= rrc_release_info.num_UEs)
                   break;
               }
+              pthread_mutex_unlock(&rrc_release_freelist);
             }
 
-            pthread_mutex_unlock(&rrc_release_freelist);
 
             for (ra_ii = 0, ra = &eNB->common_channels[CC_id].ra[0]; ra_ii < NB_RA_PROC_MAX; ra_ii++, ra++) {
               if ((ra->rnti == rnti) && (ra->state == MSGCRNTI)) {

openair2/LAYER2/MAC/eNB_scheduler_fairRR.c

@@ -1291,9 +1291,9 @@ schedule_ue_spec_fairRR(module_id_t module_idP,
                                               ,0, 0
 #endif
                                              );
-            pthread_mutex_lock(&rrc_release_freelist);
 
-            if((rrc_release_info.num_UEs > 0) && (rlc_am_mui.rrc_mui_num > 0)) {
+            if((rrc_release_info.num_UEs > 0) && (rlc_am_mui.rrc_mui_num > 0)){
+              pthread_mutex_lock(&rrc_release_freelist);
               uint16_t release_total = 0;
 
               for(uint16_t release_num = 0; release_num < NUMBER_OF_UE_MAX; release_num++) {
@@ -1330,9 +1330,9 @@ schedule_ue_spec_fairRR(module_id_t module_idP,
                 if(release_total >= rrc_release_info.num_UEs)
                   break;
               }
+              pthread_mutex_unlock(&rrc_release_freelist);
             }
 
-            pthread_mutex_unlock(&rrc_release_freelist);
             RA_t *ra = &eNB->common_channels[CC_id].ra[0];
 
             for (uint8_t ra_ii = 0; ra_ii < NB_RA_PROC_MAX; ra_ii++) {

openair2/LAYER2/MAC/eNB_scheduler_primitives.c

@@ -2319,9 +2319,8 @@ rrc_mac_remove_ue(module_id_t mod_idP,
                    rntiP);
   }
 
+  if(rrc_release_info.num_UEs > 0){
   pthread_mutex_lock(&rrc_release_freelist);
-
-  if (rrc_release_info.num_UEs > 0) {
     uint16_t release_total = 0;
 
     for (uint16_t release_num = 0; release_num < NUMBER_OF_UE_MAX; release_num++) {
@@ -2341,6 +2340,7 @@ rrc_mac_remove_ue(module_id_t mod_idP,
         break;
       }
     }
+    pthread_mutex_unlock(&rrc_release_freelist);
   }
 
   pthread_mutex_unlock(&rrc_release_freelist);