linux-f2fs-devel.lists.sourceforge.net archive mirror
 help / color / mirror / Atom feed
From: Jaegeuk Kim <jaegeuk@kernel.org>
To: Dave Chinner <david@fromorbit.com>
Cc: fstests@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net,
	Jaegeuk Kim <jaegeuk@kernel.org>
Subject: [PATCH 0/9 v5] make xfs/tests generic by introducing shutdown for filesystems
Date: Thu,  5 Feb 2015 22:37:04 -0800	[thread overview]
Message-ID: <1423204633-66673-1-git-send-email-jaegeuk@kernel.org> (raw)

Thanks to Dave. :)

Change log from v4:
 o add mkfs in _require_scratch_shutdown
 o add _require_fiemap and _extent_hole_counts
 o add _require_logstate
 o modify xfs/086,087 output rules for the generic usage
 o add _get_log_configs to introduce consistent testing options
 o add _get_quota_option to define quota option for each filesystems

This patch-set modifies several existing xfs's tests to be used by generic
filesystems.

1. FS requirement

In order to activate these testcases, filesystem should support a new feature,
shutdown, triggered by the following ioctl command.

#define FS_IOC_SHUTDOWN _IOR('X', 125, __u32)   /* Shutdown */

This ioctl needs one of three modes passed by a flag according to the syncing
policy.

#define FS_GOING_DOWN_FULLSYNC  0x0     /* going down with full sync */
#define FS_GOING_DOWN_METASYNC  0x1     /* going down with metadata */
#define FS_GOING_DOWN_NOSYNC    0x2     /* going down */

Whenever shutdown is requested with one of the above mode, filesystem must
stop any IOs, and wait for unmount, which is very useful to test power-off-
recovery.

2. Changes in xfstests

In this patch set, common/rc adds macros:
 o _require_scratch_shutdown : check whether FS supports shutdown
 o _require_fiemap           : check bmap tool is defined
 o _require_norecovery       : check filesystem supports norecovery option.
 o _require_logstate         : check logstate tools is defined
 o _get_log_configs          : define mount options for tests
 o _get_quota_option         : define quota options for filesystems

 The following tests become generic.
  xfs/053 -> generic/042
  xfs/137 -> generic/043
  xfs/138 -> generic/044
  xfs/139 -> generic/045
  xfs/140 -> generic/046
  xfs/179 -> generic/047
  xfs/180 -> generic/048
  xfs/182 -> generic/049
  xfs/200 -> generic/050
  xfs/306 -> generic/051

  xfs/085 -> generic/052
  xfs/086 -> generic/054
  xfs/087 -> generic/055

