2013-04-15 11:09:47

by Dmitry Monakhov

[permalink] [raw]
Subject: [PATCH 1/2] xfstests: dump _require_fio output to log

This should helps to find incompatibility issues.

Signed-off-by: Dmitry Monakhov <[email protected]>
---
common/rc | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
mode change 100644 => 100755 tests/ext4/301

diff --git a/common/rc b/common/rc
index 09fb83f..ad3e378 100644
--- a/common/rc
+++ b/common/rc
@@ -1967,8 +1967,8 @@ _require_fio()
return 1;
fi

- $FIO_PROG --warnings-fatal --showcmd $job >/dev/null 2>&1
- [ $? -eq 0 ] || _notrun "$FIO_PROG too old"
+ $FIO_PROG --warnings-fatal --showcmd $job >> $seqres.full 2>&1
+ [ $? -eq 0 ] || _notrun "$FIO_PROG too old, see $seqres.full"
}

# Does freeze work on this fs?
diff --git a/tests/ext4/301 b/tests/ext4/301
old mode 100644
new mode 100755
--
1.7.1

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs


2013-04-15 11:09:48

by Dmitry Monakhov

[permalink] [raw]
Subject: [PATCH 2/2] xfstests: fix fio's arguments

Option 'group_reporting' semantics was changes for recent fio versions
In fact we do not need it here, let's just drop it.

Signed-off-by: Dmitry Monakhov <[email protected]>
---
tests/ext4/301 | 1 -
tests/ext4/302 | 1 -
tests/ext4/303 | 1 -
tests/ext4/304 | 1 -
4 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/tests/ext4/301 b/tests/ext4/301
index b6c2f11..e1583d4 100755
--- a/tests/ext4/301
+++ b/tests/ext4/301
@@ -59,7 +59,6 @@ filesize=${FILE_SIZE}
size=999G
buffered=0
fadvise_hint=0
-group_reporting

#################################
# Test1
diff --git a/tests/ext4/302 b/tests/ext4/302
index 9307ab4..abca805 100644
--- a/tests/ext4/302
+++ b/tests/ext4/302
@@ -59,7 +59,6 @@ filesize=${FILE_SIZE}
size=999G
buffered=0
fadvise_hint=0
-group_reporting

##########################################
# Test2
diff --git a/tests/ext4/303 b/tests/ext4/303
index 84d8f7b..7051576 100644
--- a/tests/ext4/303
+++ b/tests/ext4/303
@@ -59,7 +59,6 @@ filesize=${FILE_SIZE}
size=999G
buffered=0
fadvise_hint=0
-group_reporting

#################################
# Test3
diff --git a/tests/ext4/304 b/tests/ext4/304
index 7b90f1d..5d15ecb 100644
--- a/tests/ext4/304
+++ b/tests/ext4/304
@@ -60,7 +60,6 @@ filesize=${FILE_SIZE}
size=999G
buffered=0
fadvise_hint=0
-group_reporting

#################################
# Test4
--
1.7.1

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs

2013-04-22 15:19:03

by Rich Johnston

[permalink] [raw]
Subject: Re: [PATCH 1/2] xfstests: dump _require_fio output to log

Yes I agree this will help, looks good.

--Rich

Reviewed-by: Rich Johnston <[email protected]>

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs

2013-04-22 15:21:20

by Rich Johnston

[permalink] [raw]
Subject: Re: [PATCH 2/2] xfstests: fix fio's arguments

Looks good.

--Rich

Reviewed-by: Rich Johnston <[email protected]>


2013-04-22 15:30:50

by Rich Johnston

[permalink] [raw]
Subject: Re: [PATCH 1/2] xfstests: dump _require_fio output to log

On 04/15/2013 06:09 AM, Dmitry Monakhov wrote:

> diff --git a/tests/ext4/301 b/tests/ext4/301
> old mode 100644
> new mode 100755
>
Sorry I forgot to mention this change was unrelated to this patch.

tests/ext4/30? are changed to 755 in commit 'ca9fe94e'.

Thanks for this patch Dmitry, it has been committed.

--Rich


commit 6105b952f46cddf88029d14857cb2658cdab2d52
Author: Dmitry Monakhov <[email protected]>
Date: Mon Apr 15 11:09:47 2013 +0000

xfstests: dump _require_fio output to log

This should help find incompatibility issues.

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs

2013-04-22 15:31:26

by Rich Johnston

[permalink] [raw]
Subject: Re: [PATCH 2/2] xfstests: fix fio's arguments

Thanks for this patch Dmitry, it has been committed.

--Rich


commit ca9fe94e2d5fa4a8e405384ff5dd867b60a00864
Author: Dmitry Monakhov <[email protected]>
Date: Mon Apr 15 11:09:48 2013 +0000

xfstests: fix fio's arguments



_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs