net/tcp: improve tcp framework, use better state machine
authorMikhail Kshevetskiy <mikhail.kshevetskiy@iopsys.eu>
Sat, 28 Dec 2024 10:46:32 +0000 (13:46 +0300)
committerTom Rini <trini@konsulko.com>
Sat, 28 Dec 2024 17:59:42 +0000 (11:59 -0600)
commitbf962de97c375886fee0532f9d282a895d257ebb
treee7c275c42cd1f928cc21bea3c2fb62e1bc183368
parent82bf3aafa6378f1fdb3b9e3daca74db6aac38ed3
net/tcp: improve tcp framework, use better state machine

Changes:
 * Fix initial send sequence always zero issue
 * Use state machine close to RFC 9293. This should make TCP
   transfers more reliable (now we can upload a huge array
   of data from the board to external server)
 * Improve TCP framework a lot. This should make tcp client
   code much more simple.
 * rewrite wget with new tcp stack
 * rewrite fastboot_tcp with new tcp stack

It's quite hard to fix the initial send sequence (ISS) issue
with the separate patch. A naive attempt to fix an issue
inside the tcp_set_tcp_header() function will break tcp packet
retransmit logic in wget and other clients.

Example:
  Wget stores tcp_seq_num value before tcp_set_tcp_header() will
  be called and (on failure) retransmit the packet with the stored
  tcp_seq_num value. Thus:
    * the same ISS must allways be used (current case)
    * or tcp clients needs to generate a proper ISS when
      required.

A proper ISS fix will require a big redesing comparable with
a this one.

Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy@iopsys.eu>
Reviewed-by: Simon Glass <sjg@chromium.org>
include/net/tcp.h
include/net/wget.h
net/fastboot_tcp.c
net/net.c
net/tcp.c
net/wget.c