Pau Espin Pedrol has submitted this change and it was merged. (
https://gerrit.osmocom.org/13789 )
Change subject: lms: Catch and log dropped packets by HW during recv
......................................................................
lms: Catch and log dropped packets by HW during recv
Change-Id: I23554d95b0aff585024610fc12920c9da4f3ba9e
---
M Transceiver52M/device/lms/LMSDevice.cpp
M Transceiver52M/device/lms/LMSDevice.h
2 files changed, 9 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
Harald Welte: Looks good to me, approved
diff --git a/Transceiver52M/device/lms/LMSDevice.cpp
b/Transceiver52M/device/lms/LMSDevice.cpp
index e65c93d..7071589 100644
--- a/Transceiver52M/device/lms/LMSDevice.cpp
+++ b/Transceiver52M/device/lms/LMSDevice.cpp
@@ -54,6 +54,7 @@
m_last_rx_underruns.resize(chans, 0);
m_last_rx_overruns.resize(chans, 0);
+ m_last_rx_dropped.resize(chans, 0);
m_last_tx_underruns.resize(chans, 0);
}
@@ -587,6 +588,13 @@
<< status.overrun << ")";
}
m_last_rx_overruns[chan] = status.overrun;
+
+ if (status.droppedPackets > m_last_rx_dropped[chan]) {
+ LOGC(DDEV, ERROR) << "chan " << chan << ": recv Dropped
packets by HW! ("
+ << m_last_rx_dropped[chan] << " -> "
+ << status.droppedPackets << ")";
+ }
+ m_last_rx_dropped[chan] = m_last_rx_overruns[chan];
}
}
diff --git a/Transceiver52M/device/lms/LMSDevice.h
b/Transceiver52M/device/lms/LMSDevice.h
index 4bf2b32..225839d 100644
--- a/Transceiver52M/device/lms/LMSDevice.h
+++ b/Transceiver52M/device/lms/LMSDevice.h
@@ -50,6 +50,7 @@
std::vector<uint32_t> m_last_rx_underruns;
std::vector<uint32_t> m_last_rx_overruns;
+ std::vector<uint32_t> m_last_rx_dropped;
std::vector<uint32_t> m_last_tx_underruns;
double actualSampleRate; ///< the actual USRP sampling rate
--
To view, visit https://gerrit.osmocom.org/13789
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I23554d95b0aff585024610fc12920c9da4f3ba9e
Gerrit-Change-Number: 13789
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
