public inbox for fstests@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] generic/47[23]: remove from auto/quick groups
@ 2018-02-23  5:01 Dave Chinner
  2018-11-28  7:12 ` Xiao Yang
  0 siblings, 1 reply; 2+ messages in thread
From: Dave Chinner @ 2018-02-23  5:01 UTC (permalink / raw)
  To: fstests

From: Dave Chinner <dchinner@redhat.com>

generic/472 is for changes that are not upstream and seem dead in
the water at the moment, so remove this test from the auto and quick
groups until it's been resolved upstream and the changes merged.

generic/473 really doesn't seme useful. FIEMAP is a debugging
interface, xfs_io is a debugging tool and so trying to make every
filesytem report exactly the same thing for a ranged query just
strikes me as the wrong thing to be doing. This fails on XFS, and
there's no apparent resolution to that in sight, so remove the test
from the auto and quick, too.

Signed-Off-By: Dave Chinner <dchinner@redhat.com>
---
 tests/generic/group | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/generic/group b/tests/generic/group
index b2ce7a66ce81..1e808865772f 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -474,8 +474,8 @@
 469 auto quick
 470 auto quick dax
 471 auto quick rw
-472 auto quick rw
-473 auto quick
+472 rw
+473 broken
 474 auto quick shutdown metadata
 475 shutdown auto log metadata
 476 auto rw
-- 
2.16.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] generic/47[23]: remove from auto/quick groups
  2018-02-23  5:01 [PATCH] generic/47[23]: remove from auto/quick groups Dave Chinner
@ 2018-11-28  7:12 ` Xiao Yang
  0 siblings, 0 replies; 2+ messages in thread
From: Xiao Yang @ 2018-11-28  7:12 UTC (permalink / raw)
  To: Dave Chinner, Eryu Guan; +Cc: fstests

On 2018/02/23 13:01, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> generic/472 is for changes that are not upstream and seem dead in
> the water at the moment, so remove this test from the auto and quick
> groups until it's been resolved upstream and the changes merged.
>
> generic/473 really doesn't seme useful. FIEMAP is a debugging
> interface, xfs_io is a debugging tool and so trying to make every
> filesytem report exactly the same thing for a ranged query just
> strikes me as the wrong thing to be doing. This fails on XFS, and
> there's no apparent resolution to that in sight, so remove the test
> from the auto and quick, too.
Hi Dave, Eryu

generic/473 with XFS got the mismatched extent info and was marked as a
broken test:
-------------------------------
Hole + Data
0: [0..127]: hole
-1: [128..255]: data
+1: [128..135]: data
-------------------------------

Is this a correct behavior? If not, is this issue going to be fixed in
the future?

Best Regards,
Xiao Yang
> Signed-Off-By: Dave Chinner <dchinner@redhat.com>
> ---
>  tests/generic/group | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/generic/group b/tests/generic/group
> index b2ce7a66ce81..1e808865772f 100644
> --- a/tests/generic/group
> +++ b/tests/generic/group
> @@ -474,8 +474,8 @@
>  469 auto quick
>  470 auto quick dax
>  471 auto quick rw
> -472 auto quick rw
> -473 auto quick
> +472 rw
> +473 broken
>  474 auto quick shutdown metadata
>  475 shutdown auto log metadata
>  476 auto rw

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-11-28 18:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-23  5:01 [PATCH] generic/47[23]: remove from auto/quick groups Dave Chinner
2018-11-28  7:12 ` Xiao Yang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox