The patch titled Subject: ocfs2: clean up dead code in alloc.c has been added to the -mm tree. Its filename is ocfs2-clean-dead-code-up-in-allocc.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/ocfs2-clean-dead-code-up-in-allocc.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/ocfs2-clean-dead-code-up-in-allocc.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Changwei Ge <ge.changwei@xxxxxxx> Subject: ocfs2: clean up dead code in alloc.c Some stack variables are no longer used but still assigned. Trim them. Link: http://lkml.kernel.org/r/1516105069-12643-1-git-send-email-ge.changwei@xxxxxxx Signed-off-by: Changwei Ge <ge.changwei@xxxxxxx> Cc: Mark Fasheh <mfasheh@xxxxxxxxxxx> Cc: Joel Becker <jlbec@xxxxxxxxxxxx> Cc: Junxiao Bi <junxiao.bi@xxxxxxxxxx> Cc: Joseph Qi <jiangqi903@xxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/ocfs2/alloc.c | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff -puN fs/ocfs2/alloc.c~ocfs2-clean-dead-code-up-in-allocc fs/ocfs2/alloc.c --- a/fs/ocfs2/alloc.c~ocfs2-clean-dead-code-up-in-allocc +++ a/fs/ocfs2/alloc.c @@ -2598,11 +2598,8 @@ static void ocfs2_unlink_subtree(handle_ int i; struct buffer_head *root_bh = left_path->p_node[subtree_index].bh; struct ocfs2_extent_list *root_el = left_path->p_node[subtree_index].el; - struct ocfs2_extent_list *el; struct ocfs2_extent_block *eb; - el = path_leaf_el(left_path); - eb = (struct ocfs2_extent_block *)right_path->p_node[subtree_index + 1].bh->b_data; for(i = 1; i < le16_to_cpu(root_el->l_next_free_rec); i++) @@ -3938,7 +3935,7 @@ static void ocfs2_adjust_rightmost_recor struct ocfs2_path *path, struct ocfs2_extent_rec *insert_rec) { - int ret, i, next_free; + int i, next_free; struct buffer_head *bh; struct ocfs2_extent_list *el; struct ocfs2_extent_rec *rec; @@ -3955,7 +3952,6 @@ static void ocfs2_adjust_rightmost_recor ocfs2_error(ocfs2_metadata_cache_get_super(et->et_ci), "Owner %llu has a bad extent list\n", (unsigned long long)ocfs2_metadata_cache_owner(et->et_ci)); - ret = -EIO; return; } @@ -5057,7 +5053,6 @@ int ocfs2_split_extent(handle_t *handle, struct buffer_head *last_eb_bh = NULL; struct ocfs2_extent_rec *rec = &el->l_recs[split_index]; struct ocfs2_merge_ctxt ctxt; - struct ocfs2_extent_list *rightmost_el; if (le32_to_cpu(rec->e_cpos) > le32_to_cpu(split_rec->e_cpos) || ((le32_to_cpu(rec->e_cpos) + le16_to_cpu(rec->e_leaf_clusters)) < @@ -5093,9 +5088,7 @@ int ocfs2_split_extent(handle_t *handle, } eb = (struct ocfs2_extent_block *) last_eb_bh->b_data; - rightmost_el = &eb->h_list; - } else - rightmost_el = path_root_el(path); + } if (rec->e_cpos == split_rec->e_cpos && rec->e_leaf_clusters == split_rec->e_leaf_clusters) _ Patches currently in -mm which might be from ge.changwei@xxxxxxx are ocfs2-dlm-clean-dead-code-up.patch ocfs2-cluster-neaten-a-member-of-o2net_msg_handler.patch ocfs2-clean-dead-code-in-suballocc.patch ocfs2-clean-dead-code-up-in-allocc.patch ocfs2-make-metadata-estimation-accurate-and-clear.patch ocfs2-try-to-reuse-extent-block-in-dealloc-without-meta_alloc.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html