[Lldb-commits] [PATCH] D17099: Add reverse-connect functionality to the gdb-remote command

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Feb 11 01:43:27 PST 2016

labath added a subscriber: labath.
labath added a comment.

I think this would be better if you could specify the connection direction in the url. Also, a test for this functionality would be great.

Comment at: source/Interpreter/CommandInterpreter.cpp:638
@@ +637,3 @@
+            connect_gdb_remote_cmd_ap->AddRegexCommand("^([^:]+:[[:digit:]]+) reverse$",
+                                                       "process connect --plugin gdb-remote connect://%1 reverse") &&
+            connect_gdb_remote_cmd_ap->AddRegexCommand("^([[:digit:]]+) reverse$",
how about just using `listen://...` to mean reverse connection?

Comment at: source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.h:222
@@ -226,1 +221,3 @@
+    ConnectProcess(const char *connect_url, const char *plugin_name, lldb_private::Debugger &debugger,
+                   lldb_private::Target *target, lldb_private::Error &error, bool reverse = false) override;
I don't think we need to pass this `reverse=false` around everywhere. We already support `listen://host:port` url scheme (see e.g. ConnectionFileDescriptor::Connect`). Could you use that to specify connection direction?


More information about the lldb-commits mailing list