Message ID | 1456191325-1217-1-git-send-email-yangx.jy@cn.fujitsu.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Tue, Feb 23, 2016 at 09:35:25AM +0800, Xiao Yang wrote: > 'mke2fs 1.42.9 (28-Dec-2013)' produces as standard error when > we run generic/135 in ext4 file system. This is a regression > bug which bring in by commit: > 4a5cda8 xfstests: do not unmount tmpfs during remount > That patch remove '2>&1', so i bring that back. Thanks for the revision! But I noticed that Ted's patch fixed it too, "common: filter warning messages from mkfs.ext4" which filters the warnings in _scratch_mkfs_ext4, so that other mkfs errors can be noticed in the test. So I think it's a better fix :) Thanks, Eryu > > Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> > --- > tests/generic/135 | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tests/generic/135 b/tests/generic/135 > index 52418f6..5aa880d 100755 > --- a/tests/generic/135 > +++ b/tests/generic/135 > @@ -41,7 +41,7 @@ _supported_os Linux IRIX > > _require_odirect > _require_scratch > -_scratch_mkfs >/dev/null > +_scratch_mkfs &>/dev/null > _scratch_mount > > cd $SCRATCH_MNT > -- > 1.7.1 > > > > -- > To unsubscribe from this list: send the line "unsubscribe fstests" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Feb 23, 2016 at 11:26:52AM +0800, Eryu Guan wrote: > On Tue, Feb 23, 2016 at 09:35:25AM +0800, Xiao Yang wrote: > > 'mke2fs 1.42.9 (28-Dec-2013)' produces as standard error when > > we run generic/135 in ext4 file system. This is a regression > > bug which bring in by commit: > > 4a5cda8 xfstests: do not unmount tmpfs during remount > > That patch remove '2>&1', so i bring that back. > > Thanks for the revision! But I noticed that Ted's patch fixed it too, > > "common: filter warning messages from mkfs.ext4" > > which filters the warnings in _scratch_mkfs_ext4, so that other mkfs > errors can be noticed in the test. So I think it's a better fix :) Yes, I had revised patch that added the 2>&1 but that wasn't the one that ended up xfstests repository. I started with a patch to add it back that looked much like yours (except I used '2>&1' instead of '>&'), but I started thinking about the fact that this would cause errors to be masked which is not cool. I believe Dave or Cristoph has said that they don't like redirecting stderr to /dev/null for _scratch_mount, and it seems to me that assuming my change makes it in, it would probably be good to get rid of the other stderr redirections to /dev/null. - Ted -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
? 2016/02/23 11:26, Eryu Guan ??: > On Tue, Feb 23, 2016 at 09:35:25AM +0800, Xiao Yang wrote: >> 'mke2fs 1.42.9 (28-Dec-2013)' produces as standard error when >> we run generic/135 in ext4 file system. This is a regression >> bug which bring in by commit: >> 4a5cda8 xfstests: do not unmount tmpfs during remount >> That patch remove '2>&1', so i bring that back. > Thanks for the revision! But I noticed that Ted's patch fixed it too, > > "common: filter warning messages from mkfs.ext4" > > which filters the warnings in _scratch_mkfs_ext4, so that other mkfs > errors can be noticed in the test. So I think it's a better fix :) > > Thanks, > Eryu > Hi After applying Ted's pacth, 'mke2fs 1.42.9 (28-Dec-2013)' still produces when we run generic/135 in ext4 file system. I think that this is standard error instead of warning . >> Signed-off-by: Xiao Yang<yangx.jy@cn.fujitsu.com> >> --- >> tests/generic/135 | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/tests/generic/135 b/tests/generic/135 >> index 52418f6..5aa880d 100755 >> --- a/tests/generic/135 >> +++ b/tests/generic/135 >> @@ -41,7 +41,7 @@ _supported_os Linux IRIX >> >> _require_odirect >> _require_scratch >> -_scratch_mkfs>/dev/null >> +_scratch_mkfs&>/dev/null >> _scratch_mount >> >> cd $SCRATCH_MNT >> -- >> 1.7.1 >> >> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe fstests" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/tests/generic/135 b/tests/generic/135 index 52418f6..5aa880d 100755 --- a/tests/generic/135 +++ b/tests/generic/135 @@ -41,7 +41,7 @@ _supported_os Linux IRIX _require_odirect _require_scratch -_scratch_mkfs >/dev/null +_scratch_mkfs &>/dev/null _scratch_mount cd $SCRATCH_MNT
'mke2fs 1.42.9 (28-Dec-2013)' produces as standard error when we run generic/135 in ext4 file system. This is a regression bug which bring in by commit: 4a5cda8 xfstests: do not unmount tmpfs during remount That patch remove '2>&1', so i bring that back. Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> --- tests/generic/135 | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)