Jaegeuk Kim (9):
  common/rc: add _require_scratch_shtudown
  common/rc: add _require_fiemap and _extent_hole_counts
  common/rc: add _require_norecovery
  tests/generic: relocate xfs's tests into tests/generic/
  common/log: define _require_logstate
  xfs/086,087: remove specific testing options in output
  common/log: add _get_log_configs for testing options
  common/quota: give quota mount option per filesystem
  tests/generic: relocate xfs's tests into tests/generic/

 common/config                          |  1 +
 common/log                             | 98 +++++++++++++++++++++++++++++++---
 common/quota                           | 15 ++++++
 common/rc                              | 33 ++++++++++++
 tests/{xfs/053 => generic/042}         |  5 +-
 tests/{xfs/053.out => generic/042.out} |  2 +-
 tests/{xfs/137 => generic/043}         | 19 ++++---
 tests/generic/043.out                  |  1 +
 tests/{xfs/138 => generic/044}         | 19 ++++---
 tests/generic/044.out                  |  1 +
 tests/{xfs/139 => generic/045}         | 19 ++++---
 tests/generic/045.out                  |  1 +
 tests/{xfs/140 => generic/046}         | 15 +++---
 tests/generic/046.out                  |  1 +
 tests/{xfs/179 => generic/047}         | 13 +++--
 tests/generic/047.out                  |  1 +
 tests/{xfs/180 => generic/048}         | 13 +++--
 tests/generic/048.out                  |  1 +
 tests/{xfs/182 => generic/049}         | 13 +++--
 tests/generic/049.out                  |  1 +
 tests/{xfs/200 => generic/050}         | 10 ++--
 tests/{xfs/200.out => generic/050.out} |  2 +-
 tests/{xfs/306 => generic/051}         |  5 +-
 tests/{xfs/306.out => generic/051.out} |  2 +-
 tests/{xfs/085 => generic/052}         |  8 +--
 tests/{xfs/085.out => generic/052.out} |  2 +-
 tests/{xfs/086 => generic/054}         | 36 +++++--------
 tests/{xfs/086.out => generic/054.out} | 22 +-------
 tests/{xfs/087 => generic/055}         | 40 ++++++--------
 tests/{xfs/087.out => generic/055.out} | 53 ++++++++++++++----
 tests/generic/group                    | 13 +++++
 tests/xfs/137.out                      |  1 -
 tests/xfs/138.out                      |  1 -
 tests/xfs/139.out                      |  1 -
 tests/xfs/140.out                      |  1 -
 tests/xfs/179.out                      |  1 -
 tests/xfs/180.out                      |  1 -
 tests/xfs/182.out                      |  1 -
 tests/xfs/group                        | 13 -----
 39 files changed, 319 insertions(+), 166 deletions(-)
 rename tests/{xfs/053 => generic/042} (97%)
 rename tests/{xfs/053.out => generic/042.out} (92%)
 rename tests/{xfs/137 => generic/043} (87%)
 create mode 100644 tests/generic/043.out
 rename tests/{xfs/138 => generic/044} (87%)
 create mode 100644 tests/generic/044.out
 rename tests/{xfs/139 => generic/045} (87%)
 create mode 100644 tests/generic/045.out
 rename tests/{xfs/140 => generic/046} (90%)
 create mode 100644 tests/generic/046.out
 rename tests/{xfs/179 => generic/047} (90%)
 create mode 100644 tests/generic/047.out
 rename tests/{xfs/180 => generic/048} (91%)
 create mode 100644 tests/generic/048.out
 rename tests/{xfs/182 => generic/049} (90%)
 create mode 100644 tests/generic/049.out
 rename tests/{xfs/200 => generic/050} (95%)
 rename tests/{xfs/200.out => generic/050.out} (97%)
 rename tests/{xfs/306 => generic/051} (97%)
 rename tests/{xfs/306.out => generic/051.out} (60%)
 rename tests/{xfs/085 => generic/052} (94%)
 rename tests/{xfs/085.out => generic/052.out} (94%)
 rename tests/{xfs/086 => generic/054} (81%)
 rename tests/{xfs/086.out => generic/054.out} (93%)
 rename tests/{xfs/087 => generic/055} (79%)
 rename tests/{xfs/087.out => generic/055.out} (89%)
 delete mode 100644 tests/xfs/137.out
 delete mode 100644 tests/xfs/138.out
 delete mode 100644 tests/xfs/139.out
 delete mode 100644 tests/xfs/140.out
 delete mode 100644 tests/xfs/179.out
 delete mode 100644 tests/xfs/180.out
 delete mode 100644 tests/xfs/182.out

-- 
2.1.1


             reply	other threads:[~2015-02-06  6:37 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-06  6:37 Jaegeuk Kim [this message]
2015-02-06  6:37 ` [PATCH 1/9 v5] common/rc: add _require_scratch_shtudown Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 2/9 v5] common/rc: add _require_fiemap and _extent_hole_counts Jaegeuk Kim
2015-02-10  5:47   ` Dave Chinner
2015-02-10 21:52     ` [PATCH 2/9 v6] common/rc: add _require_fiemap and counting extents and holes Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 3/9 v5] common/rc: add _require_norecovery Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 4/9 v5] tests/generic: relocate xfs's tests into tests/generic/ Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 5/9 v5] common/log: define _require_logstate Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 6/9 v5] xfs/086,087: remove specific testing options in output Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 7/9 v5] common/log: add _get_log_configs for testing options Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 8/9 v5] common/quota: give quota mount option per filesystem Jaegeuk Kim
2015-02-10  6:01   ` Dave Chinner
2015-02-10 21:54     ` [PATCH 8/9 v6] xfs/087: " Jaegeuk Kim
2015-02-06  6:37 ` [PATCH 9/9 v5] tests/generic: relocate xfs's tests into tests/generic/ Jaegeuk Kim
2015-02-10  6:04 ` [PATCH 0/9 v5] make xfs/tests generic by introducing shutdown for filesystems Dave Chinner
2015-02-10 21:55   ` Jaegeuk Kim

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=1423204633-66673-1-git-send-email-jaegeuk@kernel.org \
    --to=jaegeuk@kernel.org \
    --cc=david@fromorbit.com \
    --cc=fstests@vger.kernel.org \
    --cc=linux-f2fs-devel@lists.sourceforge.net \
    /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).