pespin has uploaded this change for review.

View Change

bts-trx: define TRXD_MSG_BUF_SIZE as a more realistic lower value

Change-Id: Ic725e26637f8c116cfd5e64ff54d60a1d1bf9b87
---
M src/osmo-bts-trx/trx_if.h
1 file changed, 7 insertions(+), 2 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/60/41660/1
diff --git a/src/osmo-bts-trx/trx_if.h b/src/osmo-bts-trx/trx_if.h
index 13c537d..5f96ea2 100644
--- a/src/osmo-bts-trx/trx_if.h
+++ b/src/osmo-bts-trx/trx_if.h
@@ -1,9 +1,14 @@
#pragma once

+#include <osmocom/core/utils.h>
+
/* TRXC read/send buffer size */
#define TRXC_MSG_BUF_SIZE 1500
-/* TRXD read/send buffer size (max. IPv4 MTU) */
-#define TRXD_MSG_BUF_SIZE 65535U
+/* TRXD read/send buffer size: (2*GSM_BURST_LEN = VAMOS)
+ * NOTE: a legacy transceiver may append two garbage bytes.
+ * This ends up being (444+2)*8 = 3568 bytes */
+#define TRXD_BURST_SIZE_MAX OSMO_MAX(2 * (GSM_BURST_LEN + 2), (EGPRS_BURST_LEN + 2))
+#define TRXD_MSG_BUF_SIZE (TRXD_BURST_SIZE_MAX * TRX_NR_TS)

struct trx_dl_burst_req;
struct trx_l1h;

To view, visit change 41660. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ic725e26637f8c116cfd5e64ff54d60a1d1bf9b87
Gerrit-Change-Number: 41660
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>