Downgrade gNB power limit LOG from "warning" to "debug"
Downgrade the UE power limited LOG from LOG_W to LOG_D. This is to avoid flooding stdout when UE in low coverage enters power limited state for a longer period of time. The same information can be inferred from the periodic UE print which also contains PH value (negative PH values indicate that the UE is power limited)
Also, correct some typos in PH calculation.
Closes #885 (closed)
Merge request reports
Activity
changed milestone to %REVIEW_CAN_START
added 5G-NR label
requested review from @francescomani
OAI RAN-Container-Parent build (11799): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/11799/)
List of failing test stages:
- Resolved by Francesco Mani
Before demoting the LOG, are we sure we shouldn't be worried about that?
mentioned in merge request !3233 (merged)
- Resolved by Francesco Mani
While trying to better understand the implication of the log, I realized there is a bug in PH calculation
diff --git a/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c b/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c index a43492fcfc..1e27cf1ef0 100644 --- a/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c +++ b/openair2/LAYER2/NR_MAC_gNB/gNB_scheduler_ulsch.c @@ -453,7 +453,7 @@ static int nr_process_mac_pdu(instance_t module_idP, if (phr->PH < 55) { PH = phr->PH - 32; } else if (phr->PH < 63) { - PH = 28 + (phr->PH - 55) * 2; + PH = 24 + (phr->PH - 55) * 2; } else { PH = 42; }
changed milestone to %REVIEW_IN_PROGRESS
added 210 commits
-
c11b0608...64a472ae - 208 commits from branch
develop
- 1e667b37 - Downgrade gNB power limit LOG from "warning" to "debug"
- d44279a9 - Fix PHR PH value to power headroom conversion.
-
c11b0608...64a472ae - 208 commits from branch
changed milestone to %REVIEW_COMPLETED_AND_APPROVED
- Resolved by Robert Schmidt
OAI RAN-Container-Parent build (11971): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/11971/)
List of failing test stages:
changed milestone to %OK_TO_BE_MERGED
mentioned in commit 616c6775
mentioned in commit 0053a3d0
merged with merge request !3233 (merged)