From 8bd2d709cf4a7fb5df60f843dd901a58c833167d Mon Sep 17 00:00:00 2001 From: Robert Schmidt Date: Wed, 9 Apr 2025 09:53:01 +0200 Subject: [PATCH 1/3] Revert "Fix PMI condition: avoid division by zero" This reverts commit 17e36029fae805cd0ed688701c669e7b18167bb8. --- openair1/PHY/NR_TRANSPORT/nr_dlsch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openair1/PHY/NR_TRANSPORT/nr_dlsch.c b/openair1/PHY/NR_TRANSPORT/nr_dlsch.c index aed99ff358..517d0ab974 100644 --- a/openair1/PHY/NR_TRANSPORT/nr_dlsch.c +++ b/openair1/PHY/NR_TRANSPORT/nr_dlsch.c @@ -468,7 +468,7 @@ static inline void do_txdataF(c16_t **txdataF, nfapi_nr_tx_precoding_and_beamforming_t *pb = &rel15->precodingAndBeamforming; while (rb < rel15->rbSize) { // get pmi info - const int pmi = (pb->num_prgs > 0 && pb->prg_size > 0) ? (pb->prgs_list[(int)rb / pb->prg_size].pm_idx) : 0; + const int pmi = (pb->prg_size > 0) ? (pb->prgs_list[(int)rb / pb->prg_size].pm_idx) : 0; const int pmi2 = (rb < (rel15->rbSize - 1) && pb->prg_size > 0) ? (pb->prgs_list[(int)(rb + 1) / pb->prg_size].pm_idx) : -1; // If pmi of next RB and pmi of current RB are the same, we do 2 RB in a row From e30fe594896b9682664bf49d7fb34682533a60c3 Mon Sep 17 00:00:00 2001 From: Robert Schmidt Date: Wed, 9 Apr 2025 09:53:32 +0200 Subject: [PATCH 2/3] increase RLC buffers for reduced losses --- openair2/LAYER2/nr_rlc/nr_rlc_entity.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openair2/LAYER2/nr_rlc/nr_rlc_entity.c b/openair2/LAYER2/nr_rlc/nr_rlc_entity.c index edfce652b8..6d35460331 100644 --- a/openair2/LAYER2/nr_rlc/nr_rlc_entity.c +++ b/openair2/LAYER2/nr_rlc/nr_rlc_entity.c @@ -100,8 +100,8 @@ nr_rlc_entity_t *new_nr_rlc_entity_am( exit(1); } - ret->tx_maxsize = tx_maxsize; - ret->rx_maxsize = rx_maxsize; + ret->tx_maxsize = tx_maxsize * 5; + ret->rx_maxsize = rx_maxsize * 5; ret->t_poll_retransmit = t_poll_retransmit; ret->t_reassembly = t_reassembly; From d2f332e18e581b081ddd2d52d42a3b9c2b332615 Mon Sep 17 00:00:00 2001 From: Robert Schmidt Date: Mon, 28 Apr 2025 10:12:38 +0200 Subject: [PATCH 3/3] Limit FHI-72 to two layers With PMI reporting working (see HEAD^^), it seems that four layers put two much processing strain on the machine (cacofonix), which asserts immediately. In tests, it was found that the same test throughput (600Mbps) can be achieved with two layers, while not crashing the machine. --- ci-scripts/conf_files/gnb.sa.band77.273prb.fhi72.4x4-vvdn.conf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ci-scripts/conf_files/gnb.sa.band77.273prb.fhi72.4x4-vvdn.conf b/ci-scripts/conf_files/gnb.sa.band77.273prb.fhi72.4x4-vvdn.conf index 4837e4f775..74ab33a28d 100644 --- a/ci-scripts/conf_files/gnb.sa.band77.273prb.fhi72.4x4-vvdn.conf +++ b/ci-scripts/conf_files/gnb.sa.band77.273prb.fhi72.4x4-vvdn.conf @@ -19,7 +19,7 @@ gNBs = pdsch_AntennaPorts_XP = 2; pdsch_AntennaPorts_N1 = 2; - maxMIMO_layers = 4; + maxMIMO_layers = 2; pusch_AntennaPorts = 4; do_CSIRS = 1; do_SRS = 0;