Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752735AbbB0Jn7 (ORCPT ); Fri, 27 Feb 2015 04:43:59 -0500 Received: from mga03.intel.com ([134.134.136.65]:15991 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752493AbbB0Jn5 (ORCPT ); Fri, 27 Feb 2015 04:43:57 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,658,1418112000"; d="scan'208";a="533643869" Date: Fri, 27 Feb 2015 17:25:35 +0800 From: Wanpeng Li To: Chao Yu Cc: Changman Lee , linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, Wanpeng Li Subject: Re: [f2fs-dev] [PATCH] f2fs: fix max orphan inodes calculation Message-ID: <20150227092535.GA23044@kernel> Reply-To: Wanpeng Li References: <1425020704-10355-1-git-send-email-wanpeng.li@linux.intel.com> <018001d05271$6cc02c20$46408460$@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <018001d05271$6cc02c20$46408460$@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1969 Lines: 55 Hi Chao, On Fri, Feb 27, 2015 at 05:39:39PM +0800, Chao Yu wrote: >Hi Wanpeng, > >> -----Original Message----- >> From: Wanpeng Li [mailto:wanpeng.li@linux.intel.com] >> Sent: Friday, February 27, 2015 3:05 PM >> To: Jaegeuk Kim >> Cc: Wanpeng Li; linux-kernel@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net >> Subject: [f2fs-dev] [PATCH] f2fs: fix max orphan inodes calculation >> >> cp_payload is introduced for sit bitmap to support large volume, and it is >> just after the block of f2fs_checkpoint + nat bitmap, so the first segment >> should include F2FS_CP_PACKS + NR_CURSEG_TYPE + cp_payload + orphan blocks. >> However, current max orphan inodes calculation don't consider cp_payload, >> this patch fix it by reducing the number of cp_payload from total blocks of >> the first segment when calculate max orphan inodes. >> >> Signed-off-by: Wanpeng Li >> --- >> fs/f2fs/checkpoint.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c >> index db82e09..3c9d677 100644 >> --- a/fs/f2fs/checkpoint.c >> +++ b/fs/f2fs/checkpoint.c >> @@ -1109,7 +1109,7 @@ void init_ino_entry_info(struct f2fs_sb_info *sbi) >> * for cp pack we can have max 1020*504 orphan entries > >The description above should be wrong, can you please update it in this patch? Ok, I will adjust the comments above the codes. > >> */ >> sbi->max_orphans = (sbi->blocks_per_seg - F2FS_CP_PACKS - >> - NR_CURSEG_TYPE) * F2FS_ORPHANS_PER_BLOCK; >> + NR_CURSEG_TYPE - __cp_payload(sbi)) * F2FS_ORPHANS_PER_BLOCK; > >There is a trivial coding style issue: "line over 80 characters" Will fix it. Regards, Wanpeng Li > >Thanks, > -- 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/