On 5/4/15 5:01 PM, Dave Chinner wrote: > From: Dave Chinner <dchinner@xxxxxxxxxx> > > On devices that have a logical sector smaller than physical sector, > this extra, harmless output now occurs: > > QA output created by 060 > +specified blocksize 1024 is less than device physical sector size 4096 > +switching to logical sector size 512 > Creating directory system to dump using src/fill. > Setup ....................................... > Dumping to files... > > And it causes lots of tests to fail unnecessarily. Filter it. > > Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx> Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx> But should we just remove that from mkfs output? It's probably my fault, but looking at it now, it seems like unnecessary noise. -Eric > --- > common/rc | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/common/rc b/common/rc > index 6ea107e..242dedb 100644 > --- a/common/rc > +++ b/common/rc > @@ -365,9 +365,12 @@ _scratch_mkfs_xfs() > mkfs_status=$? > fi > > - # output stored mkfs output > - cat $tmp_dir.mkfserr >&2 > + # output stored mkfs output, filtering unnecessary warnings from stderr > cat $tmp_dir.mkfsstd > + cat $tmp_dir.mkfserr | sed \ > + -e '/less than device physical sector/d' \ > + -e '/switching to logical sector/d' \ > + >&2 > rm -f $tmp_dir.mkfserr $tmp_dir.mkfsstd > > return $mkfs_status > -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
