[Top] [All Lists]

Re: crash with latest code drop.

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: crash with latest code drop.
From: Lachlan McIlroy <lachlan@xxxxxxx>
Date: Fri, 17 Oct 2008 11:01:46 +1000
Cc: Peter Leckie <pleckie@xxxxxxx>, Dave Chinner <david@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <20081016090030.GA31558@xxxxxxxxxxxxx>
References: <48F54C20.8060704@xxxxxxx> <20081015011857.GS10716@disturbed> <20081015022948.GA20966@xxxxxxxxxxxxx> <20081015031645.GA25906@disturbed> <20081015032431.GA7426@xxxxxxxxxxxxx> <20081015035116.GB25906@disturbed> <48F584B8.8060907@xxxxxxx> <20081015061917.GC25906@disturbed> <48F5A104.5060506@xxxxxxx> <48F6AA3E.8030902@xxxxxxx> <20081016090030.GA31558@xxxxxxxxxxxxx>
Reply-to: lachlan@xxxxxxx
User-agent: Thunderbird (X11/20080914)
Christoph Hellwig wrote:
On Thu, Oct 16, 2008 at 12:43:10PM +1000, Peter Leckie wrote:
Adding the following resolved the issue however you may wish to solve it in another manner.

@@ -102,7 +102,7 @@ xfs_sync_inodes_ag(
                * in reclaim. Leave it for the reclaim code to flush.
               inode = VFS_I(ip);
-               if (!igrab(inode)) {
+               if (!inode || !igrab(inode)) {

This would be the correct fix for the current code, but can you please
put in the Inode/XFS Inode unification patches?  At least I have only
QAed the whole patchkit, and from the issues here it seems like Dave did
the same.
That patchset is already in.

And with that it would also be very good if there was any chance Dave
and me (and all others) could actually access the current tree.  The
ptools -> CVS export has been broken for mor than two days, and there
is no uptodate git tree eiter, so us external developers are completely
tapping in the dark vs the current tree.
We had no idea the CVS tree was not updating.  I'll get the OSS trees
updated today.

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