From: Zhilong Liu <zlliu@suse.com>
To: Jes.Sorensen@gmail.com
Cc: linux-raid@vger.kernel.org, Zhilong Liu <zlliu@suse.com>
Subject: [PATCH v2 0/3] Refactor and revamp the 'test' script
Date: Wed, 14 Jun 2017 21:02:50 +0800 [thread overview]
Message-ID: <1497445373-27985-1-git-send-email-zlliu@suse.com> (raw)
Hi, Jes;
Mainly added the suggestions from v1. I'm awared that these patches
maybe still to improve, draft them just from my understanding. I would
improve for any defect, and thanks for your patience to point out.
Still need ideas about testdev():
- Now the 'test' has supported new arguments like --disks= to test physical
devices, but testdev() in all cases was designed by loop mode, we should
decide whether or not the 'disk' mode also supports the testdev() testing.
Currently, I have ignored the testdev() testing and make it 'return 0' if
has set --dev=disk mode in this patch version.
For --raidtype:
- I have devided the raid levels to
raid0|linear|raid1|raid456|raid10|ddf|imsm
For --disks:
- It supports user to specify physical devices as follow.
Example: ./test --dev=disk --disks=/dev/sda{5..16} or
./test --dev=disk --disks=/dev/sd{a..m}
Changes from v1:
- Refactor and improve all functions of test 'script'.
- Add --raidtype argument to support testing different raid
level cases.
- Add --disks argument to support testing physical devices
v1 mail-tree:
[PATCH 0/3] mdadm/test: Refactor the codes of 'test' script
RFC patch mail-tree:
[PATCH RFC] test: revise 'test' and make it easier to understand
Thanks very much,
-Zhilong
Zhilong Liu (3):
mdadm/test: Refactor and revamp 'test' script
mdadm/test: Add '--raidtype=' to run different raidlevel cases
mdadm/test: Add '--disks=' to support testing phsical devices
test | 464 +++++++++++++++++++++++++++++++++++++++++--------------------------
1 file changed, 285 insertions(+), 179 deletions(-)
--
2.6.6
next reply other threads:[~2017-06-14 13:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-14 13:02 Zhilong Liu [this message]
2017-06-14 13:02 ` [PATCH v2 1/3] mdadm/test: Refactor and revamp 'test' script Zhilong Liu
2017-06-28 22:20 ` Anthony Youngman
2017-06-29 2:42 ` Zhilong Liu
2017-07-10 17:31 ` Jes Sorensen
2017-06-14 13:02 ` [PATCH v2 2/3] mdadm/test: Add '--raidtype=' to run different raidlevel cases Zhilong Liu
2017-07-10 17:35 ` Jes Sorensen
2017-06-14 13:02 ` [PATCH v2 3/3] mdadm/test: Add '--disks=' to support testing phsical devices Zhilong Liu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1497445373-27985-1-git-send-email-zlliu@suse.com \
--to=zlliu@suse.com \
--cc=Jes.Sorensen@gmail.com \
--cc=linux-raid@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.