Skip to content

Commit 8cce136

Browse files
frankistcodebot
authored andcommitted
gnb: test mode now operates in two modes: with and without ul phy
1 parent 097a079 commit 8cce136

File tree

7 files changed

+297
-135
lines changed

7 files changed

+297
-135
lines changed

apps/gnb/gnb_appconfig.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -863,6 +863,10 @@ struct test_mode_ue_appconfig {
863863
rnti_t rnti = rnti_t::INVALID_RNTI;
864864
/// Number of test UE(s) to create.
865865
uint16_t nof_ues = 1;
866+
/// \brief Delay, in slots, before the MAC test mode auto-generates the UCI/CRC indication to pass to the scheduler.
867+
/// This feature should be avoided if the OFH/UL PHY are operational, otherwise the auto-generated indications
868+
/// may interfere with the UL PHY HARQ handling.
869+
optional<unsigned> auto_ack_indication_delay;
866870
/// Whether PDSCH grants are automatically assigned to the test UE.
867871
bool pdsch_active = true;
868872
/// Whether PUSCH grants are automatically assigned to the test UE.

apps/gnb/gnb_appconfig_cli11_schema.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1603,6 +1603,11 @@ static void configure_cli11_test_ue_mode_args(CLI::App& app, test_mode_ue_appcon
16031603
app.add_option("--nof_ues", test_params.nof_ues, "Number of test UE(s) to create.")
16041604
->capture_default_str()
16051605
->check(CLI::Range((uint16_t)1, (uint16_t)MAX_NOF_DU_UES));
1606+
app.add_option("--auto_ack_indication_delay",
1607+
test_params.auto_ack_indication_delay,
1608+
"Delay before the UL and DL HARQs are automatically ACKed. This feature should only be used if the UL "
1609+
"PHY is not operational")
1610+
->capture_default_str();
16061611
app.add_option("--pdsch_active", test_params.pdsch_active, "PDSCH enabled")->capture_default_str();
16071612
app.add_option("--pusch_active", test_params.pusch_active, "PUSCH enabled")->capture_default_str();
16081613
app.add_option("--cqi", test_params.cqi, "Channel Quality Information (CQI) to be forwarded to test UE.")

apps/gnb/gnb_du_factory.cpp

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -183,16 +183,18 @@ std::vector<std::unique_ptr<du>> srsran::make_gnb_dus(const gnb_appconfig&
183183

184184
// Configure test mode
185185
if (gnb_cfg.test_mode_cfg.test_ue.rnti != rnti_t::INVALID_RNTI) {
186-
du_hi_cfg.test_cfg.test_ue = srs_du::du_test_config::test_ue_config{gnb_cfg.test_mode_cfg.test_ue.rnti,
187-
gnb_cfg.test_mode_cfg.test_ue.nof_ues,
188-
gnb_cfg.test_mode_cfg.test_ue.pdsch_active,
189-
gnb_cfg.test_mode_cfg.test_ue.pusch_active,
190-
gnb_cfg.test_mode_cfg.test_ue.cqi,
191-
gnb_cfg.test_mode_cfg.test_ue.ri,
192-
gnb_cfg.test_mode_cfg.test_ue.pmi,
193-
gnb_cfg.test_mode_cfg.test_ue.i_1_1,
194-
gnb_cfg.test_mode_cfg.test_ue.i_1_3,
195-
gnb_cfg.test_mode_cfg.test_ue.i_2};
186+
du_hi_cfg.test_cfg.test_ue =
187+
srs_du::du_test_config::test_ue_config{gnb_cfg.test_mode_cfg.test_ue.rnti,
188+
gnb_cfg.test_mode_cfg.test_ue.nof_ues,
189+
gnb_cfg.test_mode_cfg.test_ue.auto_ack_indication_delay,
190+
gnb_cfg.test_mode_cfg.test_ue.pdsch_active,
191+
gnb_cfg.test_mode_cfg.test_ue.pusch_active,
192+
gnb_cfg.test_mode_cfg.test_ue.cqi,
193+
gnb_cfg.test_mode_cfg.test_ue.ri,
194+
gnb_cfg.test_mode_cfg.test_ue.pmi,
195+
gnb_cfg.test_mode_cfg.test_ue.i_1_1,
196+
gnb_cfg.test_mode_cfg.test_ue.i_1_3,
197+
gnb_cfg.test_mode_cfg.test_ue.i_2};
196198
}
197199
// FAPI configuration.
198200
du_cfg.fapi.log_level = gnb_cfg.log_cfg.fapi_level;

include/srsran/du/du_test_config.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ struct du_test_config {
2020
struct test_ue_config {
2121
rnti_t rnti;
2222
uint16_t nof_ues;
23+
optional<unsigned> auto_ack_indication_delay;
2324
bool pdsch_active;
2425
bool pusch_active;
2526
unsigned cqi;

0 commit comments

Comments
 (0)