Btrfs: Explicitly send a root objectid to count_snapshots_in_path

Signed-off-by: Chris Mason <chris.mason@oracle.com>

2 files changed, 11 insertions, 44 deletions

diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index d6243c2afc73..91c2b27d9689 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -688,7 +688,7 @@ u32 btrfs_count_snapshots_in_path(struct btrfs_root *root,
         struct btrfs_path *path;
         u64 bytenr;
         u64 found_objectid;
-        u64 root_objectid = 0;
+        u64 root_objectid = root->root_key.objectid;
         u32 total_count = 0;
         u32 cur_count;
         u32 refs;
@@ -749,15 +749,11 @@ again:
                                           struct btrfs_extent_ref);
                 found_objectid = btrfs_ref_root(l, ref_item);
 
-                if (found_objectid != root_objectid)
-                        total_count++;
-
-                if (total_count > 1)
-                        goto out;
-
-                if (root_objectid == 0)
-                        root_objectid = found_objectid;
-
+                if (found_objectid != root_objectid) {
+                        total_count = 2;
+                        break;
+                }
+                total_count = 1;
                 path->slots[0]++;
         }
         if (cur_count == 0) {
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index d6de46961887..5003a86510a1 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -1770,6 +1770,11 @@ struct extent_map *btrfs_get_extent(struct inode *inode, struct page *page,
 again:
         em = lookup_extent_mapping(em_tree, start, end);
         if (em) {
+                if (em->start > start) {
+                        printk("get_extent start %Lu em start %Lu\n",
+                               start, em->start);
+                        WARN_ON(1);
+                }
                 goto out;
         }
         if (!em) {
@@ -1952,23 +1957,6 @@ static sector_t btrfs_bmap(struct address_space *mapping, sector_t iblock)
         return extent_bmap(mapping, iblock, btrfs_get_extent);
 }
 
-static int btrfs_prepare_write(struct file *file, struct page *page,
-                               unsigned from, unsigned to)
-{
-        struct btrfs_root *root = BTRFS_I(page->mapping->host)->root;
-        int err;
-
-        mutex_lock(&root->fs_info->fs_mutex);
-        err = btrfs_check_free_space(root, PAGE_CACHE_SIZE, 0);
-        mutex_unlock(&root->fs_info->fs_mutex);
-        if (err)
-                return -ENOSPC;
-
-        return extent_prepare_write(&BTRFS_I(page->mapping->host)->extent_tree,
-                                    page->mapping->host, page, from, to,
-                                    btrfs_get_extent);
-}
-
 int btrfs_readpage(struct file *file, struct page *page)
 {
         struct extent_map_tree *tree;
@@ -2120,21 +2108,6 @@ static void btrfs_truncate(struct inode *inode)
         btrfs_btree_balance_dirty(root, nr);
 }
 
-int btrfs_commit_write(struct file *file, struct page *page,
-                       unsigned from, unsigned to)
-{
-        loff_t pos = ((loff_t)page->index << PAGE_CACHE_SHIFT) + to;
-        struct inode *inode = page->mapping->host;
-
-        btrfs_cow_one_page(inode, page, PAGE_CACHE_SIZE);
-
-        if (pos > inode->i_size) {
-                i_size_write(inode, pos);
-                mark_inode_dirty(inode);
-        }
-        return 0;
-}
-
 static int create_subvol(struct btrfs_root *root, char *name, int namelen)
 {
         struct btrfs_trans_handle *trans;
@@ -2930,8 +2903,6 @@ static struct address_space_operations btrfs_aops = {
         .writepages     = btrfs_writepages,
         .readpages      = btrfs_readpages,
         .sync_page      = block_sync_page,
-        .prepare_write  = btrfs_prepare_write,
-        .commit_write   = btrfs_commit_write,
         .bmap           = btrfs_bmap,
         .invalidatepage = btrfs_invalidatepage,
         .releasepage    = btrfs_releasepage,