From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20259 invoked by alias); 11 Jul 2008 15:11:15 -0000 Received: (qmail 20247 invoked by uid 22791); 11 Jul 2008 15:11:13 -0000 X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 11 Jul 2008 15:10:50 +0000 Received: (qmail 19339 invoked from network); 11 Jul 2008 15:10:47 -0000 Received: from unknown (HELO orlando.local) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 11 Jul 2008 15:10:47 -0000 From: Pedro Alves To: Paul Koning Subject: Re: [remote protocol] support for disabling packet acknowledgement Date: Fri, 11 Jul 2008 15:11:00 -0000 User-Agent: KMail/1.9.9 Cc: sandra@codesourcery.com, gdb@sourceware.org, gdb-patches@sourceware.org References: <48765B8A.6080805@codesourcery.com> <48766999.6070001@codesourcery.com> <18550.28000.759268.379468@gargle.gargle.HOWL> In-Reply-To: <18550.28000.759268.379468@gargle.gargle.HOWL> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200807111610.47304.pedro@codesourcery.com> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-07/txt/msg00213.txt.bz2 [so, comming back to the original functionality] On Thursday 10 July 2008 21:13:20, Paul Koning wrote: > By the way, note that "target remote host:port" does NOT mean that you > have TCP end to end. You may have TCP to a terminal server and raw > UART the rest of the way, so you still need ACKs (of one kind or > another). Right, that's why we made it so that only if the stub reports qSupported:QStartNoAck+, GDB will turn it on by default. If you notice the gdbserver patch, it will only report that, if the connection is coming from TCP. It is my understanding that situations like these are not usually seen: GDB UART <-> something <-> TCP/IP stub Would it make people happier if GDB would only enable noack mode *automatically* if: - The "set remote noack-packet" setting is set to auto - The stub reported support and *preference* to it with qSupported (it knows it's using a reliable connection on its end) - GDB also knows it is using a reliable connection (tcp/pipe) That last bit was there until the last version of the patch before submission, but I ended up removing it from the final patch. We can always go with or without that bit for now, and change it later, as the difference is all on GDB's side. -- Pedro Alves