Ciro Santilli (Gerrit)
2018-11-21 18:08:52 UTC
Ciro Santilli has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/14497
Change subject: base: increase the GDB buffer size
......................................................................
base: increase the GDB buffer size
The size was not large enough for the 'G' packet on aarch64, which the
client sends to set registers.
This would lead to the stub not to be able to find the end of the input
packet and keep waiting forever.
Change-Id: Icb149f15a6c769371ebcb6ec5fbebc6170c31fc6
---
M src/base/remote_gdb.cc
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/base/remote_gdb.cc b/src/base/remote_gdb.cc
index 37ecef9..1257d5c 100644
--- a/src/base/remote_gdb.cc
+++ b/src/base/remote_gdb.cc
@@ -152,7 +152,9 @@
static const char GDBGoodP = '+';
static const char GDBBadP = '-';
-static const int GDBPacketBufLen = 1024;
+// This value was found by trial and error to be enough for
+// all tested packets.
+static const int GDBPacketBufLen = 16384;
vector<BaseRemoteGDB *> debuggers;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14497
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Icb149f15a6c769371ebcb6ec5fbebc6170c31fc6
Gerrit-Change-Number: 14497
Gerrit-PatchSet: 1
Gerrit-Owner: Ciro Santilli <***@arm.com>
Gerrit-MessageType: newchange
https://gem5-review.googlesource.com/c/public/gem5/+/14497
Change subject: base: increase the GDB buffer size
......................................................................
base: increase the GDB buffer size
The size was not large enough for the 'G' packet on aarch64, which the
client sends to set registers.
This would lead to the stub not to be able to find the end of the input
packet and keep waiting forever.
Change-Id: Icb149f15a6c769371ebcb6ec5fbebc6170c31fc6
---
M src/base/remote_gdb.cc
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/base/remote_gdb.cc b/src/base/remote_gdb.cc
index 37ecef9..1257d5c 100644
--- a/src/base/remote_gdb.cc
+++ b/src/base/remote_gdb.cc
@@ -152,7 +152,9 @@
static const char GDBGoodP = '+';
static const char GDBBadP = '-';
-static const int GDBPacketBufLen = 1024;
+// This value was found by trial and error to be enough for
+// all tested packets.
+static const int GDBPacketBufLen = 16384;
vector<BaseRemoteGDB *> debuggers;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14497
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Icb149f15a6c769371ebcb6ec5fbebc6170c31fc6
Gerrit-Change-Number: 14497
Gerrit-PatchSet: 1
Gerrit-Owner: Ciro Santilli <***@arm.com>
Gerrit-MessageType: newchange