From: Simon Glass <[email protected]>
To: Duncan Hare <[email protected]> Cc: "[email protected]" <[email protected]>; U-Boot Mailing List <[email protected]> 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 _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
