ocfs2: clean dead code in suballoc.c (dd7b5f9d) · Commits · 戴 / test

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

+3 −3

Original line number	Diff line number	Diff line
		@@ -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);

Admin message