On 1.3.2018 22:18, Stephen Warren wrote: > On 03/01/2018 04:48 AM, Liam Beguin wrote: >> On 1 March 2018 at 01:59, Michal Simek <michal.si...@xilinx.com> wrote: > ... >>> Also these randomized data/lengths were able to find out more issues >>> than fixed one. I was playing with that randomized test order and I see >>> that not all current tests are self contained and this will be good to >>> run too. >> >> I'll add options to randomize in v2. > > I'd strongly prefer the tests to be deterministic and not do random > different actions each time they're run. That way, any failures are much > easier to track down. Writing randomly generated data is fine, but not > random access patterns or sets of operations. > > Although I suppose you could implement two sets of tests; one that does > a static set of operations driven by a table in the board config file, > and one that does a more randomized set of operations controlled by > limits set by a table in the board config file.
The thing is that I have real experience with random tests which were able to find out issues in the driver compare to fixed tests. But I have not a problem with two set of tests. Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Xilinx Microblaze Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs
signature.asc
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot