Re: [PATCH] JBD checkpoint cleanup strikes back

[Mark Fasheh <mark.fasheh@oracle.com>]

On Tue, May 02, 2006 at 08:46:46PM +0200, Jan Kara wrote:
>   The patch was already in 2.6.16-rc3 but was dropped because of
> problems with OCFS2. I've now tracked down the problem - OCFS2 relies on
> the fact that buffer does not have journal_head if it is not on any
> transaction's list. It assumes that if the buffer has buffer_jbd set,
> then it is journaled and hence the node has uptodate data in the
> buffer. My patch broke that assumption as in one path I forgot to call
> journal_remove_journal_head() and hence I was leaving behind some
> buffers not attached to any transaction but with journal_head. Now that
> leak is fixed and OCFS2 seems to work fine also with my patch.
Ahh, ok that makes sense and it definitely sounds like the type of thing
that would cause OCFS2 to pick up stale data.

>   Andrew, could you please put the patch into -mm? Thanks.
Without commenting any further on the patch, I can definitely offer up some
more testing on my end should Andrew decide to pick this up.
	--Mark

--
Mark Fasheh
Senior Software Developer, Oracle
mark.fasheh@oracle.com