Hello all,
buffer_insert_list puts buffers onto the head of bh->b_inode_buffers,
which means that on fsync we are writing things out in reverse order. I
think we either want this patch, or we want to walk the list in reverse
in fsync_buffers_list
(this has not been well tested, but I can't think of any problems it
would cause)
-chris
--- linux.marcelo/fs/buffer.c Thu Jun 19 09:09:28 2003
+++ linux/fs/buffer.c Thu Jun 19 09:04:17 2003
@@ -591,7 +604,7 @@
if (buffer_attached(bh))
list_del(&bh->b_inode_buffers);
set_buffer_attached(bh);
- list_add(&bh->b_inode_buffers, list);
+ list_add_tail(&bh->b_inode_buffers, list);
spin_unlock(&lru_list_lock);
}
-
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/
This archive was generated by hypermail 2b29 : Mon Jun 23 2003 - 22:00:29 EST