Re: [PATCH] JBD checkpoint cleanup strikes back

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
[email protected]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux