Hi Yasuharu, On Wed, 14 Aug 2024 at 06:48, Yasuharu Shibata <[email protected]> wrote: > > After applying the following patch, wget test on sandbox failed[1]. > > Commit: cab7867cff ("net: wget: Support retransmission a dropped packet") > > Here are two reasons why the test is failed and how to fix it: > > 1. tcp_ack is calculated by the wrong value. > tcp_ack needs to be calculated by the received TCP payload size. > 2. wget command may have a problem that HTTP response from server > must be divided into more than two packets. > In this commit, HTTP response is divided into two packets. > > In addition, I fixed the HTTP response returned at the correct timing. > > [1] > https://lore.kernel.org/u-boot/caflszthebk2jr8oz6hj21wpsnjjgjhade037rqwhvwt1kjb...@mail.gmail.com/ > > Signed-off-by: Yasuharu Shibata <[email protected]> > Reported-by: Simon Glass <[email protected]> > --- > test/cmd/wget.c | 40 +++++++++++++++++++++++++++++++--------- > 1 file changed, 31 insertions(+), 9 deletions(-)
Reviewed-by: Simon Glass <[email protected]> Tested-by: Simon Glass <[email protected]> # sandbox Thanks for sorting this out. Regards, Simon
