From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Darrick J. Wong" Subject: Re: [PATCH v3] jbd2: Fix block tag checksum verification brokenness Date: Mon, 3 Jun 2013 00:55:21 -0700 Message-ID: <20130603075521.GA6088@blackbox.djwong.org> References: <20130603021630.13790.qmail@science.horizon.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org To: George Spelvin Return-path: Received: from aserp1040.oracle.com ([141.146.126.69]:48376 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751350Ab3FCHz1 (ORCPT ); Mon, 3 Jun 2013 03:55:27 -0400 Content-Disposition: inline In-Reply-To: <20130603021630.13790.qmail@science.horizon.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sun, Jun 02, 2013 at 10:16:30PM -0400, George Spelvin wrote: > > Fortunately metadata checksumming is still "experimental" and not in a shipping > > e2fsprogs, so there should be few users affected by this. > > Am I reading this patch correctly that this changes which half of > the 32-bit checksum is stored on little-endian (e.g. x86) machines? > Before, it stored the low 16 bits of cpu_to_be32(csum), meaning the > high 16 bits of csum. ...unless you're using a BE system in which case it's the low 16 bits. :) > Now, it's cpu_to_be16(csum32), which is the low 16 bits of csum32. > > It so happens that I have multiple metadata_csum file systems. > (I enabled it a while ago on a machine where I wasn't sure if corruption > was RAM or disk, and have been using it on SSE4.2 machines since.) > > Is there an upgrade path? Also, what's the corresponding e2fsprogs > commit that supports this change? The kernel patch fixes journal bogosity when moving a disk between big and little endian systems. e2fsprogs didn't have the brokenness, so there's no change needed. In theory you'd only hit this if you happened to crash an x86 box with an ext4 fs, move the disk to a ppc, and try to recover it there. The upgrade path is to umount cleanly and reboot with a patched kernel while hoping that you don't crash while rebooting. Sorry about the bumpy metadata_csum ride. --D