public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* xfs_check
@ 2008-05-27 16:26 Christoph Hellwig
  2008-05-27 16:48 ` xfs_check Eric Sandeen
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Christoph Hellwig @ 2008-05-27 16:26 UTC (permalink / raw)
  To: xfs

In the past we had quite a few cases where we told people to run
xfs_repair -n instead of xfs_check.  I think that makes a lot of sense
because xfs_repair -n generally gives output at least as useful as
xfs_check if not more so and also is a lot faster.  Is there any reason
why we shouldn't simply kill xfs_check and replaced it with a wrapper
around xfs_repair?

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

* Re: xfs_check
  2008-05-27 16:26 xfs_check Christoph Hellwig
@ 2008-05-27 16:48 ` Eric Sandeen
  2008-05-27 16:50   ` xfs_check Chris Wedgwood
  2008-05-27 23:21 ` xfs_check Dave Chinner
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 8+ messages in thread
From: Eric Sandeen @ 2008-05-27 16:48 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: xfs

Christoph Hellwig wrote:
> In the past we had quite a few cases where we told people to run
> xfs_repair -n instead of xfs_check.  I think that makes a lot of sense
> because xfs_repair -n generally gives output at least as useful as
> xfs_check if not more so and also is a lot faster.  Is there any reason
> why we shouldn't simply kill xfs_check and replaced it with a wrapper
> around xfs_repair?
> 
> 

xfs_check checks... $SOMETHING that xfs_repair still does not, I think?

But, if you can't run it on any fs of reasonable size due to memory
piggishness, then... *shrug*

-Eric

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

* Re: xfs_check
  2008-05-27 16:48 ` xfs_check Eric Sandeen
@ 2008-05-27 16:50   ` Chris Wedgwood
  2008-05-27 16:55     ` xfs_check Eric Sandeen
  0 siblings, 1 reply; 8+ messages in thread
From: Chris Wedgwood @ 2008-05-27 16:50 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: Christoph Hellwig, xfs

On Tue, May 27, 2008 at 11:48:28AM -0500, Eric Sandeen wrote:

> xfs_check checks... $SOMETHING that xfs_repair still does not, I
> think?

does that imply that that if $SOMETHING is bad it won't be seen then
repaired then?

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

* Re: xfs_check
  2008-05-27 16:50   ` xfs_check Chris Wedgwood
@ 2008-05-27 16:55     ` Eric Sandeen
  2008-05-27 23:50       ` xfs_check Barry Naujok
  0 siblings, 1 reply; 8+ messages in thread
From: Eric Sandeen @ 2008-05-27 16:55 UTC (permalink / raw)
  To: Chris Wedgwood; +Cc: Christoph Hellwig, xfs

Chris Wedgwood wrote:
> On Tue, May 27, 2008 at 11:48:28AM -0500, Eric Sandeen wrote:
> 
>> xfs_check checks... $SOMETHING that xfs_repair still does not, I
>> think?
> 
> does that imply that that if $SOMETHING is bad it won't be seen then
> repaired then?

I can't remember what $SOMETHING is but I bet Barry knows :)

-Eric

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

* Re: xfs_check
  2008-05-27 16:26 xfs_check Christoph Hellwig
  2008-05-27 16:48 ` xfs_check Eric Sandeen
@ 2008-05-27 23:21 ` Dave Chinner
  2008-05-27 23:49 ` xfs_check Barry Naujok
  2008-05-28  8:20 ` xfs_check Emmanuel Florac
  3 siblings, 0 replies; 8+ messages in thread
From: Dave Chinner @ 2008-05-27 23:21 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: xfs

On Tue, May 27, 2008 at 06:26:05PM +0200, Christoph Hellwig wrote:
> In the past we had quite a few cases where we told people to run
> xfs_repair -n instead of xfs_check.  I think that makes a lot of sense
> because xfs_repair -n generally gives output at least as useful as
> xfs_check if not more so and also is a lot faster.  Is there any reason
> why we shouldn't simply kill xfs_check and replaced it with a wrapper
> around xfs_repair?

