Btrfs: trivial sparse fixes
authorChristoph Hellwig <hch@lst.de>
Fri, 5 Sep 2008 20:43:31 +0000 (16:43 -0400)
committerChris Mason <chris.mason@oracle.com>
Thu, 25 Sep 2008 15:04:07 +0000 (11:04 -0400)
Fix a bunch of trivial sparse complaints.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
fs/btrfs/ctree.c
fs/btrfs/extent_io.c
fs/btrfs/ioctl.c
fs/btrfs/sysfs.c

index 6cd4e929bd4ddffbe2e7482c6ca676d7095f4de8..ab07557bac071c8cc168ffa7a477ed39f02a3ebc 100644 (file)
@@ -2749,7 +2749,6 @@ int btrfs_insert_empty_items(struct btrfs_trans_handle *trans,
        BUG_ON(slot < 0);
 
        if (slot != nritems) {
-               int i;
                unsigned int old_data = btrfs_item_end_nr(leaf, slot);
 
                if (old_data < data_end) {
@@ -2913,7 +2912,6 @@ int btrfs_del_items(struct btrfs_trans_handle *trans, struct btrfs_root *root,
        nritems = btrfs_header_nritems(leaf);
 
        if (slot + nr != nritems) {
-               int i;
                int data_end = leaf_data_end(root, leaf);
 
                memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
index 7ca89c45d4010ccce889abddfa1d8cf71439a13e..239e7c908abf884dc65c6ab1aa2c471210691848 100644 (file)
@@ -3400,10 +3400,8 @@ int try_release_extent_buffer(struct extent_io_tree *tree, struct page *page)
        }
        /* at this point we can safely release the extent buffer */
        num_pages = num_extent_pages(eb->start, eb->len);
-       for (i = 0; i < num_pages; i++) {
-               struct page *page = extent_buffer_page(eb, i);
-               page_cache_release(page);
-       }
+       for (i = 0; i < num_pages; i++)
+               page_cache_release(extent_buffer_page(eb, i));
        rb_erase(&eb->rb_node, &tree->buffer);
        __free_extent_buffer(eb);
 out:
index 59b64c738fd136aac43d6a193514af078e2382e6..f84b5f6991cce80826c307050e7f0fdcafce0a30 100644 (file)
@@ -739,7 +739,7 @@ long btrfs_ioctl_trans_end(struct file *file)
                goto out;
        }
        btrfs_end_transaction(trans, root);
-       file->private_data = 0;
+       file->private_data = NULL;
 
        mutex_lock(&root->fs_info->trans_mutex);
        root->fs_info->open_ioctl_trans--;
index 1cf4f4f8d2ce3588318c2a31cc8708d4e0707161..61af5d8446e397df031d9961cc54210ac2b16558 100644 (file)
@@ -254,7 +254,7 @@ void btrfs_sysfs_del_super(struct btrfs_fs_info *fs)
        wait_for_completion(&fs->kobj_unregister);
 }
 
-int btrfs_init_sysfs()
+int btrfs_init_sysfs(void)
 {
        btrfs_kset = kset_create_and_add("btrfs", NULL, fs_kobj);
        if (!btrfs_kset)
@@ -262,7 +262,7 @@ int btrfs_init_sysfs()
        return 0;
 }
 
-void btrfs_exit_sysfs()
+void btrfs_exit_sysfs(void)
 {
        kset_unregister(btrfs_kset);
 }
@@ -289,12 +289,12 @@ void btrfs_sysfs_del_super(struct btrfs_fs_info *fs)
        return;
 }
 
-int btrfs_init_sysfs()
+int btrfs_init_sysfs(void)
 {
        return 0;
 }
 
-void btrfs_exit_sysfs()
+void btrfs_exit_sysfs(void)
 {
        return;
 }