Re: [PATCH] mkfs: don't warn about topology when mkfsing a file

xfs

Re: [PATCH] mkfs: don't warn about topology when mkfsing a file

To:	Eric Sandeen <sandeen@xxxxxxxxxxx>
Subject:	Re: [PATCH] mkfs: don't warn about topology when mkfsing a file
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Tue, 19 Jan 2010 04:09:54 -0500
Cc:	xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to:	<4B54B4D1.7020500@xxxxxxxxxxx>
References:	<4B54B4D1.7020500@xxxxxxxxxxx>
User-agent:	Mutt/1.5.19 (2009-01-05)

On Mon, Jan 18, 2010 at 01:21:53PM -0600, Eric Sandeen wrote:
> Some tests were failing for me like:
> 
>  QA output created by 206
>  === truncate file ===
>  === mkfs.xfs ===
> +warning: unable to probe device toplology for device /mnt/test/fsfile.21615
> ...
> 
> I thought about filtering the tests, but ...
> 
> It seems that we shouldn't even be trying to get topology for a
> plain file, should we?  And then we won't warn about failure.

Looks good to me.  Would be even nicer if libblkid handled that case,
but let's fix it where we can easily do it.


Signed-off-by: Christoph Hellwig <hch@xxxxxx>

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH] mkfs: don't warn about topology when mkfsing a file, Eric Sandeen Re: [PATCH] mkfs: don't warn about topology when mkfsing a file, Dave Chinner Re: [PATCH] mkfs: don't warn about topology when mkfsing a file, Christoph Hellwig <=

Previous by Date:	[PATCH 0/3] xfsqa: Fix various test-suite related failures, Dave Chinner
Next by Date:	Re: [PATCH] xfsteststs: get fsstress DIO working on non-xfs filesystems, Christoph Hellwig
Previous by Thread:	Re: [PATCH] mkfs: don't warn about topology when mkfsing a file, Dave Chinner
Next by Thread:	[PATCH] xfsteststs: get fsstress DIO working on non-xfs filesystems, Eric Sandeen
Indexes:	[Date] [Thread] [Top] [All Lists]