On Sun, Aug 11, 2024 at 08:50:18AM -0600, Simon Glass wrote:
> +Ying-Chun Liu (PaulLiu) <paul....@linaro.org>
> 
> Hi Yasuharu,
> 
> On Sat, 10 Aug 2024 at 20:48, Yasuharu Shibata
> <yasuharu.shib...@gmail.com> wrote:
> >
> > Dear Simon,
> >
> > Could you inform me how the wget test was broken?
> > As I explained in the commit log, I fixed the bug in specific conditions.
> > Without the details of how the break happened,
> > it is difficult to find a proper patch.
> >
> > Best regards,
> > Yasuharu Shibata
> 
> Yes, I should have mentioned that. It is test/cmd/wget.c - see also [1].
> 
> This is a good run:
> Test: net_test_wget: wget.c
> HTTP/1.1 200 OK
> Packets received 5, Transfer Successful
> Bytes transferred = 32 (20 hex)
> md5 for 00020000 ... 0002001f ==> 234af48e94b0085060249ecb5942ab57
> Failures: 0
> 
> With your patch it hangs:
> 
> Test: net_test_wget: wget.c
> HTTP/1.1 200 OKT

Can you please figure out what's going on with the sandbox test then?
The change in question fixes wget support on real hardware, and also the
other wget tests (as part of the lwIP series) work fine as well. This
seems like something specific to that test.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to