The patch titled ext4: error processing and coding enhancement for mballoc has been added to the -mm tree. Its filename is ext4-error-processing-and-coding-enhancement-for-mballoc.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: ext4: error processing and coding enhancement for mballoc From: Shen Feng <shen@xxxxxxxxxxxxxx> *ext4_mb_seq_history_open: check if sbi->s_mb_history is NULL *ext4_mb_history_init: replace kmalloc and memset with kzalloc *ext4_mb_init_backend: remove memset since kzalloc is used *ext4_mb_init: the return value of ext4_mb_init_backend is int, but i is unsigned, replace it with a new int variable. Signed-off-by: Shen Feng <shen@xxxxxxxxxxxxxx> Reviewed-by: Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx> Cc: Mingming Cao <cmm@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/ext4/mballoc.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff -puN fs/ext4/mballoc.c~ext4-error-processing-and-coding-enhancement-for-mballoc fs/ext4/mballoc.c --- a/fs/ext4/mballoc.c~ext4-error-processing-and-coding-enhancement-for-mballoc +++ a/fs/ext4/mballoc.c @@ -1969,6 +1969,8 @@ static int ext4_mb_seq_history_open(stru int rc; int size; + if (unlikely(sbi->s_mb_history == NULL)) + return -ENOMEM; s = kmalloc(sizeof(*s), GFP_KERNEL); if (s == NULL) return -ENOMEM; @@ -2171,9 +2173,7 @@ static void ext4_mb_history_init(struct sbi->s_mb_history_cur = 0; spin_lock_init(&sbi->s_mb_history_lock); i = sbi->s_mb_history_max * sizeof(struct ext4_mb_history); - sbi->s_mb_history = kmalloc(i, GFP_KERNEL); - if (likely(sbi->s_mb_history != NULL)) - memset(sbi->s_mb_history, 0, i); + sbi->s_mb_history = kzalloc(i, GFP_KERNEL); /* if we can't allocate history, then we simple won't use it */ } @@ -2287,7 +2287,6 @@ static int ext4_mb_init_backend(struct s i++; goto err_freebuddy; } - memset(meta_group_info[j], 0, len); set_bit(EXT4_GROUP_INFO_NEED_INIT_BIT, &(meta_group_info[j]->bb_state)); @@ -2342,6 +2341,7 @@ int ext4_mb_init(struct super_block *sb, unsigned i; unsigned offset; unsigned max; + int ret; if (!test_opt(sb, MBALLOC)) return 0; @@ -2376,12 +2376,12 @@ int ext4_mb_init(struct super_block *sb, } while (i <= sb->s_blocksize_bits + 1); /* init file for buddy data */ - i = ext4_mb_init_backend(sb); - if (i) { + ret = ext4_mb_init_backend(sb); + if (ret != 0) { clear_opt(sbi->s_mount_opt, MBALLOC); kfree(sbi->s_mb_offsets); kfree(sbi->s_mb_maxs); - return i; + return ret; } spin_lock_init(&sbi->s_md_lock); _ Patches currently in -mm which might be from shen@xxxxxxxxxxxxxx are linux-next.patch ext3-improve-some-code-in-rb-tree-part-of-dirc.patch ext4-error-proc-entry-creation-when-the-fs-ext4-is-not-correctly-created.patch ext4-improve-some-code-in-rb-tree-part-of-dirc.patch ext4-remove-double-definitions-of-xattr-macros.patch ext4-error-processing-and-coding-enhancement-for-mballoc.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