[PATCH v17 1/2] net: Add TCP protocol

[PaulLiu]

Hi Simon,

I think it is a bit hard for me to test it right now. It seems that we need
to setup a fake HTTP server?
It can be easily done by some python scripts but I'm not sure how to start.
Is there some example on testing the network commands? Like tftp or nfs?

Yours,
Paul


On Tue, Jul 12, 2022 at 7:00 PM Simon Glass <sjg at chromium.org> wrote:

> Hi Ying-Chun,
>
> On Fri, 8 Jul 2022 at 12:02, Ying-Chun Liu (PaulLiu)
> <paul.liu at linaro.org> wrote:
> >
> > From: "Ying-Chun Liu (PaulLiu)" <paul.liu at linaro.org>
> >
> > Currently file transfers are done using tftp or NFS both
> > over udp. This requires a request to be sent from client
> > (u-boot) to the boot server.
> >
> > The current standard is TCP with selective acknowledgment.
> >
> > Signed-off-by: Duncan Hare <DH at Synoia.com>
> > Signed-off-by: Duncan Hare <DuncanCHare at yahoo.com>
> > Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu at linaro.org>
> > Cc: Christian Gmeiner <christian.gmeiner at gmail.com>
> > Cc: Joe Hershberger <joe.hershberger at ni.com>
> > Cc: Michal Simek <michal.simek at xilinx.com>
> > Cc: Ramon Fried <rfried.dev at gmail.com>
> > ---
> > v1-v12: Made by Duncan, didn't tracked.
> > v13: Fix some issues which is reviewed by Christian
> > v14: Add options to enable/disable SACK.
> > v15: Fix various syntax errors reviewed by Michal.
> >      Remove magic numbers. Use kernel-doc format.
> > v16: Add more kernel-doc. Fix more double spaces.
> > ---
> >  include/net.h     |  36 ++-
> >  include/net/tcp.h | 312 ++++++++++++++++++++
> >  net/Kconfig       |  16 ++
> >  net/Makefile      |   1 +
> >  net/net.c         |  30 ++
> >  net/tcp.c         | 720 ++++++++++++++++++++++++++++++++++++++++++++++
> >  6 files changed, 1106 insertions(+), 9 deletions(-)
> >  create mode 100644 include/net/tcp.h
> >  create mode 100644 net/tcp.c
>
> This looks good to me.
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> Can we get a test for this? Perhaps a fake Ethernet driver in sandbox
> / drivers/net?
>