[Top] [All Lists]

Re: bad inode causes xfs_bulkstat to loop

To: Roger Willcocks <willcor@xxxxxxxxx>
Subject: Re: bad inode causes xfs_bulkstat to loop
From: Nathan Scott <nathans@xxxxxxx>
Date: Fri, 3 Feb 2006 08:55:35 +1100
Cc: linux-xfs@xxxxxxxxxxx
In-reply-to: <cfac95650602020702i32fe6553sa81e341ad9f9e825@xxxxxxxxxxxxxx>; from willcor@xxxxxxxxx on Thu, Feb 02, 2006 at 03:02:21PM +0000
References: <cfac95650602020702i32fe6553sa81e341ad9f9e825@xxxxxxxxxxxxxx>
Sender: linux-xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.2.5i
On Thu, Feb 02, 2006 at 03:02:21PM +0000, Roger Willcocks wrote:
> xfs_bulkstat can get into a tight loop if there are bad inodes. (Seen
> running xfs_fsr - it took 100% of one cpu on a dual cpu machine.
> Worked fine after xfs_repair.)
> It looks like the 'break' caused by the xfs_itobp call in xfs_bulkstat
> returning an error means no terminating condition is set for the outer
> loop.
> The break looks slightly suspicious anyway - shouldn't the inode
> simply be ignored? (Ref. the comment at the top of
> xfs_bulkstat_single).

Ayup, I think you're right.

> Perhaps add 'ubleft = 0; rval = error;' before the break (as happens
> if the formatter returns 'bulkstat_rv_giveup') ?

Sounds about right - since you've got an FS handy which is giving
you this problem, could you try that change out and report back?



<Prev in Thread] Current Thread [Next in Thread>