linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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


             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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).