Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro@codesourcery.com>
To: Paul Koning <Paul_Koning@dell.com>
Cc: sandra@codesourcery.com,  gdb@sourceware.org,
	 gdb-patches@sourceware.org
Subject: Re: [remote protocol] support for disabling packet acknowledgement
Date: Fri, 11 Jul 2008 15:11:00 -0000	[thread overview]
Message-ID: <200807111610.47304.pedro@codesourcery.com> (raw)
In-Reply-To: <18550.28000.759268.379468@gargle.gargle.HOWL>

[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


  parent reply	other threads:[~2008-07-11 15:11 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-10 18:59 Sandra Loosemore
2008-07-10 19:09 ` Paul Koning
2008-07-10 19:59   ` Sandra Loosemore
2008-07-10 20:13     ` Paul Koning
2008-07-10 22:33       ` Daniel Jacobowitz
2008-07-11  0:22         ` Sandra Loosemore
2008-07-11  0:43           ` Daniel Jacobowitz
2008-07-11 13:43         ` Paul Koning
2008-07-11 14:35           ` Daniel Jacobowitz
     [not found]       ` <20080710223312.GA19058__14539.8706700236$1215729298$gmane$org@caradoc.them.org>
2008-07-11  3:05         ` Frank Ch. Eigler
2008-07-11  3:25           ` Daniel Jacobowitz
2008-07-11 15:11       ` Pedro Alves [this message]
2008-07-11 15:24         ` Daniel Jacobowitz
2008-07-11 15:54           ` Paul Koning
2008-07-11 17:59             ` Pedro Alves
2008-07-11 18:54               ` Paul Koning
2008-07-11 19:10                 ` Pedro Alves
2008-07-25 13:41 ` Eli Zaretskii
2008-07-25 14:14   ` Sandra Loosemore
2008-07-26  5:54     ` Eli Zaretskii
  -- strict thread matches above, loose matches on Subject: below --
2008-07-10 17:23 Pedro Alves
2008-07-10 18:44 ` Paul Koning
2008-07-10 19:08   ` Daniel Jacobowitz
2008-07-25 13:39 ` Eli Zaretskii
2008-08-11 19:40 ` Daniel Jacobowitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200807111610.47304.pedro@codesourcery.com \
    --to=pedro@codesourcery.com \
    --cc=Paul_Koning@dell.com \
    --cc=gdb-patches@sourceware.org \
    --cc=gdb@sourceware.org \
    --cc=sandra@codesourcery.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox