reiserfs: cleanup, remove sb argument from journal_mark_dirty
authorJeff Mahoney <jeffm@suse.com>
Wed, 23 Apr 2014 14:00:39 +0000 (10:00 -0400)
committerJan Kara <jack@suse.cz>
Tue, 6 May 2014 21:10:37 +0000 (23:10 +0200)
journal_mark_dirty doesn't need a separate sb argument; It's provided
by the transaction handle.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Jan Kara <jack@suse.cz>
12 files changed:
fs/reiserfs/bitmap.c
fs/reiserfs/do_balan.c
fs/reiserfs/file.c
fs/reiserfs/fix_node.c
fs/reiserfs/inode.c
fs/reiserfs/journal.c
fs/reiserfs/namei.c
fs/reiserfs/objectid.c
fs/reiserfs/reiserfs.h
fs/reiserfs/resize.c
fs/reiserfs/stree.c
fs/reiserfs/super.c

index 70daba6..46359bf 100644 (file)
@@ -256,14 +256,14 @@ static int scan_bitmap_block(struct reiserfs_transaction_handle *th,
                                }
                        }
                        bi->free_count -= (end - *beg);
-                       journal_mark_dirty(th, s, bh);
+                       journal_mark_dirty(th, bh);
                        brelse(bh);
 
                        /* free block count calculation */
                        reiserfs_prepare_for_journal(s, SB_BUFFER_WITH_SB(s),
                                                     1);
                        PUT_SB_FREE_BLOCKS(s, SB_FREE_BLOCKS(s) - (end - *beg));
-                       journal_mark_dirty(th, s, SB_BUFFER_WITH_SB(s));
+                       journal_mark_dirty(th, SB_BUFFER_WITH_SB(s));
 
                        return end - (*beg);
                } else {
@@ -453,14 +453,14 @@ static void _reiserfs_free_block(struct reiserfs_transaction_handle *th,
                               "block %lu: bit already cleared", block);
        }
        apbi[nr].free_count++;
-       journal_mark_dirty(th, s, bmbh);
+       journal_mark_dirty(th, bmbh);
        brelse(bmbh);
 
        reiserfs_prepare_for_journal(s, sbh, 1);
        /* update super block */
        set_sb_free_blocks(rs, sb_free_blocks(rs) + 1);
 
-       journal_mark_dirty(th, s, sbh);
+       journal_mark_dirty(th, sbh);
        if (for_unformatted) {
                int depth = reiserfs_write_unlock_nested(s);
                dquot_free_block_nodirty(inode, 1);
index 399b200..7decd93 100644 (file)
@@ -56,8 +56,7 @@ static inline void buffer_info_init_bh(struct tree_balance *tb,
 inline void do_balance_mark_leaf_dirty(struct tree_balance *tb,
                                       struct buffer_head *bh, int flag)
 {
-       journal_mark_dirty(tb->transaction_handle,
-                          tb->transaction_handle->t_super, bh);
+       journal_mark_dirty(tb->transaction_handle, bh);
 }
 
 #define do_balance_mark_internal_dirty do_balance_mark_leaf_dirty
index a059320..db8e3ae 100644 (file)
@@ -212,7 +212,7 @@ int reiserfs_commit_page(struct inode *inode, struct page *page,
                        set_buffer_uptodate(bh);
                        if (logit) {
                                reiserfs_prepare_for_journal(s, bh, 1);
-                               journal_mark_dirty(&th, s, bh);
+                               journal_mark_dirty(&th, bh);
                        } else if (!buffer_dirty(bh)) {
                                mark_buffer_dirty(bh);
                                /*
index 144bd62..38163d7 100644 (file)
@@ -2572,7 +2572,7 @@ int fix_nodes(int op_mode, struct tree_balance *tb,
         */
        reiserfs_prepare_for_journal(tb->tb_sb,
                                     SB_BUFFER_WITH_SB(tb->tb_sb), 1);
-       journal_mark_dirty(tb->transaction_handle, tb->tb_sb,
+       journal_mark_dirty(tb->transaction_handle,
                           SB_BUFFER_WITH_SB(tb->tb_sb));
        if (FILESYSTEM_CHANGED_TB(tb))
                return REPEAT_SEARCH;
index 5cc5583..ee532c9 100644 (file)
@@ -816,7 +816,7 @@ int reiserfs_get_block(struct inode *inode, sector_t block,
                                reiserfs_add_ordered_list(inode, bh_result);
                        put_block_num(item, pos_in_item, allocated_block_nr);
                        unfm_ptr = allocated_block_nr;
-                       journal_mark_dirty(th, inode->i_sb, bh);
+                       journal_mark_dirty(th, bh);
                        reiserfs_update_sd(th, inode);
                }
                set_block_dev_mapped(bh_result, unfm_ptr, inode);
@@ -1505,7 +1505,7 @@ void reiserfs_update_sd_size(struct reiserfs_transaction_handle *th,
                break;
        }
        update_stat_data(&path, inode, size);
-       journal_mark_dirty(th, th->t_super, bh);
+       journal_mark_dirty(th, bh);
        pathrelse(&path);
        return;
 }
@@ -2457,7 +2457,7 @@ static int map_block_for_writepage(struct inode *inode,
                memcpy(ih_item_body(bh, ih) + pos_in_item, p + bytes_copied,
                       copy_size);
 
-               journal_mark_dirty(&th, inode->i_sb, bh);
+               journal_mark_dirty(&th, bh);
                bytes_copied += copy_size;
                set_block_dev_mapped(bh_result, 0, inode);
 
@@ -2627,7 +2627,7 @@ static int reiserfs_write_full_page(struct page *page,
 
                if (checked) {
                        reiserfs_prepare_for_journal(s, bh, 1);
-                       journal_mark_dirty(&th, s, bh);
+                       journal_mark_dirty(&th, bh);
                        continue;
                }
                /*
index 5b4ccd5..688c7d2 100644 (file)
@@ -1929,8 +1929,7 @@ static int do_journal_release(struct reiserfs_transaction_handle *th,
                        reiserfs_prepare_for_journal(sb,
                                                     SB_BUFFER_WITH_SB(sb),
                                                     1);
-                       journal_mark_dirty(&myth, sb,
-                                          SB_BUFFER_WITH_SB(sb));
+                       journal_mark_dirty(&myth, SB_BUFFER_WITH_SB(sb));
                        do_journal_end(&myth, FLUSH_ALL);
                        flushed = 1;
                }
@@ -1943,8 +1942,7 @@ static int do_journal_release(struct reiserfs_transaction_handle *th,
                        reiserfs_prepare_for_journal(sb,
                                                     SB_BUFFER_WITH_SB(sb),
                                                     1);
-                       journal_mark_dirty(&myth, sb,
-                                          SB_BUFFER_WITH_SB(sb));
+                       journal_mark_dirty(&myth, SB_BUFFER_WITH_SB(sb));
                        do_journal_end(&myth, FLUSH_ALL);
                }
        }
@@ -3268,8 +3266,9 @@ int journal_begin(struct reiserfs_transaction_handle *th,
  * if j_len, is bigger than j_len_alloc, it pushes j_len_alloc to 10 + j_len.
  */
 int journal_mark_dirty(struct reiserfs_transaction_handle *th,
-                      struct super_block *sb, struct buffer_head *bh)
+                      struct buffer_head *bh)
 {
+       struct super_block *sb = th->t_super;
        struct reiserfs_journal *journal = SB_JOURNAL(sb);
        struct reiserfs_journal_cnode *cn = NULL;
        int count_already_incd = 0;
@@ -3522,7 +3521,7 @@ int journal_end_sync(struct reiserfs_transaction_handle *th)
        if (journal->j_len == 0) {
                reiserfs_prepare_for_journal(sb, SB_BUFFER_WITH_SB(sb),
                                             1);
-               journal_mark_dirty(th, sb, SB_BUFFER_WITH_SB(sb));
+               journal_mark_dirty(th, SB_BUFFER_WITH_SB(sb));
        }
        return do_journal_end(th, COMMIT_NOW | WAIT);
 }
@@ -3576,8 +3575,7 @@ void reiserfs_flush_old_commits(struct super_block *sb)
                        reiserfs_prepare_for_journal(sb,
                                                     SB_BUFFER_WITH_SB(sb),
                                                     1);
-                       journal_mark_dirty(&th, sb,
-                                          SB_BUFFER_WITH_SB(sb));
+                       journal_mark_dirty(&th, SB_BUFFER_WITH_SB(sb));
 
                        /*
                         * we're only being called from kreiserfsd, it makes
@@ -3863,7 +3861,7 @@ static int __commit_trans_jl(struct inode *inode, unsigned long id,
                if (journal->j_trans_id != id) {
                        reiserfs_prepare_for_journal(sb, SB_BUFFER_WITH_SB(sb),
                                                     1);
-                       journal_mark_dirty(&th, sb, SB_BUFFER_WITH_SB(sb));
+                       journal_mark_dirty(&th, SB_BUFFER_WITH_SB(sb));
                        ret = journal_end(&th);
                        goto flush_commit_only;
                }
@@ -4014,7 +4012,7 @@ static int do_journal_end(struct reiserfs_transaction_handle *th, int flags)
        if (journal->j_len == 0) {
                reiserfs_prepare_for_journal(sb, SB_BUFFER_WITH_SB(sb),
                                             1);
-               journal_mark_dirty(th, sb, SB_BUFFER_WITH_SB(sb));
+               journal_mark_dirty(th, SB_BUFFER_WITH_SB(sb));
        }
 
        lock_journal(sb);
index df321eb..b8a873d 100644 (file)
@@ -1531,10 +1531,10 @@ static int reiserfs_rename(struct inode *old_dir, struct dentry *old_dentry,
 
        mark_de_visible(new_de.de_deh + new_de.de_entry_num);
        set_ino_in_dir_entry(&new_de, INODE_PKEY(old_inode));
-       journal_mark_dirty(&th, old_dir->i_sb, new_de.de_bh);
+       journal_mark_dirty(&th, new_de.de_bh);
 
        mark_de_hidden(old_de.de_deh + old_de.de_entry_num);
-       journal_mark_dirty(&th, old_dir->i_sb, old_de.de_bh);
+       journal_mark_dirty(&th, old_de.de_bh);
        ctime = CURRENT_TIME_SEC;
        old_dir->i_ctime = old_dir->i_mtime = ctime;
        new_dir->i_ctime = new_dir->i_mtime = ctime;
@@ -1558,7 +1558,7 @@ static int reiserfs_rename(struct inode *old_dir, struct dentry *old_dentry,
        if (S_ISDIR(old_inode_mode)) {
                /* adjust ".." of renamed directory */
                set_ino_in_dir_entry(&dot_dot_de, INODE_PKEY(new_dir));
-               journal_mark_dirty(&th, new_dir->i_sb, dot_dot_de.de_bh);
+               journal_mark_dirty(&th, dot_dot_de.de_bh);
 
                /*
                 * there (in new_dir) was no directory, so it got new link
index 99f66f8..99a5d5d 100644 (file)
@@ -89,7 +89,7 @@ __u32 reiserfs_get_unused_objectid(struct reiserfs_transaction_handle *th)
                set_sb_oid_cursize(rs, sb_oid_cursize(rs) - 2);
        }
 
-       journal_mark_dirty(th, s, SB_BUFFER_WITH_SB(s));
+       journal_mark_dirty(th, SB_BUFFER_WITH_SB(s));
        return unused_objectid;
 }
 
@@ -107,7 +107,7 @@ void reiserfs_release_objectid(struct reiserfs_transaction_handle *th,
        check_objectid_map(s, map);
 
        reiserfs_prepare_for_journal(s, SB_BUFFER_WITH_SB(s), 1);
-       journal_mark_dirty(th, s, SB_BUFFER_WITH_SB(s));
+       journal_mark_dirty(th, SB_BUFFER_WITH_SB(s));
 
        /*
         * start at the beginning of the objectid map (i = 0) and go to
index 7755fe5..86929c5 100644 (file)
@@ -2864,7 +2864,7 @@ void reiserfs_free_jh(struct buffer_head *bh);
 int reiserfs_add_tail_list(struct inode *inode, struct buffer_head *bh);
 int reiserfs_add_ordered_list(struct inode *inode, struct buffer_head *bh);
 int journal_mark_dirty(struct reiserfs_transaction_handle *,
-                      struct super_block *, struct buffer_head *bh);
+                      struct buffer_head *bh);
 
 static inline int reiserfs_file_data_log(struct inode *inode)
 {
index 99eb325..6052d32 100644 (file)
@@ -193,7 +193,7 @@ int reiserfs_resize(struct super_block *s, unsigned long block_count_new)
                reiserfs_clear_le_bit(i, bh->b_data);
        info->free_count += s->s_blocksize * 8 - block_r;
 
-       journal_mark_dirty(&th, s, bh);
+       journal_mark_dirty(&th, bh);
        brelse(bh);
 
        /* Correct new last bitmap block - It may not be full */
@@ -209,7 +209,7 @@ int reiserfs_resize(struct super_block *s, unsigned long block_count_new)
        reiserfs_prepare_for_journal(s, bh, 1);
        for (i = block_r_new; i < s->s_blocksize * 8; i++)
                reiserfs_set_le_bit(i, bh->b_data);
-       journal_mark_dirty(&th, s, bh);
+       journal_mark_dirty(&th, bh);
        brelse(bh);
 
        info->free_count -= s->s_blocksize * 8 - block_r_new;
@@ -222,7 +222,7 @@ int reiserfs_resize(struct super_block *s, unsigned long block_count_new)
        PUT_SB_BLOCK_COUNT(s, block_count_new);
        PUT_SB_BMAP_NR(s, bmap_would_wrap(bmap_nr_new) ? : bmap_nr_new);
 
-       journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+       journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
 
        SB_JOURNAL(s)->j_must_wait = 1;
        return journal_end(&th);
index 28c593a..6d697d7 100644 (file)
@@ -1101,7 +1101,7 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th,
                    if (block != 0) {
                        reiserfs_prepare_for_journal(sb, bh, 1);
                        put_block_num(unfm, 0, 0);
-                       journal_mark_dirty(th, sb, bh);
+                       journal_mark_dirty(th, bh);
                        reiserfs_free_block(th, inode, block, 1);
                    }
 
index 4d13cff..fa65179 100644 (file)
@@ -136,7 +136,7 @@ static int reiserfs_freeze(struct super_block *s)
                } else {
                        reiserfs_prepare_for_journal(s, SB_BUFFER_WITH_SB(s),
                                                     1);
-                       journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+                       journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
                        reiserfs_block_writes(&th);
                        journal_end_sync(&th);
                }
@@ -551,7 +551,7 @@ static void reiserfs_put_super(struct super_block *s)
                                                     1);
                        set_sb_umount_state(SB_DISK_SUPER_BLOCK(s),
                                            REISERFS_SB(s)->s_mount_state);
-                       journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+                       journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
                }
        }
 
@@ -1477,7 +1477,7 @@ static int reiserfs_remount(struct super_block *s, int *mount_flags, char *arg)
                /* Mounting a rw partition read-only. */
                reiserfs_prepare_for_journal(s, SB_BUFFER_WITH_SB(s), 1);
                set_sb_umount_state(rs, REISERFS_SB(s)->s_mount_state);
-               journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+               journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
        } else {
                /* remount read-write */
                if (!(s->s_flags & MS_RDONLY)) {
@@ -1509,7 +1509,7 @@ static int reiserfs_remount(struct super_block *s, int *mount_flags, char *arg)
                if (!old_format_only(s))
                        set_sb_mnt_count(rs, sb_mnt_count(rs) + 1);
                /* mark_buffer_dirty (SB_BUFFER_WITH_SB (s), 1); */
-               journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+               journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
                REISERFS_SB(s)->s_mount_state = REISERFS_VALID_FS;
        }
        /* this will force a full flush of all journal lists */
@@ -2106,7 +2106,7 @@ static int reiserfs_fill_super(struct super_block *s, void *data, int silent)
                        set_sb_mnt_count(rs, sb_mnt_count(rs) + 1);
 
 
-               journal_mark_dirty(&th, s, SB_BUFFER_WITH_SB(s));
+               journal_mark_dirty(&th, SB_BUFFER_WITH_SB(s));
                errval = journal_end(&th);
                if (errval) {
                        dput(s->s_root);
@@ -2481,7 +2481,7 @@ static ssize_t reiserfs_quota_write(struct super_block *sb, int type,
                unlock_buffer(bh);
                reiserfs_write_lock(sb);
                reiserfs_prepare_for_journal(sb, bh, 1);
-               journal_mark_dirty(current->journal_info, sb, bh);
+               journal_mark_dirty(current->journal_info, bh);
                if (!journal_quota)
                        reiserfs_add_ordered_list(inode, bh);
                reiserfs_write_unlock(sb);