sched: log a warning when the CSI is invalid

Author: frankist

lib/scheduler/ue_scheduling/ue_cell.h

@@ -85,7 +85,9 @@ class ue_cell
   void handle_csi_report(const csi_report_data& csi_report)
   {
     set_fallback_state(false);
-    channel_state.handle_csi_report(csi_report);
+    if (not channel_state.handle_csi_report(csi_report)) {
+      logger.warning("ue={} rnti={:#x}: Invalid CSI report received", ue_index, rnti());
+    }
   }
 
   /// \brief Get the current UE cell metrics.

lib/scheduler/ue_scheduling/ue_channel_state_manager.cpp

@@ -44,7 +44,7 @@ ue_channel_state_manager::ue_channel_state_manager(const serving_cell_config&
   }
 }
 
-void ue_channel_state_manager::handle_csi_report(const csi_report_data& csi_report)
+bool ue_channel_state_manager::handle_csi_report(const csi_report_data& csi_report)
 {
   latest_csi_report = csi_report;
 
@@ -55,12 +55,16 @@ void ue_channel_state_manager::handle_csi_report(const csi_report_data& csi_repo
 
   // Update recommended number of layers based on RI.
   if (csi_report.ri.has_value()) {
+    if (csi_report.ri.value() > nof_dl_ports) {
+      return false;
+    }
     recommended_dl_layers = csi_report.ri.value().to_uint();
-    srsran_sanity_check(recommended_dl_layers <= nof_dl_ports, "Invalid RI reported");
   }
 
   if (csi_report.pmi.has_value()) {
     const unsigned table_idx        = nof_layers_to_index(recommended_dl_layers);
     recommended_prg_info[table_idx] = csi_report.pmi.value();
   }
+
+  return true;
 }

lib/scheduler/ue_scheduling/ue_channel_state_manager.h

@@ -55,7 +55,7 @@ class ue_channel_state_manager
   }
 
   /// Update UE with the latest CSI report for a given cell.
-  void handle_csi_report(const csi_report_data& csi_report);
+  bool handle_csi_report(const csi_report_data& csi_report);
 
 private:
   /// \brief Number of indexes -> nof_layers for precoding (Options: 1 layer, 2 layers, 4 layers).