* [PATCH] xfstest: an regression test for btrfs balance
@ 2013-10-08 1:21 Gui Hecheng
2013-10-08 17:21 ` Josef Bacik
2013-10-16 20:44 ` Rich Johnston
0 siblings, 2 replies; 4+ messages in thread
From: Gui Hecheng @ 2013-10-08 1:21 UTC (permalink / raw)
To: xfs; +Cc: linux-btrfs, miaox, jbacik, Gui Hecheng
Originally, when executing "btrfs balance" right after
"btrfs subvolume snaphot" & "btrfs subvolume delete",
a kernel BUG arises.
This problem is caused by the patch:
[PATCH 1/2] Btrfs: fix for patch "cleanup: don't check
the same thing twice"
The commit id: 48475471728f060bfd2e686f592ef208d3ba8b7d
(in kernel/git/torvalds/linux.git)
handled by the patch:
[PATCH 2/3] Btrfs: fix oops caused by the space balance
and dead roots
Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
---
tests/btrfs/319 | 71 +++++++++++++++++++++++++++++++++++++++++++++++++++++
tests/btrfs/319.out | 2 ++
tests/btrfs/group | 1 +
3 files changed, 74 insertions(+)
create mode 100755 tests/btrfs/319
create mode 100644 tests/btrfs/319.out
diff --git a/tests/btrfs/319 b/tests/btrfs/319
new file mode 100755
index 0000000..b49e760
--- /dev/null
+++ b/tests/btrfs/319
@@ -0,0 +1,71 @@
+#! /bin/bash
+# FS QA Test No. btrfs/319
+#
+# Btrfs balance tests
+#
+#-----------------------------------------------------------------------
+# Copyright (c) 2013 Fujitsu. All Rights Reserved.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#-----------------------------------------------------------------------
+#
+
+seq=`basename $0`
+seqres=$RESULT_DIR/$seq
+echo "QA output created by $seq"
+
+here="`pwd`"
+tmp=/tmp/$$
+status=1 # failure is the default!
+
+_create_snapshot()
+{
+ for i in $(seq 20)
+ do
+ btrfs sub snapshot $SCRATCH_MNT $SCRATCH_MNT/snapshot0 >/dev/null
+ btrfs sub delete $SCRATCH_MNT/snapshot0 >/dev/null
+ done
+}
+
+_balance()
+{
+ for i in $(seq 20)
+ do
+ btrfs balance start $SCRATCH_MNT >/dev/null
+ done
+}
+
+#get standard environment, filters and checks
+. ./common/rc
+. ./common/filter
+
+# real QA test starts here
+_supported_fs btrfs
+_supported_os Linux
+
+_setup_testdir
+_require_scratch
+_scratch_mkfs >/dev/null 2>&1
+_scratch_mount
+
+echo "Tip: Run dmesg to see the error on failure"
+
+_create_snapshot &
+
+_balance &
+
+wait
+
+status=0
+exit
diff --git a/tests/btrfs/319.out b/tests/btrfs/319.out
new file mode 100644
index 0000000..1d21523
--- /dev/null
+++ b/tests/btrfs/319.out
@@ -0,0 +1,2 @@
+QA output created by 319
+Tip: Run dmesg to see the error on failure
diff --git a/tests/btrfs/group b/tests/btrfs/group
index d2fe030..201a475 100644
--- a/tests/btrfs/group
+++ b/tests/btrfs/group
@@ -14,3 +14,4 @@
009 auto quick
010 auto quick
011 auto
+319 auto
--
1.8.0.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] xfstest: an regression test for btrfs balance
2013-10-08 1:21 [PATCH] xfstest: an regression test for btrfs balance Gui Hecheng
@ 2013-10-08 17:21 ` Josef Bacik
2013-10-09 2:13 ` Gui Hecheng
2013-10-16 20:44 ` Rich Johnston
1 sibling, 1 reply; 4+ messages in thread
From: Josef Bacik @ 2013-10-08 17:21 UTC (permalink / raw)
To: Gui Hecheng; +Cc: xfs, linux-btrfs, miaox, jbacik
On Tue, Oct 08, 2013 at 09:21:15AM +0800, Gui Hecheng wrote:
> Originally, when executing "btrfs balance" right after
> "btrfs subvolume snaphot" & "btrfs subvolume delete",
> a kernel BUG arises.
>
> This problem is caused by the patch:
> [PATCH 1/2] Btrfs: fix for patch "cleanup: don't check
> the same thing twice"
> The commit id: 48475471728f060bfd2e686f592ef208d3ba8b7d
> (in kernel/git/torvalds/linux.git)
>
> handled by the patch:
> [PATCH 2/3] Btrfs: fix oops caused by the space balance
> and dead roots
>
> Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
> ---
> tests/btrfs/319 | 71 +++++++++++++++++++++++++++++++++++++++++++++++++++++
> tests/btrfs/319.out | 2 ++
> tests/btrfs/group | 1 +
> 3 files changed, 74 insertions(+)
> create mode 100755 tests/btrfs/319
> create mode 100644 tests/btrfs/319.out
>
> diff --git a/tests/btrfs/319 b/tests/btrfs/319
> new file mode 100755
> index 0000000..b49e760
> --- /dev/null
> +++ b/tests/btrfs/319
> @@ -0,0 +1,71 @@
> +#! /bin/bash
> +# FS QA Test No. btrfs/319
> +#
Just pick the next available number, so 012, no reason to jack up the numbers
like this.
> +# Btrfs balance tests
> +#
> +#-----------------------------------------------------------------------
> +# Copyright (c) 2013 Fujitsu. All Rights Reserved.
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License as
> +# published by the Free Software Foundation.
> +#
> +# This program is distributed in the hope that it would be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write the Free Software Foundation,
> +# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
> +#-----------------------------------------------------------------------
> +#
> +
> +seq=`basename $0`
> +seqres=$RESULT_DIR/$seq
> +echo "QA output created by $seq"
> +
> +here="`pwd`"
> +tmp=/tmp/$$
> +status=1 # failure is the default!
> +
> +_create_snapshot()
> +{
> + for i in $(seq 20)
> + do
> + btrfs sub snapshot $SCRATCH_MNT $SCRATCH_MNT/snapshot0 >/dev/null
> + btrfs sub delete $SCRATCH_MNT/snapshot0 >/dev/null
> + done
> +}
> +
> +_balance()
> +{
> + for i in $(seq 20)
> + do
> + btrfs balance start $SCRATCH_MNT >/dev/null
> + done
> +}
> +
> +#get standard environment, filters and checks
> +. ./common/rc
> +. ./common/filter
> +
> +# real QA test starts here
> +_supported_fs btrfs
> +_supported_os Linux
> +
> +_setup_testdir
> +_require_scratch
> +_scratch_mkfs >/dev/null 2>&1
> +_scratch_mount
> +
> +echo "Tip: Run dmesg to see the error on failure"
> +
> +_create_snapshot &
> +
> +_balance &
> +
> +wait
> +
> +status=0
> +exit
> diff --git a/tests/btrfs/319.out b/tests/btrfs/319.out
> new file mode 100644
> index 0000000..1d21523
> --- /dev/null
> +++ b/tests/btrfs/319.out
> @@ -0,0 +1,2 @@
> +QA output created by 319
> +Tip: Run dmesg to see the error on failure
> diff --git a/tests/btrfs/group b/tests/btrfs/group
> index d2fe030..201a475 100644
> --- a/tests/btrfs/group
> +++ b/tests/btrfs/group
> @@ -14,3 +14,4 @@
> 009 auto quick
> 010 auto quick
> 011 auto
> +319 auto
> --
> 1.8.0.1
Otherwise this looks nice and straightforward, did you verify that it made the
box panic without Miao's fixes? If you did just fixup the test number and you
can add
Reviewed-by: Josef Bacik <jbacik@fusionio.com>
Thanks,
Josef
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] xfstest: an regression test for btrfs balance
2013-10-08 17:21 ` Josef Bacik
@ 2013-10-09 2:13 ` Gui Hecheng
0 siblings, 0 replies; 4+ messages in thread
From: Gui Hecheng @ 2013-10-09 2:13 UTC (permalink / raw)
To: Josef Bacik; +Cc: xfs, linux-btrfs, miaox
On Tue, 2013-10-08 at 13:21 -0400, Josef Bacik wrote:
> On Tue, Oct 08, 2013 at 09:21:15AM +0800, Gui Hecheng wrote:
> > Originally, when executing "btrfs balance" right after
> > "btrfs subvolume snaphot" & "btrfs subvolume delete",
> > a kernel BUG arises.
> >
> > This problem is caused by the patch:
> > [PATCH 1/2] Btrfs: fix for patch "cleanup: don't check
> > the same thing twice"
> > The commit id: 48475471728f060bfd2e686f592ef208d3ba8b7d
> > (in kernel/git/torvalds/linux.git)
> >
> > handled by the patch:
> > [PATCH 2/3] Btrfs: fix oops caused by the space balance
> > and dead roots
> >
> > Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
> > ---
> > tests/btrfs/319 | 71 +++++++++++++++++++++++++++++++++++++++++++++++++++++
> > tests/btrfs/319.out | 2 ++
> > tests/btrfs/group | 1 +
> > 3 files changed, 74 insertions(+)
> > create mode 100755 tests/btrfs/319
> > create mode 100644 tests/btrfs/319.out
> >
> > diff --git a/tests/btrfs/319 b/tests/btrfs/319
> > new file mode 100755
> > index 0000000..b49e760
> > --- /dev/null
> > +++ b/tests/btrfs/319
> > @@ -0,0 +1,71 @@
> > +#! /bin/bash
> > +# FS QA Test No. btrfs/319
> > +#
>
> Just pick the next available number, so 012, no reason to jack up the numbers
> like this.
Test number has been changed 319 -> 012 in V2.
> > +# Btrfs balance tests
> > +#
> > +#-----------------------------------------------------------------------
> > +# Copyright (c) 2013 Fujitsu. All Rights Reserved.
> > +#
> > +# This program is free software; you can redistribute it and/or
> > +# modify it under the terms of the GNU General Public License as
> > +# published by the Free Software Foundation.
> > +#
> > +# This program is distributed in the hope that it would be useful,
> > +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > +# GNU General Public License for more details.
> > +#
> > +# You should have received a copy of the GNU General Public License
> > +# along with this program; if not, write the Free Software Foundation,
> > +# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
> > +#-----------------------------------------------------------------------
> > +#
> > +
> > +seq=`basename $0`
> > +seqres=$RESULT_DIR/$seq
> > +echo "QA output created by $seq"
> > +
> > +here="`pwd`"
> > +tmp=/tmp/$$
> > +status=1 # failure is the default!
> > +
> > +_create_snapshot()
> > +{
> > + for i in $(seq 20)
> > + do
> > + btrfs sub snapshot $SCRATCH_MNT $SCRATCH_MNT/snapshot0 >/dev/null
> > + btrfs sub delete $SCRATCH_MNT/snapshot0 >/dev/null
> > + done
> > +}
> > +
> > +_balance()
> > +{
> > + for i in $(seq 20)
> > + do
> > + btrfs balance start $SCRATCH_MNT >/dev/null
> > + done
> > +}
> > +
> > +#get standard environment, filters and checks
> > +. ./common/rc
> > +. ./common/filter
> > +
> > +# real QA test starts here
> > +_supported_fs btrfs
> > +_supported_os Linux
> > +
> > +_setup_testdir
> > +_require_scratch
> > +_scratch_mkfs >/dev/null 2>&1
> > +_scratch_mount
> > +
> > +echo "Tip: Run dmesg to see the error on failure"
> > +
> > +_create_snapshot &
> > +
> > +_balance &
> > +
> > +wait
> > +
> > +status=0
> > +exit
> > diff --git a/tests/btrfs/319.out b/tests/btrfs/319.out
> > new file mode 100644
> > index 0000000..1d21523
> > --- /dev/null
> > +++ b/tests/btrfs/319.out
> > @@ -0,0 +1,2 @@
> > +QA output created by 319
> > +Tip: Run dmesg to see the error on failure
> > diff --git a/tests/btrfs/group b/tests/btrfs/group
> > index d2fe030..201a475 100644
> > --- a/tests/btrfs/group
> > +++ b/tests/btrfs/group
> > @@ -14,3 +14,4 @@
> > 009 auto quick
> > 010 auto quick
> > 011 auto
> > +319 auto
> > --
> > 1.8.0.1
>
> Otherwise this looks nice and straightforward, did you verify that it made the
> box panic without Miao's fixes? If you did just fixup the test number and you
> can add
>
> Reviewed-by: Josef Bacik <jbacik@fusionio.com>
>
> Thanks,
>
> Josef
> --
I have verify that it did make the box panic without Miao's fixes.
Thank you for your comments.
Gui
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfstest: an regression test for btrfs balance
2013-10-08 1:21 [PATCH] xfstest: an regression test for btrfs balance Gui Hecheng
2013-10-08 17:21 ` Josef Bacik
@ 2013-10-16 20:44 ` Rich Johnston
1 sibling, 0 replies; 4+ messages in thread
From: Rich Johnston @ 2013-10-16 20:44 UTC (permalink / raw)
To: Gui Hecheng, xfs; +Cc: jbacik, miaox, linux-btrfs
This patch has been committed.
Thanks
--Rich
NOTE: Test numbers are adjusted at commit time.
commit 2268142f6f036cdc257a7c031c511043804b5c63
Author: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
Date: Wed Oct 9 02:16:01 2013 +0000
xfstests: an regression test for btrfs balance
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-10-16 20:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-08 1:21 [PATCH] xfstest: an regression test for btrfs balance Gui Hecheng
2013-10-08 17:21 ` Josef Bacik
2013-10-09 2:13 ` Gui Hecheng
2013-10-16 20:44 ` Rich Johnston
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).