From 13757862fb00de2adf2ccefa7000211f742cabf2 Mon Sep 17 00:00:00 2001 From: Cedric Roux Date: Sat, 18 Nov 2023 21:56:42 +0100 Subject: [PATCH] hotfix: set PDCP t-reordering to 100ms The function generateDRB() sets the value to 100ms but the function rrc_gNB_process_NGAP_PDUSESSION_SETUP_REQ() sets it to 0ms. So what happens is that the UE gets the value 100ms in the RRC Reconfiguration but the gNB creates the PDCP entity with 0ms. Let's create it with 100ms instead, to be aligned with how we configure the UE. And also avoid some PDCP warnings at runtime when some packets are retransmitted but rejected because of the 0ms timer. This is the warning: [PDCP] discard NR PDU rcvd_count=68469, entity->rx_deliv 68481,sdu_in_list 0 This is just a hotfix. A real solution is to have the RB creation in only one place. And we should also have those parameters configurable. Work for later. --- openair2/RRC/NR/rrc_gNB_NGAP.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openair2/RRC/NR/rrc_gNB_NGAP.c b/openair2/RRC/NR/rrc_gNB_NGAP.c index 57bb9b1bdf..f6059a91a9 100644 --- a/openair2/RRC/NR/rrc_gNB_NGAP.c +++ b/openair2/RRC/NR/rrc_gNB_NGAP.c @@ -807,7 +807,7 @@ void rrc_gNB_process_NGAP_PDUSESSION_SETUP_REQ(MessageDef *msg_p, instance_t ins drb->pDCP_SN_Size_DL = E1AP_PDCP_SN_Size_s_18; drb->discardTimer = E1AP_DiscardTimer_infinity; - drb->reorderingTimer = E1AP_T_Reordering_ms0; + drb->reorderingTimer = E1AP_T_Reordering_ms100; drb->rLC_Mode = E1AP_RLC_Mode_rlc_am;