From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755291Ab0LGVE5 (ORCPT ); Tue, 7 Dec 2010 16:04:57 -0500 Received: from rcsinet10.oracle.com ([148.87.113.121]:20142 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755256Ab0LGVEz (ORCPT ); Tue, 7 Dec 2010 16:04:55 -0500 Content-Type: text/plain; charset=UTF-8 From: Chris Mason To: Jon Nelson Cc: Mike Snitzer , Matt , Milan Broz , Andi Kleen , linux-btrfs , dm-devel , Linux Kernel , htd , htejun , linux-ext4 Subject: Re: hunt for 2.6.37 dm-crypt+ext4 corruption? (was: Re: dm-crypt barrier support is effective) In-reply-to: References: <4CF692D1.1010906@redhat.com> <4CF6B3E8.2000406@redhat.com> <20101201212310.GA15648@redhat.com> <20101204193828.GB13871@redhat.com> <20101207142145.GA27861@think> <20101207182243.GB21112@redhat.com> <1291747731-sup-3099@think> <1291751698-sup-9297@think> <1291754340-sup-1631@think> Date: Tue, 07 Dec 2010 16:02:30 -0500 Message-Id: <1291755258-sup-8760@think> User-Agent: Sup/git Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Excerpts from Jon Nelson's message of 2010-12-07 15:48:58 -0500: > On Tue, Dec 7, 2010 at 2:41 PM, Chris Mason wrote: > > Excerpts from Jon Nelson's message of 2010-12-07 15:25:47 -0500: > >> On Tue, Dec 7, 2010 at 2:02 PM, Chris Mason wrote: > >> > Excerpts from Jon Nelson's message of 2010-12-07 14:34:40 -0500: > >> >> On Tue, Dec 7, 2010 at 12:52 PM, Chris Mason wrote: > >> >> >> postgresql errors. Typically, header corruption but from the limited > >> >> >> visibility I've had into this via strace, what I see is zeroed pages > >> >> >> where there shouldn't be. > >> >> > > >> >> > This sounds a lot like a bug higher up than dm-crypt.  Zeros tend to > >> >> > come from some piece of code explicitly filling a page with zeros, and > >> >> > that often happens in the corner cases for O_DIRECT and a few other > >> >> > places in the filesystem. > >> >> > > >> >> > Have you tried triggering this with a regular block device? > >> >> > >> >> I just tried the whole set of tests, but with /dev/sdb directly (as > >> >> ext4) without any crypt-y bits. > >> >> It takes more iterations but out of 6 tests I had one failure: same > >> >> type of thing, 'invalid page header in block ....'. > >> >> > >> >> I can't guarantee that it is a full-page of zeroes, just what I saw > >> >> from the (limited) stracing I did. > >> > > >> > Fantastic. Now for our usual suspects: > >> > > >> > 1) Is postgres using O_DIRECT?  If yes, please turn it off > >> > >> According to strace, O_DIRECT didn't show up once during the test. > >> > >> > 2) Is postgres allocating sparse files?  If yes, please have it fully > >> > allocate the file instead. > >> > >> That's a tough one. I don't think postgresql does that, but I'm not an > >> expert here. > > > > Ok, please compare du -k and du -k --apparent-size for each of the > > files involved in the postgres run. > > Because this is all done in a transaction (which fails), and because > the table is a TEMPORARY table, there *are* no files once the > transaction fails because postgresql unlinks them. > > I can modify the test to use real tables and do things outside of a > transaction, however. That would really help. > > I was using fdatasync[1] and now I'm using sync. I'm on 9 iterations > without a failure (on ext4 - no crypt). Theoretically, these settings > only make a difference in the event of a crash. However, could they > make a difference in terms of the paths taken in the kernel? Yes, the paths are different but similar. If ext4 is causing zeros to get stuffed somewhere, it should happen for both O_SYNC and fdatasync. -chris