From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (Postfix) with ESMTP id 4D0CB7F4E for ; Mon, 22 Jul 2013 17:54:56 -0500 (CDT) Received: from cuda.sgi.com (cuda1.sgi.com [192.48.157.11]) by relay2.corp.sgi.com (Postfix) with ESMTP id 1EA9C304039 for ; Mon, 22 Jul 2013 15:54:53 -0700 (PDT) Received: from ipmail07.adl2.internode.on.net (ipmail07.adl2.internode.on.net [150.101.137.131]) by cuda.sgi.com with ESMTP id 2vxy3yCROKbYtpw0 for ; Mon, 22 Jul 2013 15:54:51 -0700 (PDT) Date: Tue, 23 Jul 2013 08:54:45 +1000 From: Dave Chinner Subject: Re: [Bisected] Corruption of root fs during git bisect of drm system hang Message-ID: <20130722225445.GA19986@dastard> References: <20130713090523.GA362@x4> <20130712070721.GA359@x4> <20130715022841.GH5228@dastard> <20130715064734.GA361@x4> <20130719122235.GA360@x4> <20130720014836.GZ11674@dastard> <20130722102223.GG11674@dastard> <20130722104743.GA361@x4> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20130722104743.GA361@x4> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Markus Trippelsdorf Cc: Ben Myers , Mark Tinguely , Stan Hoeppner , xfs@oss.sgi.com On Mon, Jul 22, 2013 at 12:47:43PM +0200, Markus Trippelsdorf wrote: > On 2013.07.22 at 20:22 +1000, Dave Chinner wrote: > > On Sat, Jul 20, 2013 at 11:48:36AM +1000, Dave Chinner wrote: > > > On Fri, Jul 19, 2013 at 02:22:35PM +0200, Markus Trippelsdorf wrote: > > > > On 2013.07.15 at 08:47 +0200, Markus Trippelsdorf wrote: > > > > I've bisected this issue to the following commit: > > > > > > > > commit cca9f93a52d2ead50b5da59ca83d5f469ee4be5f > > > > Author: Dave Chinner > > > > Date: Thu Jun 27 16:04:49 2013 +1000 > > > > > > > > xfs: don't do IO when creating an new inode > > > > > > > > Reverting this commit on top of the Linus tree "solves" all problems for > > > > me. IOW I no longer loose my KDE and LibreOffice config files during a > > > > crash. Log recovery now works fine and xfs_repair shows no issues. > > > > > > Thanks for bisecting this, Marcus. > > > > > > I'll admit, right now it doesn't make a lot of sense to me - I don't > > > immediately see a connection between not reading an inode during the > > > create phase and unlinked list and directory corruption after a > > > crash. But now you've identified a change that might be the cause, > > > I have an avenue of investigation I can follow. > > > > > > Indeed, in the time I've taken to write this mail I've thought of > > > 2-3 possible causes that I need to investigate.... > > > > Hi Markus, can you test the patch I just posted to the list titled > > "xfs: di_flushiter considered harmful" and see if it fixes your > > problem? Archive link here: > > > > http://oss.sgi.com/pipermail/xfs/2013-July/028331.html > > Unfortunately no. I still get the same corruption with this patch > applied. Umm, really? can you please put together a simple reproducer then? Because it definitely fixes the problem that Mark reproduced... > (It's embarrassing to mention, but please add: > Reported-by: Markus Trippelsdorf > to the next iteration of this patch. > Thanks.) The reason I asked you to test it was so I could confirm that it was fixing the problem you've reported and so I could added reported-by and tested-by tags to it. Indeed, if it doesn't fix your problem, then it's not fixing the bug you reported, and so adding such tags is wrong.... ;) Cheers, Dave. -- Dave Chinner david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs