ext4: remove redundant goto in ext4_ext_insert_extent()
authorRobin Dong <sanbai@taobao.com>
Mon, 11 Jul 2011 15:43:59 +0000 (11:43 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Mon, 11 Jul 2011 15:43:59 +0000 (11:43 -0400)
If eh->eh_entries is smaller than eh->eh_max, the routine will
go to the "repeat" and then go to "has_space" directlly ,
since argument "depth" and "eh" are not even changed.

Therefore, goto "has_space" directly and remove redundant "repeat" tag.

Signed-off-by: Robin Dong <sanbai@taobao.com>
fs/ext4/extents.c

index c969ae23a535d686ea037694d374b299dab783f6..9cbdcb2110f55cdbc5b226152c823636e1cb06bc 100644 (file)
@@ -1723,7 +1723,6 @@ int ext4_ext_insert_extent(handle_t *handle, struct inode *inode,
                goto merge;
        }
 
-repeat:
        depth = ext_depth(inode);
        eh = path[depth].p_hdr;
        if (le16_to_cpu(eh->eh_entries) < le16_to_cpu(eh->eh_max))
@@ -1745,7 +1744,7 @@ repeat:
                        ext_debug("next leaf isn't full(%d)\n",
                                  le16_to_cpu(eh->eh_entries));
                        path = npath;
-                       goto repeat;
+                       goto has_space;
                }
                ext_debug("next leaf has no free space(%d,%d)\n",
                          le16_to_cpu(eh->eh_entries), le16_to_cpu(eh->eh_max));