On 02/28/2017 10:47 AM, Zhilong Liu wrote:
1. use 'Tab' as the code style.
2. arrange the testing steps and provide the 'main' entrance.
3. draft the log_save feature, it captures the /proc/mdstat,
md superblock info, bitmap info and the detail dmesg.
4. modified the mdadm() func, adding the operation that clear
the superblock when create or build one new array, and it
would exit testing when mdadm command returned non-0 value.
5. delete no_errors() func, it only used in tests/04update-uuid,
I recommend the new mdadm() using method.
6. delete fast_sync() func.
7. testdev(), add the object file checking, otherwise this command
would create one regular file, it's one trouble thing.
8. add dmesg checking in do_test() func, it's necessary to check
dmesg whether or not printed abnormal message.
9. add checking conditions in main(), such as $pwd/raid6check need
exists, here is a prompt to remind users to 'make everything'
before testing; the $targetdir should mount under ext[2-4] FS,
because the external bitmap only supports ext, the bmap() API
of bitmap.c doesn't exist in all filesystem, such as btrfs.
I like the improvement for the test, and I would suggest you split
those changes into smaller patches, make each patch do one thing,
it would be easier for Jes to review I think, and you still can merge
them into one finally if Jes prefer one patch with huge changes, :-) .
Cheers,
Guoqing
--
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
