From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [PATCH] xfs: Fix possible truncation of log data in xlog_bread_noalign() Date: Tue, 5 Mar 2013 08:03:11 +1100 Message-ID: <20130304210311.GQ23616@dastard> References: <20130223000802.GB26081@dastard> <20130223235546.GA5551@dastard> <20130224141017.GC5551@dastard> <5130CE9E.9080501@sgi.com> <51310E7A.1000905@sgi.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Alex Elder , Mark Tinguely , "linux-kernel@vger.kernel.org" , Chris Metcalf , "xfs@oss.sgi.com" , Ben Myers , Dave Chinner , "linux-fsdevel@vger.kernel.org" To: Tony Lu Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com List-Id: linux-fsdevel.vger.kernel.org On Mon, Mar 04, 2013 at 08:32:45AM +0000, Tony Lu wrote: > Thanks for you following up. > > My apologize that I just found that it is one change I made before > that causes this problem. This change forces mkfs.xfs to format > xfs partitions whose sectorsize were not smaller than 4096 bytes, > which was due to a bug that earlier versions of xfs used (struct > *page)->private(long) as a bitmap to represent each block's state > within a page (the size of a page could be 64K or larger, then it > needs 128 bit or more to represent each block's state within a > page). You do realise that bug doesn't affect x86-64 platforms as they don't support 64k pages? > This is reproducible on 2.6.38.6 kernel on X86. But I do not get > why this change makes the xfs log inconsistent during > mount/cp/umount operations. Neither do I, and I don't care to look any further because the problem is of your own making. In future, please check first that the bug you are reporting is reproducable on a current upstream kernel and userspace. Cheers, Dave. -- Dave Chinner david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs