Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753790AbbGBMkM (ORCPT ); Thu, 2 Jul 2015 08:40:12 -0400 Received: from col004-omc1s2.hotmail.com ([65.55.34.12]:60130 "EHLO COL004-OMC1S2.hotmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753770AbbGBMjs (ORCPT ); Thu, 2 Jul 2015 08:39:48 -0400 X-TMN: [VNljJmOe1mDrHBGsdilg08jF+I3+Db29] X-Originating-Email: [yuchaochina@hotmail.com] Message-ID: From: Chao Yu To: "'Jaegeuk Kim'" CC: , , References: <1435713946-6910-1-git-send-email-jaegeuk@kernel.org> <1435713946-6910-2-git-send-email-jaegeuk@kernel.org> In-Reply-To: <1435713946-6910-2-git-send-email-jaegeuk@kernel.org> Subject: RE: [f2fs-dev] [PATCH 2/2] f2fs: shrink unreferenced extent_caches first Date: Thu, 2 Jul 2015 20:39:43 +0800 MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 15.0 Thread-index: AQABAgME4JsGE396BuhbuOjqYqX77aFncDoA Content-Language: zh-cn X-OriginalArrivalTime: 02 Jul 2015 12:39:47.0976 (UTC) FILETIME=[2E751880:01D0B4C4] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1873 Lines: 51 > -----Original Message----- > From: Jaegeuk Kim [mailto:jaegeuk@kernel.org] > Sent: Wednesday, July 01, 2015 9:26 AM > To: linux-kernel@vger.kernel.org; linux-fsdevel@vger.kernel.org; > linux-f2fs-devel@lists.sourceforge.net; linux-kernel@vger.kernel.org; > linux-fsdevel@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net > Cc: Jaegeuk Kim; Jaegeuk Kim > Subject: [f2fs-dev] [PATCH 2/2] f2fs: shrink unreferenced extent_caches first > > If an extent_tree entry has a zero reference count, we can drop it from the > cache in higher priority rather than currently referencing entries. > > Signed-off-by: Jaegeuk Kim > --- > fs/f2fs/data.c | 51 +++++++++++++++++++++++++++++++++++++++++---------- > 1 file changed, 41 insertions(+), 10 deletions(-) > > diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c > index 8f059e0..a0a0e2b 100644 > --- a/fs/f2fs/data.c > +++ b/fs/f2fs/data.c > @@ -661,21 +661,54 @@ unsigned int f2fs_shrink_extent_tree(struct f2fs_sb_info *sbi, int > nr_shrink) > struct radix_tree_root *root = &sbi->extent_tree_root; > unsigned int found; > unsigned int node_cnt = 0, tree_cnt = 0; > + int remained; > > if (!test_opt(sbi, EXTENT_CACHE)) > return 0; > > + if (!down_write_trylock(&sbi->extent_tree_lock)) > + goto out; > + > + /* 1. remove unreferenced extent tree */ We always release extent node and tree from inode with lowest ino, why not random ino? And this step releasing breaks the rule of lru runs. Some unreferenced file has high hit ratio and some referenced file may have low hit ratio. Why not release low hit ratio extent tree at first? 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/