Change in osmo-gsm-tester[master]: enb_srs: add extra head room for max rate with 6 PRB in MIMO mode

[pespin]

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/18636 )

Change subject: enb_srs: add extra head room for max rate with 6 PRB in MIMO mode
......................................................................

enb_srs: add extra head room for max rate with 6 PRB in MIMO mode

the overhead with 6 PRB and MIMO is a bit higher when compared
to other PRBs resulting in lower achievable throughput

Change-Id: I63888435553bba4f7be88cc745e24472921a7fb4
---
M src/osmo_gsm_tester/obj/enb_srs.py
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py b/src/osmo_gsm_tester/obj/enb_srs.py
index 33ee16b..9dc8113 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -252,8 +252,13 @@
             max_rate = max_phy_rate_tm1_ul[self.num_prb()]
 
         # MIMO only supported for Downlink
-        if downlink and self._txmode > 2:
-            max_rate *= 2
+        if downlink:
+            if self._txmode > 2:
+                max_rate *= 2
+
+            # For 6 PRBs the max throughput is significantly lower
+            if self._txmode >= 2 and self.num_prb() == 6:
+                max_rate *= 0.85
 
         return max_rate
 

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/18636
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I63888435553bba4f7be88cc745e24472921a7fb4
Gerrit-Change-Number: 18636
Gerrit-PatchSet: 5
Gerrit-Owner: srs_andre <andre at softwareradiosystems.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200610/5588d79b/attachment.htm>