public inbox for ecos-discuss@sourceware.org
 help / color / mirror / Atom feed
From: "Øyvind Harboe" <oyvind.harboe@zylin.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: ecos-discuss@sources.redhat.com
Subject: Re: [ECOS] Problems when aborting tftp transfers
Date: Tue, 03 Aug 2004 09:56:00 -0000	[thread overview]
Message-ID: <1091526992.13266.22.camel@famine> (raw)
In-Reply-To: <20040803094913.GN14248@lunn.ch>

On Tue, 2004-08-03 at 11:49, Andrew Lunn wrote:
> On Tue, Aug 03, 2004 at 08:40:46AM +0200, ?yvind Harboe wrote:
> > I haven't got the setup available right now, but I think I remember the steps 
> > to reproduce a bit better:
> > 
> > 1. start tftp transfer
> > 2. abort tftp transfer
> > 3. at this point the new tftp server thread is waiting for timeout on
> > #1
> > 4. start new tftp transfer
> > 5. wait a bit(but not long enough for tftp transfer to start).
> > 6. abort tftp transfer
> > 7. at this point the tftp server is stuck in the main loop timing out on
> > WRQ requests. Each WRQ request requires a tftpd_write_file() to time out, so
> > this takes a loooong time.
> 
> This now make sense. It is the starting of the new connection before
> the old one times out that causes the delay. Since the system is
> single threaded it has two choices when the new connection request
> arrives. The current implementation simply buffers the request until
> later. The alternative implementation would be to discard the request
> and let the client retry. Both implementations are probably correct
> but have pros and cons. All these aborted connections is a corner case
> that should not happen very often so i would not worry about it.
> 
> Having said that, i think the real solution is to teach your users to
> have some patients and not abort the connections....

:-)

The problem is that the timeout is *very* long because it is number of 
queued WRQ (ca 5) * timeout for tftpd_write_file().

A single timeout is acceptable, but minutes is very confusing for the
user.

>         Andrew

-- 
Øyvind Harboe
http://www.zylin.com



--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

      reply	other threads:[~2004-08-03  9:56 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-21 10:52 Øyvind Harboe
2004-08-02 13:01 ` Andrew Lunn
2004-08-02 13:45   ` Øyvind Harboe
2004-08-02 14:02     ` Andrew Lunn
2004-08-02 14:14       ` Øyvind Harboe
2004-08-02 14:27         ` Andrew Lunn
2004-08-02 14:49           ` [ECOS] Question on wigglers, debugging Redboot in flash for the XScale target in single step mode Krishna Ganugapati
2004-08-02 14:59             ` Gary Thomas
2004-08-02 15:15               ` [ECOS] Question on wigglers, debugging Redboot in flash forthe " Krishna Ganugapati
2004-08-02 15:21                 ` Gary Thomas
2004-08-02 16:51                   ` [ECOS] Question on wigglers, debugging Redboot in flash fortheXScale " Krishna Ganugapati
2004-08-03 15:59                   ` Krishna Ganugapati
2004-08-03  6:40       ` [ECOS] Problems when aborting tftp transfers Øyvind Harboe
2004-08-03  9:49         ` Andrew Lunn
2004-08-03  9:56           ` Øyvind Harboe [this message]

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=1091526992.13266.22.camel@famine \
    --to=oyvind.harboe@zylin.com \
    --cc=andrew@lunn.ch \
    --cc=ecos-discuss@sources.redhat.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).