Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753300Ab3HBB2O (ORCPT ); Thu, 1 Aug 2013 21:28:14 -0400 Received: from mail-pb0-f54.google.com ([209.85.160.54]:33127 "EHLO mail-pb0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751061Ab3HBB2N (ORCPT ); Thu, 1 Aug 2013 21:28:13 -0400 MIME-Version: 1.0 In-Reply-To: <51F9D407.8040703@cn.fujitsu.com> References: <1375281193-15560-1-git-send-email-linkinjeon@gmail.com> <51F9D407.8040703@cn.fujitsu.com> Date: Fri, 2 Aug 2013 10:28:12 +0900 Message-ID: Subject: Re: [PATCH 2/2] f2fs: add sysfs entries to select the gc policy From: Namjae Jeon 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 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: 3329 Lines: 90 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! > >> + >> >> -- 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/