xfs_repair doesn't yet check free space btrees - it simply
blows them away and rebuilds htem from scratch. Hence errors
in those btrees will go unreported. xfs_check will tell you
about errors in those trees.

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

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

* Re: xfs_check
  2008-05-27 16:26 xfs_check Christoph Hellwig
  2008-05-27 16:48 ` xfs_check Eric Sandeen
  2008-05-27 23:21 ` xfs_check Dave Chinner
@ 2008-05-27 23:49 ` Barry Naujok
  2008-05-28  8:20 ` xfs_check Emmanuel Florac
  3 siblings, 0 replies; 8+ messages in thread
From: Barry Naujok @ 2008-05-27 23:49 UTC (permalink / raw)
  To: Christoph Hellwig, xfs

On Wed, 28 May 2008 02:26:05 +1000, Christoph Hellwig <hch@lst.de> wrote:

> In the past we had quite a few cases where we told people to run
> xfs_repair -n instead of xfs_check.  I think that makes a lot of sense
> because xfs_repair -n generally gives output at least as useful as
> xfs_check if not more so and also is a lot faster.  Is there any reason
> why we shouldn't simply kill xfs_check and replaced it with a wrapper
> around xfs_repair?

It's on my "todo" list! Some stuff called case-insensitive support is
delaying work like that :)

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

* Re: xfs_check
  2008-05-27 16:55     ` xfs_check Eric Sandeen
@ 2008-05-27 23:50       ` Barry Naujok
  0 siblings, 0 replies; 8+ messages in thread
From: Barry Naujok @ 2008-05-27 23:50 UTC (permalink / raw)
  To: Eric Sandeen, Chris Wedgwood; +Cc: Christoph Hellwig, xfs

On Wed, 28 May 2008 02:55:24 +1000, Eric Sandeen <sandeen@sandeen.net>  
wrote:

> Chris Wedgwood wrote:
>> On Tue, May 27, 2008 at 11:48:28AM -0500, Eric Sandeen wrote:
>>
>>> xfs_check checks... $SOMETHING that xfs_repair still does not, I
>>> think?
>>
>> does that imply that that if $SOMETHING is bad it won't be seen then
>> repaired then?
>
> I can't remember what $SOMETHING is but I bet Barry knows :)

Free space btrees!

But I would be enhancing xfs_repair -n to actually check them.

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

* Re: xfs_check
  2008-05-27 16:26 xfs_check Christoph Hellwig
                   ` (2 preceding siblings ...)
  2008-05-27 23:49 ` xfs_check Barry Naujok
@ 2008-05-28  8:20 ` Emmanuel Florac
  3 siblings, 0 replies; 8+ messages in thread
From: Emmanuel Florac @ 2008-05-28  8:20 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: xfs

Le Tue, 27 May 2008 18:26:05 +0200 vous écriviez:

>  Is there any reason
> why we shouldn't simply kill xfs_check and replaced it with a wrapper
> around xfs_repair?

I may add that I hardly even had any filesystem recently small enough
to fit xfs_check hunger for memory. All attempts to use xfs_check
invariably ended with "out of memory" for 5 years or more. Actually
IIRC I  used xfs_check succesfully only on IRIX, back in the time of
9GB usrroot drives :)

-- 
--------------------------------------------------
Emmanuel Florac               www.intellique.com   
--------------------------------------------------

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

end of thread, other threads:[~2008-05-28  8:23 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-05-27 16:26 xfs_check Christoph Hellwig
2008-05-27 16:48 ` xfs_check Eric Sandeen
2008-05-27 16:50   ` xfs_check Chris Wedgwood
2008-05-27 16:55     ` xfs_check Eric Sandeen
2008-05-27 23:50       ` xfs_check Barry Naujok
2008-05-27 23:21 ` xfs_check Dave Chinner
2008-05-27 23:49 ` xfs_check Barry Naujok
2008-05-28  8:20 ` xfs_check Emmanuel Florac

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