From: Simon Glass sjg@chromium.org
To: Duncan Hare dh@synoia.com Cc: "joe.hershberger@ni.com" joe.hershberger@ni.com; U-Boot Mailing List u-boot@lists.denx.de Sent: Sunday, May 13, 2018 3:00 PM Subject: Re: net: [U-Boot] [PATCH v10 3/3] Adding wget
Please setup a test that can run in an environment without the Internet. That is critical for unit tests.
Hand tests for Internet usage and the environmental effects are great, but that can't be what we include in the auto tests for repeat-ability reasons. Simon is asking for a separate type of test.
Is the test environment another patch in the series or an addition to the wget patch?
I suggest a separate patch.
Regards,>>Simon
Separate patch, patch 4 of the series or a completely new patch? Conceptual approach:
apt-get install nginx (in debian) I'll provide a config file pointing to test kernel a a specified file location.
Test kernel will be numbered lines of printable characters. Printable, not binary, easy to expand in a spreadsheet.
Can the u-boot print command print sections of memory? That is print $loadaddr $length?
That will work for small test kernels, or
I tested by using tftp to download a kernel, then used a modified wget to download a seconf time and verify thedownloads were identical.
compare $loadaddr1 $loadaddr2 $length
Is that a preferred approach? Duncan Hare
714 931 7952