Re: [PATCH v2 1/3] mdadm/test: Refactor and revamp 'test' script

[Jes Sorensen <jes.sorensen@gmail.com>]

On 06/14/2017 09:02 AM, Zhilong Liu wrote:
> Adding functions:
> die()
>    uniform the abnormal situations that have to abort.
> check_env()
>    do various basic checking before running test suite.
> save_log()
>    collect array infos, include of dmesg, superblock,
>    bitmap and /proc/mdstat.
> main()
>    the core function of this script.
> 
> Improve functions:
> cleanup()
>    clear dmesg and remove the /vat/tmp/mdtest* files.
> mdadm()
>    clear superblock once creating or building arrays
>    every time, because it's always creating arrays
>    many times in a test case.
> check()
>    just tidy up with die(), didn't change code meanings.
> testdev()
>    add checking $1 must be a block device, add 'return 0'
>    in final because this function exists in last line of
>    test case, such as tests/05r6tor0.
> do_test()
>    add checking abnormal dmesg and changing log management.
> do_help()
>    just recommend a better way to print Usage.
> parse_args()
>    revamp and improve.
> 
> Delete function:
> fast_sync()
>    It's no longer used, so get rid of it.
> 
> Signed-off-by: Zhilong Liu <zlliu@suse.com>

Applied!

I fixed the s/vat/var/ issue.

Thanks,
Jes