The patch titled mm/hugetlb.c: fix duplicate variable has been added to the -mm tree. Its filename is mm-hugetlbc-fix-duplicate-variable.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 *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: mm/hugetlb.c: fix duplicate variable From: Adrian Bunk <bunk@xxxxxxxxxx> It's confusing that set_max_huge_pages() contained two different variables named "ret", and although the code works correctly this should be fixed. The inner of the two variables can simply be removed. Spotted by sparse. Signed-off-by: Adrian Bunk <bunk@xxxxxxxxxx> Cc: "KOSAKI Motohiro" <kosaki.motohiro@xxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/hugetlb.c | 1 - 1 file changed, 1 deletion(-) diff -puN mm/hugetlb.c~mm-hugetlbc-fix-duplicate-variable mm/hugetlb.c --- a/mm/hugetlb.c~mm-hugetlbc-fix-duplicate-variable +++ a/mm/hugetlb.c @@ -606,7 +606,6 @@ static unsigned long set_max_huge_pages( } while (count > persistent_huge_pages) { - int ret; /* * If this allocation races such that we no longer need the * page, free_huge_page will handle it by freeing the page _ Patches currently in -mm which might be from bunk@xxxxxxxxxx are origin.patch linux-next.patch md-make-dm_dirty_log_initexit-static.patch drm-make-drm_minors_cleanup-static.patch git-udf.patch nfs-make-nfs4_drop_state_owner-static.patch show_schedstat-fix-memleak.patch if-0-ses_match_host.patch git-watchdog.patch xen-drivers-xen-balloonc-make-a-function-static.patch reiser4.patch remove-is_tty.patch make-cgroup_seqfile_release-static.patch make-kprobe_blacklist-static.patch make-pnp_add_card_id-static.patch mfd-sm501c-if-0-unused-functions.patch pcmcia-kill-in_card_services.patch video-sis-remove-compat-code.patch make-struct-mpt_proc_root_dir-static.patch make-parport_cs_release-static.patch parport-sharec-proper-externs.patch remove-include-asm-h8300-keyboardh.patch proper-spawn_ksoftirqd-prototype.patch proper-pidhashmap_init-prototypes.patch mm-migratec-should-include-linux-syscallsh.patch make-mm-sparsec-make-a-function-static.patch mm-allocpercpuc-make-4-functions-static.patch make-mm-memoryc-print_bad_pte-static.patch mm-swapfilec-make-code-static.patch make-mm-rmapc-anon_vma_cachep-static.patch minix-remove-no_truncate-code.patch remove-the-v850-port.patch mm-vmstatc-proper-externs.patch mm-hugetlbc-fix-duplicate-variable.patch drivers-char-rtcc-make-2-functions-static.patch make-init-do_mountsc-root_device_name-static.patch init-do_mountsc-should-include-linux-initrdh.patch proper-prototype-for-acpi_processor_tstate_has_changed.patch if-0-hpet_unregister.patch tpm_biosc-make-2-structs-static.patch proper-extern-for-mwave_s_mdd.patch xtensa-remove-the-nonexisting-highmem-support.patch serial-8250_gscc-add-module_license.patch move-proc_kmsg_operations-to-fs-proc-internalh.patch unexport-proc_clear_tty.patch unexport-uts_sem.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