Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753579Ab3C1AFh (ORCPT ); Wed, 27 Mar 2013 20:05:37 -0400 Received: from mail-da0-f50.google.com ([209.85.210.50]:62145 "EHLO mail-da0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751659Ab3C1AFf (ORCPT ); Wed, 27 Mar 2013 20:05:35 -0400 MIME-Version: 1.0 In-Reply-To: <1364428072-7301-1-git-send-email-gheorghiuandru@gmail.com> References: <1364428072-7301-1-git-send-email-gheorghiuandru@gmail.com> Date: Thu, 28 Mar 2013 09:05:35 +0900 Message-ID: Subject: Re: [PATCH] fs: f2fs: Use kmemdup From: Namjae Jeon To: Alexandru Gheorghiu Cc: Jaegeuk Kim , linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2246 Lines: 68 2013/3/28, Alexandru Gheorghiu : > Used kmemdup instead of kzalloc and memcpy. > > Signed-off-by: Alexandru Gheorghiu > --- > fs/f2fs/node.c | 11 ++++------- > fs/f2fs/segment.c | 3 +-- > 2 files changed, 5 insertions(+), 9 deletions(-) > > diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c > index e275218..920f53a 100644 > --- a/fs/f2fs/node.c > +++ b/fs/f2fs/node.c > @@ -1661,19 +1661,16 @@ static int init_node_manager(struct f2fs_sb_info > *sbi) > spin_lock_init(&nm_i->free_nid_list_lock); > rwlock_init(&nm_i->nat_tree_lock); > > - nm_i->bitmap_size = __bitmap_size(sbi, NAT_BITMAP); > nm_i->init_scan_nid = le32_to_cpu(sbi->ckpt->next_free_nid); > nm_i->next_scan_nid = le32_to_cpu(sbi->ckpt->next_free_nid); > - > - nm_i->nat_bitmap = kzalloc(nm_i->bitmap_size, GFP_KERNEL); > - if (!nm_i->nat_bitmap) > - return -ENOMEM; > + nm_i->bitmap_size = __bitmap_size(sbi, NAT_BITMAP); > version_bitmap = __bitmap_ptr(sbi, NAT_BITMAP); > if (!version_bitmap) > return -EFAULT; > > - /* copy version bitmap */ > - memcpy(nm_i->nat_bitmap, version_bitmap, nm_i->bitmap_size); > + nm_i->nat_bitmap = kmemdup(version_bitmap, nm_i->bitmap_size, > GFP_KERNEL); maybe, this line is over 80 characters. plz fix it after running checkpatch.pl. Thanks. > + if (!nm_i->nat_bitmap) > + return -ENOMEM; > return 0; > } > > diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c > index 777f17e..1758149 100644 > --- a/fs/f2fs/segment.c > +++ b/fs/f2fs/segment.c > @@ -1403,10 +1403,9 @@ static int build_sit_info(struct f2fs_sb_info *sbi) > bitmap_size = __bitmap_size(sbi, SIT_BITMAP); > src_bitmap = __bitmap_ptr(sbi, SIT_BITMAP); > > - dst_bitmap = kzalloc(bitmap_size, GFP_KERNEL); > + dst_bitmap = kmemdup(src_bitmap, bitmap_size, GFP_KERNEL); > if (!dst_bitmap) > return -ENOMEM; > - memcpy(dst_bitmap, src_bitmap, bitmap_size); > > /* init SIT information */ > sit_i->s_ops = &default_salloc_ops; > -- > 1.7.9.5 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/