On Tue, May 18, 2021 at 12:28 PM Kinsey Moore <kinsey.mo...@oarcorp.com> wrote:
>
> The socket timeout is already configured just above the while loop for
> sending the bulk
> of the data and applies to all blocking operations. It defaults to 1s locally
> if not set.
>
ok, thanks, go ahead
> Kinsey
>
> -----Original Message-----
> From: Gedare Bloom <ged...@rtems.org>
> Sent: Tuesday, May 18, 2021 10:54
> To: Kinsey Moore <kinsey.mo...@oarcorp.com>
> Cc: devel@rtems.org
> Subject: Re: [PATCH] tester: Avoid TFTP race condition
>
> seems ok, but is there a way to set a timeout in case the client
> doesn't send a final ACK? Typical kind of network problem.
>
> On Mon, May 17, 2021 at 12:29 PM Kinsey Moore <kinsey.mo...@oarcorp.com>
> wrote:
> >
> > The u-boot TFTP client will fail a fully completed file retrieval if the
> > final ACK gets an ICMP response that the port is unavailable which can
> > occur if the port is closed before the ACK arrives from the client. This
> > change causes the TFTP server to wait for the final ACK before closing
> > the socket and prevents transfers from failing due to the race
> > condition.
> > ---
> > tester/rt/tftpserver.py | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tester/rt/tftpserver.py b/tester/rt/tftpserver.py
> > index d733301..46b594f 100644
> > --- a/tester/rt/tftpserver.py
> > +++ b/tester/rt/tftpserver.py
> > @@ -418,8 +418,6 @@ class udp_handler(socketserver.BaseRequestHandler):
> > ' < ' +
> > session.decode(client_ip, client_port,
> > response))
> > sock.sendto(response, (client_ip, client_port))
> > - if session.finished:
> > - break
> > try:
> > data, address = sock.recvfrom(2 + 2 +
> >
> > session.get_block_size())
> > @@ -438,6 +436,8 @@ class udp_handler(socketserver.BaseRequestHandler):
> > log.trace('] tftp: %d: receive: %s: error: %s'
> > \
> > % (index, client, serr))
> > return
> > + if session.finished:
> > + break
> > response = session.process(address[0], address[1],
> > data)
> > except error.general as gerr:
> > self._notice('] tftp: %dd: error: %s' % (index, gerr))
> > --
> > 2.20.1
> >
> > _______________________________________________
> > devel mailing list
> > devel@rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
