From: Eryu Guan <guaneryu@gmail.com>
To: Omar Sandoval <osandov@osandov.com>
Cc: fstests@vger.kernel.org, kernel-team@fb.com, linux-btrfs@vger.kernel.org
Subject: Re: [PATCH 0/7] fstests: test Btrfs swapfile support
Date: Mon, 5 Nov 2018 00:09:31 +0800 [thread overview]
Message-ID: <20181104160931.GE12788@desktop> (raw)
In-Reply-To: <cover.1541193856.git.osandov@fb.com>
On Fri, Nov 02, 2018 at 02:29:35PM -0700, Omar Sandoval wrote:
> From: Omar Sandoval <osandov@fb.com>
>
> This series fixes a couple of generic swapfile tests and adds some
> Btrfs-specific swapfile tests. Btrfs swapfile support is scheduled for
> 4.21 [1].
>
> 1: https://www.spinics.net/lists/linux-btrfs/msg83454.html
>
> Thanks!
Thanks for the fixes and new tests!
>
> Omar Sandoval (7):
> generic/{472,496,497}: fix $seeqres typo
> generic/{472,496}: fix swap file creation on Btrfs
I've merged above two patches, they're two obvious bug fixes.
> btrfs: test swap file activation restrictions
> btrfs: test invalid operations on a swap file
> btrfs: test swap files on multiple devices
> btrfs: test device add/remove/replace with an active swap file
> btrfs: test balance and resize with an active swap file
These tests look fine to me, but it'd be really great if btrfs folks
could help review above tests and provide Reviewed-by tags.
And perhaps we could add test 17[56] to 'volume' group, as they do
device operations. And similarly, add the last test to 'balance' group.
Thanks,
Eryu
>
> tests/btrfs/173 | 55 ++++++++++++++++++++++++++++++
> tests/btrfs/173.out | 5 +++
> tests/btrfs/174 | 66 ++++++++++++++++++++++++++++++++++++
> tests/btrfs/174.out | 10 ++++++
> tests/btrfs/175 | 73 ++++++++++++++++++++++++++++++++++++++++
> tests/btrfs/175.out | 8 +++++
> tests/btrfs/176 | 82 +++++++++++++++++++++++++++++++++++++++++++++
> tests/btrfs/176.out | 5 +++
> tests/btrfs/177 | 64 +++++++++++++++++++++++++++++++++++
> tests/btrfs/177.out | 6 ++++
> tests/btrfs/group | 5 +++
> tests/generic/472 | 16 ++++-----
> tests/generic/496 | 8 ++---
> tests/generic/497 | 2 +-
> 14 files changed, 391 insertions(+), 14 deletions(-)
> create mode 100755 tests/btrfs/173
> create mode 100644 tests/btrfs/173.out
> create mode 100755 tests/btrfs/174
> create mode 100644 tests/btrfs/174.out
> create mode 100755 tests/btrfs/175
> create mode 100644 tests/btrfs/175.out
> create mode 100755 tests/btrfs/176
> create mode 100644 tests/btrfs/176.out
> create mode 100755 tests/btrfs/177
> create mode 100644 tests/btrfs/177.out
>
> --
> 2.19.1
>
next prev parent reply other threads:[~2018-11-04 16:09 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-02 21:29 [PATCH 0/7] fstests: test Btrfs swapfile support Omar Sandoval
2018-11-02 21:29 ` [PATCH 1/7] generic/{472,496,497}: fix $seeqres typo Omar Sandoval
2018-11-02 21:29 ` [PATCH 2/7] generic/{472,496}: fix swap file creation on Btrfs Omar Sandoval
2018-11-02 21:29 ` [PATCH 3/7] btrfs: test swap file activation restrictions Omar Sandoval
2018-11-02 21:29 ` [PATCH 4/7] btrfs: test invalid operations on a swap file Omar Sandoval
2018-11-02 21:29 ` [PATCH 5/7] btrfs: test swap files on multiple devices Omar Sandoval
2018-11-02 21:29 ` [PATCH 6/7] btrfs: test device add/remove/replace with an active swap file Omar Sandoval
2018-11-02 21:29 ` [PATCH 7/7] btrfs: test balance and resize " Omar Sandoval
2018-11-06 13:44 ` David Sterba
2018-11-04 16:09 ` Eryu Guan [this message]
2018-11-06 13:06 ` [PATCH 0/7] fstests: test Btrfs swapfile support David Sterba
2018-11-11 14:37 ` Eryu Guan
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=20181104160931.GE12788@desktop \
--to=guaneryu@gmail.com \
--cc=fstests@vger.kernel.org \
--cc=kernel-team@fb.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=osandov@osandov.com \
/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.