labath added a comment.

In https://reviews.llvm.org/D42195#995087, @clayborg wrote:

> I am pretty sure that the ack that gets sent should happen in response to 
> getting an ACK. Note that LLDB send an ACK and then waits for an ACK. This 
> helps us see if anything is there. So yes, this code can always just respond 
> to an unsolicited ACK with an ACK.


LLDB sends an ACK, and then waits for anything. That "anything" can also be 
"nothing" (see `GDBRemoteCommunicationClient::HandshakeWithServer`). 
lldb-server currently does not respond to the initial ACK, which is good, 
because the length of the initial client wait (10ms) would mean this reply-ack 
would often get missed, and the connection would get out of sync (as this code 
demonstrated).

If we really want to have our servers send an initial ack, then we need to fix 
the client.


Repository:
  rL LLVM

https://reviews.llvm.org/D42195



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to