lkml.org 
[lkml]   [2002]   [Aug]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[Patch 7/8] 2.4.20-pre4/ext3: Fix buffer alias problem
Fix problem in data=journal mode where writeback could be left pending on a
journaled, deleted disk block. If that block then gets reallocated, we can
end up with an alias in which the old data can be written back to disk over
the new. Thanks to Neil Brown for spotting this and coming up with the
initial fix.

--- linux-ext3-2.4merge/fs/jbd/commit.c.=K0008=.orig Tue Aug 27 23:16:07 2002
+++ linux-ext3-2.4merge/fs/jbd/commit.c Tue Aug 27 23:19:57 2002
@@ -663,6 +663,20 @@
* there's no point in keeping a checkpoint record for
* it. */
bh = jh2bh(jh);
+
+ /* A buffer which has been freed while still being
+ * journaled by a previous transaction may end up still
+ * being dirty here, but we want to avoid writing back
+ * that buffer in the future now that the last use has
+ * been committed. That's not only a performance gain,
+ * it also stops aliasing problems if the buffer is left
+ * behind for writeback and gets reallocated for another
+ * use in a different page. */
+ if (__buffer_state(bh, Freed)) {
+ clear_bit(BH_Freed, &bh->b_state);
+ clear_bit(BH_JBDDirty, &bh->b_state);
+ }
+
if (buffer_jdirty(bh)) {
JBUFFER_TRACE(jh, "add to new checkpointing trans");
__journal_insert_checkpoint(jh, commit_transaction);
-
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/
\
 
 \ /
  Last update: 2005-03-22 13:28    [W:0.019 / U:3.904 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site