From mboxrd@z Thu Jan 1 00:00:00 1970 From: Namjae Jeon Subject: Re: [PATCH 2/2] f2fs: add sysfs entries to select the gc policy Date: Fri, 2 Aug 2013 10:28:12 +0900 Message-ID: References: <1375281193-15560-1-git-send-email-linkinjeon@gmail.com> <51F9D407.8040703@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <51F9D407.8040703@cn.fujitsu.com> Sender: linux-kernel-owner@vger.kernel.org To: Gu Zheng Cc: jaegeuk.kim@samsung.com, linux-f2fs-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Namjae Jeon , Pankaj Kumar List-Id: linux-f2fs-devel.lists.sourceforge.net 2013/8/1, Gu Zheng : > Hi Jeon, > > On 07/31/2013 10:33 PM, Namjae Jeon wrote: > >> From: Namjae Jeon >> >> Add sysfs entries namely gc_long_idle and gc_short_idle to control the >> gc policy. Where long idle corresponds to selecting a cost benefit >> approach, >> while short idle corresponds to selecting a greedy approach to garbage >> collection. The selection is mutually exclusive one approach will work at >> any point. >> >> Signed-off-by: Namjae Jeon >> Signed-off-by: Pankaj Kumar >> --- >> Documentation/ABI/testing/sysfs-fs-f2fs | 12 +++++++ >> Documentation/filesystems/f2fs.txt | 8 +++++ >> fs/f2fs/gc.c | 22 ++++++++++-- >> fs/f2fs/gc.h | 4 +++ >> fs/f2fs/super.c | 59 >> +++++++++++++++++++++++++++++-- >> 5 files changed, 99 insertions(+), 6 deletions(-) >> >> diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs >> b/Documentation/ABI/testing/sysfs-fs-f2fs >> index 5f44095..96b62ea 100644 >> --- a/Documentation/ABI/testing/sysfs-fs-f2fs >> +++ b/Documentation/ABI/testing/sysfs-fs-f2fs >> @@ -19,4 +19,16 @@ Description: >> Controls the default sleep time for gc_thread. Time >> is in milliseconds. >> >> +What: /sys/fs/f2fs//gc_long_idle >> +Date: July 2013 >> +Contact: "Namjae Jeon" >> +Description: >> + Controls the selection of gc policy. long_idle is used >> + to select the cost benefit approach for garbage collection. >> >> +What: /sys/fs/f2fs//gc_short_idle >> +Date: July 2013 >> +Contact: "Namjae Jeon" >> +Description: >> + Controls the selection of gc policy. short_idle is used >> + to select the greedy approach for garbage collection. >> diff --git a/Documentation/filesystems/f2fs.txt >> b/Documentation/filesystems/f2fs.txt >> index 2e9e873..06dd5d7 100644 >> --- a/Documentation/filesystems/f2fs.txt >> +++ b/Documentation/filesystems/f2fs.txt >> @@ -158,6 +158,14 @@ Files in /sys/fs/f2fs/ >> time for the garbage collection thread. >> Time is >> in milliseconds. >> >> + gc_long_idle This parameter controls the selection of >> cost >> + benefit approach for garbage collectoin. >> Writing >> + 1 to this file will select the cost benefit >> policy. >> + >> + gc_short_idle This parameter controls the selection of >> greedy >> + approach for the garbage collection. >> Writing 1 >> + to this file will select the greedy >> policy. > Hi Gu. > Why introduce two opposite attributes? It'll cause some confusion condition > if we > double enable/disable them. Yes, it seem possible in that you're talking. Okay, I will make one gc_idle and we can use long idle and short idle accoding to setting values. Thanks for review! > >> + >> >>