ocfs2: clean dead code in suballoc.c
authorChangwei Ge <ge.changwei@h3c.com>
Thu, 1 Feb 2018 00:14:40 +0000 (16:14 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 1 Feb 2018 01:18:35 +0000 (17:18 -0800)
Stack variable fe is no longer used, so trim it to save some CPU cycles
and stack space.

Link: http://lkml.kernel.org/r/63ADC13FD55D6546B7DECE290D39E373F1F5A8DD@H3CMLB14-EX.srv.huawei-3com.com
Signed-off-by: Changwei Ge <ge.changwei@h3c.com>
Reviewed-by: Joseph Qi <jiangqi903@gmail.com>
Cc: Mark Fasheh <mfasheh@versity.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Cc: Changwei Ge <ge.changwei@h3c.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/suballoc.c

index 9f0b95a..2d8d31c 100644 (file)
@@ -2563,16 +2563,16 @@ static int _ocfs2_free_clusters(handle_t *handle,
        int status;
        u16 bg_start_bit;
        u64 bg_blkno;
-       struct ocfs2_dinode *fe;
 
        /* You can't ever have a contiguous set of clusters
         * bigger than a block group bitmap so we never have to worry
         * about looping on them.
         * This is expensive. We can safely remove once this stuff has
         * gotten tested really well. */
-       BUG_ON(start_blk != ocfs2_clusters_to_blocks(bitmap_inode->i_sb, ocfs2_blocks_to_clusters(bitmap_inode->i_sb, start_blk)));
+       BUG_ON(start_blk != ocfs2_clusters_to_blocks(bitmap_inode->i_sb,
+                               ocfs2_blocks_to_clusters(bitmap_inode->i_sb,
+                                                        start_blk)));
 
-       fe = (struct ocfs2_dinode *) bitmap_bh->b_data;
 
        ocfs2_block_to_cluster_group(bitmap_inode, start_blk, &bg_blkno,
                                     &bg_start_bit);