Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755930AbYCHQnQ (ORCPT ); Sat, 8 Mar 2008 11:43:16 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753621AbYCHQnA (ORCPT ); Sat, 8 Mar 2008 11:43:00 -0500 Received: from bombadil.infradead.org ([18.85.46.34]:57190 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753246AbYCHQm7 (ORCPT ); Sat, 8 Mar 2008 11:42:59 -0500 Date: Sat, 8 Mar 2008 11:42:58 -0500 From: Christoph Hellwig To: Duane Griffin Cc: Christoph Hellwig , Andreas Dilger , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, Theodore Tso , sct@redhat.com, akpm@linux-foundation.org Subject: Re: [PATCH 2/3] jbd2: replace potentially false assertion with if block Message-ID: <20080308164258.GA30512@infradead.org> References: <5e28cd633c71f6354a203a43000cbe5fef045589.1204844851.git.duaneg@dghda.com> <48afbb3a44aa24dc1e31835c14e86c3c6c1c3b0a.1204844851.git.duaneg@dghda.com> <20080307212336.GE1881@webber.adilger.int> <20080308150229.GA22430@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 870 Lines: 17 On Sat, Mar 08, 2008 at 04:40:46PM +0000, Duane Griffin wrote: > > That doesn't make any sense. They're initialized in module_init and > > destoryed in module_exit. They can never be zero at journal_load time > > unless you get random memory corruption overwriting exactly that pointer > > with zero. > > If jbd is a module, sure, but not if it is statically linked in. I > have the stacktraces to prove it :) That text above was in reply to Andreas comment about checking it in journal_load. Your fix obviously does make sense althrough doing it differently as in my reply to your first series would be even better. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/