Ocfs2/move_extents: Validate moving goal after the adjustment.
authorTristan Ye <tristan.ye@oracle.com>
Fri, 27 May 2011 07:24:14 +0000 (15:24 +0800)
committerTristan Ye <tristan.ye@oracle.com>
Fri, 27 May 2011 06:52:57 +0000 (14:52 +0800)
though the goal_to_be_moved will be validated again in following moving, it's
still a good idea to validate it after adjustment at the very beginning, instead
of validating it before adjustment.

Signed-off-by: Tristan Ye <tristan.ye@oracle.com>
fs/ocfs2/move_extents.c

index d184e0b4b95fe8c17de0b658de576af35db881c9..cd9427023d2e72dd1d5109113569f7dee6eb1066 100644 (file)
@@ -472,11 +472,23 @@ static int ocfs2_validate_and_adjust_move_goal(struct inode *inode,
        int ret, goal_bit = 0;
 
        struct buffer_head *gd_bh = NULL;
-       struct ocfs2_group_desc *bg;
+       struct ocfs2_group_desc *bg = NULL;
        struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
        int c_to_b = 1 << (osb->s_clustersize_bits -
                                        inode->i_sb->s_blocksize_bits);
 
+       /*
+        * make goal become cluster aligned.
+        */
+       range->me_goal = ocfs2_block_to_cluster_start(inode->i_sb,
+                                                     range->me_goal);
+       /*
+        * moving goal is not allowd to start with a group desc blok(#0 blk)
+        * let's compromise to the latter cluster.
+        */
+       if (range->me_goal == le64_to_cpu(bg->bg_blkno))
+               range->me_goal += c_to_b;
+
        /*
         * validate goal sits within global_bitmap, and return the victim
         * group desc
@@ -490,18 +502,6 @@ static int ocfs2_validate_and_adjust_move_goal(struct inode *inode,
 
        bg = (struct ocfs2_group_desc *)gd_bh->b_data;
 
-       /*
-        * make goal become cluster aligned.
-        */
-       range->me_goal = ocfs2_block_to_cluster_start(inode->i_sb,
-                                                     range->me_goal);
-       /*
-        * moving goal is not allowd to start with a group desc blok(#0 blk)
-        * let's compromise to the latter cluster.
-        */
-       if (range->me_goal == le64_to_cpu(bg->bg_blkno))
-               range->me_goal += c_to_b;
-
        /*
         * movement is not gonna cross two groups.
         */