From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joel Becker Date: Thu, 12 Aug 2010 16:03:53 -0700 Subject: [Ocfs2-devel] [PATCH 2/3] JBD2: Allow feature checks before journal recovery In-Reply-To: <20100812174318.GD6561@mail.oracle.com> References: <874ofr2myq.fsf@patl.com> <8739vb2mxr.fsf@patl.com> <20100812174318.GD6561@mail.oracle.com> Message-ID: <20100812230353.GE22777@mail.oracle.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jan Kara , Ted Ts'o , "Patrick J. LoPresti" , ocfs2-devel@oss.oracle.com, linux-fsdevel@vger.kernel.org, linux-ext4@vge On Thu, Aug 12, 2010 at 10:43:19AM -0700, Joel Becker wrote: > On Thu, Jul 22, 2010 at 03:04:16PM -0700, Patrick J. LoPresti wrote: > > Before we start accessing a huge (> 16 TiB) OCFS2 volume, we need to > > confirm that its journal supports 64-bit offsets. In particular, we > > need to check the journal's feature bits before recovering the journal. > > > > This is not possible with JBD2 at present, because the journal > > superblock (where the feature bits reside) is not loaded from disk until > > the journal is recovered. > > > > This patch loads the journal superblock in > > jbd2_journal_check_used_features() if it has not already been loaded, > > allowing us to check the feature bits before journal recovery. > > > > Signed-off-by: Patrick LoPresti > > Dear jbd2 developers, > I've pushed this patch to the merge-window branch of ocfs2.git. > I'm ready to send it to Linus, but I need your OK. To be specific, I'd really like an Acked-by. Joel -- "The doctrine of human equality reposes on this: that there is no man really clever who has not found that he is stupid." - Gilbert K. Chesterson Joel Becker Consulting Software Developer Oracle E-mail: joel.becker at oracle.com Phone: (650) 506-8127 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761092Ab0HLXEg (ORCPT ); Thu, 12 Aug 2010 19:04:36 -0400 Received: from rcsinet10.oracle.com ([148.87.113.121]:52840 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761034Ab0HLXEd (ORCPT ); Thu, 12 Aug 2010 19:04:33 -0400 Date: Thu, 12 Aug 2010 16:03:53 -0700 From: Joel Becker To: Jan Kara , "Ted Ts'o" , "Patrick J. LoPresti" , ocfs2-devel@oss.oracle.com, linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [Ocfs2-devel] [PATCH 2/3] JBD2: Allow feature checks before journal recovery Message-ID: <20100812230353.GE22777@mail.oracle.com> Mail-Followup-To: Jan Kara , Ted Ts'o , "Patrick J. LoPresti" , ocfs2-devel@oss.oracle.com, linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org References: <874ofr2myq.fsf@patl.com> <8739vb2mxr.fsf@patl.com> <20100812174318.GD6561@mail.oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100812174318.GD6561@mail.oracle.com> X-Burt-Line: Trees are cool. X-Red-Smith: Ninety feet between bases is perhaps as close as man has ever come to perfection. User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 12, 2010 at 10:43:19AM -0700, Joel Becker wrote: > On Thu, Jul 22, 2010 at 03:04:16PM -0700, Patrick J. LoPresti wrote: > > Before we start accessing a huge (> 16 TiB) OCFS2 volume, we need to > > confirm that its journal supports 64-bit offsets. In particular, we > > need to check the journal's feature bits before recovering the journal. > > > > This is not possible with JBD2 at present, because the journal > > superblock (where the feature bits reside) is not loaded from disk until > > the journal is recovered. > > > > This patch loads the journal superblock in > > jbd2_journal_check_used_features() if it has not already been loaded, > > allowing us to check the feature bits before journal recovery. > > > > Signed-off-by: Patrick LoPresti > > Dear jbd2 developers, > I've pushed this patch to the merge-window branch of ocfs2.git. > I'm ready to send it to Linus, but I need your OK. To be specific, I'd really like an Acked-by. Joel -- "The doctrine of human equality reposes on this: that there is no man really clever who has not found that he is stupid." - Gilbert K. Chesterson Joel Becker Consulting Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joel Becker Subject: Re: [PATCH 2/3] JBD2: Allow feature checks before journal recovery Date: Thu, 12 Aug 2010 16:03:53 -0700 Message-ID: <20100812230353.GE22777@mail.oracle.com> References: <874ofr2myq.fsf@patl.com> <8739vb2mxr.fsf@patl.com> <20100812174318.GD6561@mail.oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jan Kara , "Ted Ts'o" , "Patrick J. LoPresti" , ocfs2-devel@oss.oracle.com, linux-fsdevel@vger.kernel.org, linux-ext4@vge Return-path: Content-Disposition: inline In-Reply-To: <20100812174318.GD6561@mail.oracle.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ocfs2-devel-bounces@oss.oracle.com Errors-To: ocfs2-devel-bounces@oss.oracle.com List-Id: linux-fsdevel.vger.kernel.org On Thu, Aug 12, 2010 at 10:43:19AM -0700, Joel Becker wrote: > On Thu, Jul 22, 2010 at 03:04:16PM -0700, Patrick J. LoPresti wrote: > > Before we start accessing a huge (> 16 TiB) OCFS2 volume, we need to > > confirm that its journal supports 64-bit offsets. In particular, we > > need to check the journal's feature bits before recovering the journal. > > > > This is not possible with JBD2 at present, because the journal > > superblock (where the feature bits reside) is not loaded from disk until > > the journal is recovered. > > > > This patch loads the journal superblock in > > jbd2_journal_check_used_features() if it has not already been loaded, > > allowing us to check the feature bits before journal recovery. > > > > Signed-off-by: Patrick LoPresti > > Dear jbd2 developers, > I've pushed this patch to the merge-window branch of ocfs2.git. > I'm ready to send it to Linus, but I need your OK. To be specific, I'd really like an Acked-by. Joel -- "The doctrine of human equality reposes on this: that there is no man really clever who has not found that he is stupid." - Gilbert K. Chesterson Joel Becker Consulting Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127