Commit 94f880ad authored by Cedric Roux's avatar Cedric Roux

bugfix: increase RLC UM header size

The constant RLC_UM_SEGMENT_NB_MAX_LI_PER_PDU is set to 24. This is
too low. Running a downlink TCP iperf with a 10MHz or 20MHz cell
leads to lots of the following warning in the log:

[RLC][W][SN 792] Bad RLC header! Discard this RLC PDU (size=618)

The problem is that the uplink RLC PDU contains a lot of TCP ack.
We can see much more than 24 of them. The RLC layer is not happy
and discards the PDU.

Putting 256 for RLC_UM_SEGMENT_NB_MAX_LI_PER_PDU (arbitrarily chosen,
big but not too big) solves the problem, at least for 10MHz.
parent fbcc9a23
...@@ -53,7 +53,7 @@ ...@@ -53,7 +53,7 @@
// UM_Window_Size = 512 when a 10 bit SN is configured and UM_Window_Size = 0 // UM_Window_Size = 512 when a 10 bit SN is configured and UM_Window_Size = 0
// when the receiving UM RLC entity is configured for MCCH or MTCH. // when the receiving UM RLC entity is configured for MCCH or MTCH.
// li field (values shifted 1 bit left) // li field (values shifted 1 bit left)
# define RLC_UM_SEGMENT_NB_MAX_LI_PER_PDU 24 # define RLC_UM_SEGMENT_NB_MAX_LI_PER_PDU 256
//---------------------------------------------------------- //----------------------------------------------------------
// Events defined for state model of the acknowledged mode entity // Events defined for state model of the acknowledged mode entity
/** Internal event defined for state model of the RLC UM. */ /** Internal event defined for state model of the RLC UM. */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment