From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755775AbYEXN7z (ORCPT ); Sat, 24 May 2008 09:59:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752947AbYEXN7p (ORCPT ); Sat, 24 May 2008 09:59:45 -0400 Received: from 1wt.eu ([62.212.114.60]:4993 "EHLO 1wt.eu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752887AbYEXN7o (ORCPT ); Sat, 24 May 2008 09:59:44 -0400 Date: Sat, 24 May 2008 15:52:58 +0200 From: Willy Tarreau To: David Greaves Cc: Greg KH , Eric Sandeen , David Chinner , xfs@oss.sgi.com, "'linux-kernel@vger.kernel.org'" , Christoph Hellwig , LinuxRaid , stable@kernel.org Subject: Re: RFI for 2.6.25.5 : Re: Regression- XFS won't mount on partitioned md array Message-ID: <20080524135258.GF6749@1wt.eu> References: <482DC043.5000307@dgreaves.com> <482DD981.5070004@sandeen.net> <482EEFDA.50101@dgreaves.com> <482EF6A7.2020909@sandeen.net> <482F67D9.70400@sandeen.net> <482FBD4C.20608@sandeen.net> <482FED60.7060405@dgreaves.com> <4838192F.3050004@dgreaves.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4838192F.3050004@dgreaves.com> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi David, On Sat, May 24, 2008 at 02:33:35PM +0100, David Greaves wrote: > Hi Greg > Perusing: > http://git.kernel.org/?p=linux/kernel/git/stable/stable-queue.git > doesn't show the patch referenced below as in the queue for 2.6.25.5 First, please avoid top-posting. > David Greaves wrote: > > Eric Sandeen wrote: > >> Eric Sandeen wrote: > >>> Eric Sandeen wrote: > >>> > >>>> I'll see if I have a little time today to track down the problem. > >>> Does this patch fix it for you? Does for me though I can't yet explain > >>> why ;) > >>> > >>> http://www.linux.sgi.com/archives/xfs/2008-05/msg00190.html > >>> > >>> -Eric > > Yes, this fixes it for me - thanks :) > > > >> So what's happening is that xfs is trying to read a page-sized IO from > >> the last sector of the log... which goes off the end of the device. > >> This looks like another regression introduced by > >> a9759f2de38a3443d5107bddde03b4f3f550060e, but fixed by Christoph's patch > >> in the URL above, which should be headed towards -stable. > > Damn, I guess I misread my bisect readings when things crashed then. > > Still, I said 'around' :) > > > >> (aside: it seems that this breaks any external log setup where the log > >> consists of the entire device... but I'd have expected the xfsqa suite > >> to catch this...?) > >> > >> The patch avoids the problem by looking for some extra locking but it > >> seems to me that the root cause is that the buffer being read at this > >> point doesn't have it's b_offset, the offset in it's page, set. Might > >> be another little buglet but harmless it seems. It would have helped to CC stable (fixed) and to give the mainline commit ID since the stable branch only holds already merged patches. Greg, the commit is 6ab455ee... Willy