Don't allow written blocks from this transaction to be reallocated
authorChris Mason <chris.mason@oracle.com>
Fri, 4 Apr 2008 19:40:00 +0000 (15:40 -0400)
committerDavid Woodhouse <dwmw2@hera.kernel.org>
Fri, 4 Apr 2008 19:40:00 +0000 (15:40 -0400)
When a block is freed, it can be immediately reused if it is from
the current transaction.  But, an extra check is required to make sure
the block had not been written yet.  If it were reused after being written,
the transid in the block header might match the transid of the
next time the block was allocated.

The parent node records the transaction ID of the block it is pointing to,
and this is used as part of validating the block on reads.  So, there
can only be one version of a block per transaction.

extent-tree.c

index 8c39537..f3c36e1 100644 (file)
@@ -1262,7 +1262,9 @@ static int pin_down_bytes(struct btrfs_root *root, u64 bytenr, u32 num_bytes,
                        if (btrfs_buffer_uptodate(buf)) {
                                u64 transid =
                                    root->fs_info->running_transaction->transid;
-                               if (btrfs_header_generation(buf) == transid) {
+                               if (btrfs_header_generation(buf) ==
+                                   transid && !btrfs_header_flag(buf,
+                                              BTRFS_HEADER_FLAG_WRITTEN)) {
                                        free_extent_buffer(buf);
                                        return 1;
                                }