On 06/07/2017 05:31 AM, Zhilong Liu wrote:
Hi, Jes; This patchset mainly focus on improving the 'test' script, rip it apart and put it together in a more coherent form. Over 3 months passed, the RFC patch of improving 'test' hadn't received any ideas, thus I post this sample version that I have tested many times. It's just the beginning to improve the test part of mdadm, any questions is very welcome. Sorry for that, I haven't added the function suggested from Guoqing, cut the words from the email tree of "ANNOUNCE: mdadm 4.0 - A tool for managing md Soft RAID under Linux": Guoqing asked: ... Is current test can run the test for different raid level, say, "./test --raidtype=raid1" could execute all the *r1* tests, does it make sense to do it if we don't support it now. ... Just list this suggestion here, waiting for ideas. RFC patch mail-tree: [PATCH RFC] test: revise 'test' and make it easier to understand Acked-by: Neil Brown And suggest to split those changing into smaller patches from Guoqing.
The whole test infrastructure is something I would like to see rewamped and modernized, so you starting to work on this is excellent.
With regard to your question, then having an option to say 'run all raid1 tests' is probably good, we should have that for all raid levels, imsm, and ddf.
I also would like to see a more flexible way to configure the tests, like it should be possible to specify a bunch of physical devices for the tests, instead of relying on loop devices.
Cheers, Jes -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html