Vadim Yanitskiy has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/12183 )

Change subject: trx_toolkit/burst_fwd.py: also pass-filter UL bursts by 
time-slot
......................................................................

trx_toolkit/burst_fwd.py: also pass-filter UL bursts by time-slot

For some reason, the time-slot pass-filtering was only done for
DL bursts, but not for UL bursts. BurstForwarder shall not pass
UL bursts for unconfigured time-slots too.

Let's also print a warning if an UL burst is sent on a not
configured time-slot, i.e. before sending SETSLOT command.

Change-Id: Idb7f5b212e5814aeff8ca8bc875ad066674267cd
---
M src/target/trx_toolkit/burst_fwd.py
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Pau Espin Pedrol: Looks good to me, approved



diff --git a/src/target/trx_toolkit/burst_fwd.py 
b/src/target/trx_toolkit/burst_fwd.py
index 746b281..6e79d44 100644
--- a/src/target/trx_toolkit/burst_fwd.py
+++ b/src/target/trx_toolkit/burst_fwd.py
@@ -326,6 +326,11 @@
                if msg is None:
                        return None

+               # Timeslot filter
+               if msg.tn not in self.ts_pass_list:
+                       print("[!] TS %u is not configured, dropping UL 
burst..." % msg.tn)
+                       return None
+
                # Path loss simulation
                msg = self.path_loss_sim_ul(msg)
                if msg is None:

--
To view, visit https://gerrit.osmocom.org/12183
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Idb7f5b212e5814aeff8ca8bc875ad066674267cd
Gerrit-Change-Number: 12183
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>

Reply via email to