From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.sourceforge.net (lists.sourceforge.net [216.105.38.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1588EC4828F for ; Thu, 8 Feb 2024 00:21:36 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-4.v29.lw.sourceforge.com) by sfs-ml-4.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from ) id 1rXsAZ-0008JQ-Em; Thu, 08 Feb 2024 00:21:35 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-4.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1rXsAX-0008JK-Pe for linux-f2fs-devel@lists.sourceforge.net; Thu, 08 Feb 2024 00:21:33 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=In-Reply-To:Content-Type:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=QA/rT+r0d7e/tKYEeCuBVJtptferpHBN9KG5yu9pMX0=; b=PcyA2VfBglL7qOckjMDswPtw5A Jr08lGoaEWOIZz9Bh704pRmUqlK21kSvYVkX1Lmw2pHCekuBsavfRJOSrGaxOu9ZInpB3hOZodgZa L0F3kbhsizDB7DLqSzbNrZoLyNaBuus+B6RVVKp2dWV8BtHO8KwDuJe0qH8mtG/FLCjI=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To :From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=QA/rT+r0d7e/tKYEeCuBVJtptferpHBN9KG5yu9pMX0=; b=Ze4yC/pxTSqe/fuexxrW81Nx4f xFTjhbTPAowOXa1QiZawQJf4UtHhP/fM2TT7s0nKLMc2Arod550CjBQzr5sXWBZAcWQqgvtUKz75L epYo+Rw2roNmTflOiqnUM6JK0vYd+ZpbqKGIFs7TS8BIMY8SPrA3S3o91E6v1FBuxZ2A=; Received: from dfw.source.kernel.org ([139.178.84.217]) by sfi-mx-2.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.95) id 1rXsAW-0002Ad-7M for linux-f2fs-devel@lists.sourceforge.net; Thu, 08 Feb 2024 00:21:33 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 7F43661B09; Thu, 8 Feb 2024 00:21:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F3649C433C7; Thu, 8 Feb 2024 00:21:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1707351682; bh=ib1sOSEkwlcTgn6HtFduVrZxSUlkTx/HR6aWFwOhyE4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FPv10eP+1Kuvx108Ty5qI0u4aD60WvLRHAkucg9+OPZuUkqlPpPmLpXubvi0FWpva fog7/UhT9NLyuzCmvXEYdfcwXFX5H/NguG+rNqSBt/KYCZ0f6oB64OqkiEXhVWIcQy FBZzuJI629UqeuxI/e3tNVJROXTaI4pj2y7v2URkLjpVLga1q6DtF8DSyMbkYIKy78 MaXpN3cZUsh3xhwj0QNoqkOefyfu9H2LckwV+UC+rNVr5csHkWjGbSa/+9huTxf6Jz uvIEXxx5QRRvuEO5dfDosh9IsfGRnsm+2Fro14EVV9rUb3jhaza7ChkH85R5L2tKpL y6op9hBt52KwA== Date: Wed, 7 Feb 2024 16:21:20 -0800 From: Jaegeuk Kim To: liujinbao1 Message-ID: References: <3b2852b3d404ecbb53d9affa781d12d0e9ea3951.1707022643.git.liujinbao1@xiaomi.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <3b2852b3d404ecbb53d9affa781d12d0e9ea3951.1707022643.git.liujinbao1@xiaomi.com> X-Headers-End: 1rXsAW-0002Ad-7M Subject: Re: [f2fs-dev] [PATCH v3] f2fs: sysfs: support gc_io_aware X-BeenThere: linux-f2fs-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: liujinbao1 , linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net On 02/04, liujinbao1 wrote: > From: liujinbao1 > > Currently, IO can only be ignored when GC_URGENT_HIGH is set, > and the default algorithm used for GC_URGENT_HIGH is greedy. > It gives a way to enable/disable IO aware feature for background > gc, so that we can tune background gc more precisely. e.g. > force to disable IO aware and choose more suitable algorithm > if there are large number of dirty segments. > > Signed-off-by: liujinbao1 > --- > Documentation/ABI/testing/sysfs-fs-f2fs | 6 ++++++ > fs/f2fs/gc.c | 3 ++- > fs/f2fs/gc.h | 1 + > fs/f2fs/sysfs.c | 9 +++++++++ > 4 files changed, 18 insertions(+), 1 deletion(-) > > diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs b/Documentation/ABI/testing/sysfs-fs-f2fs > index 36c3cb547901..47f02fa471fe 100644 > --- a/Documentation/ABI/testing/sysfs-fs-f2fs > +++ b/Documentation/ABI/testing/sysfs-fs-f2fs > @@ -16,6 +16,12 @@ Contact: "Namjae Jeon" > Description: Controls the default sleep time for gc_thread. Time > is in milliseconds. > > +What: /sys/fs/f2fs//gc_io_aware > +Date: January 2024 > +Contact: "Jinbao Liu" > +Description: It controls to enable/disable IO aware feature for background gc. > ++ By default, the value is 1 which indicates IO aware is on. > + > What: /sys/fs/f2fs//gc_idle > Date: July 2013 > Contact: "Namjae Jeon" > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index 309da3d0faff..0b75d37acc63 100644 > --- a/fs/f2fs/gc.c > +++ b/fs/f2fs/gc.c > @@ -109,7 +109,7 @@ static int gc_thread_func(void *data) > goto next; > } > > - if (!is_idle(sbi, GC_TIME)) { > + if (gc_th->io_aware && !is_idle(sbi, GC_TIME)) { This voids GC_URGENT_HIGH, which looks wrong? > increase_sleep_time(gc_th, &wait_ms); > f2fs_up_write(&sbi->gc_lock); > stat_io_skip_bggc_count(sbi); > @@ -182,6 +182,7 @@ int f2fs_start_gc_thread(struct f2fs_sb_info *sbi) > gc_th->min_sleep_time = DEF_GC_THREAD_MIN_SLEEP_TIME; > gc_th->max_sleep_time = DEF_GC_THREAD_MAX_SLEEP_TIME; > gc_th->no_gc_sleep_time = DEF_GC_THREAD_NOGC_SLEEP_TIME; > + gc_th->io_aware = true; > > gc_th->gc_wake = false; > > diff --git a/fs/f2fs/gc.h b/fs/f2fs/gc.h > index 28a00942802c..51d6ad26b76a 100644 > --- a/fs/f2fs/gc.h > +++ b/fs/f2fs/gc.h > @@ -41,6 +41,7 @@ struct f2fs_gc_kthread { > unsigned int min_sleep_time; > unsigned int max_sleep_time; > unsigned int no_gc_sleep_time; > + bool io_aware; > > /* for changing gc mode */ > bool gc_wake; > diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c > index 417fae96890f..e8d5667cfddd 100644 > --- a/fs/f2fs/sysfs.c > +++ b/fs/f2fs/sysfs.c > @@ -516,6 +516,13 @@ static ssize_t __sbi_store(struct f2fs_attr *a, > return count; > } > > + if (!strcmp(a->attr.name, "gc_io_aware")) { > + if (t > 1) > + return -EINVAL; > + *ui = t ? true : false; > + return count; > + } > + > if (!strcmp(a->attr.name, "migration_granularity")) { > if (t == 0 || t > sbi->segs_per_sec) > return -EINVAL; > @@ -906,6 +913,7 @@ GC_THREAD_RW_ATTR(gc_urgent_sleep_time, urgent_sleep_time); > GC_THREAD_RW_ATTR(gc_min_sleep_time, min_sleep_time); > GC_THREAD_RW_ATTR(gc_max_sleep_time, max_sleep_time); > GC_THREAD_RW_ATTR(gc_no_gc_sleep_time, no_gc_sleep_time); > +GC_THREAD_RW_ATTR(gc_io_aware, io_aware); > > /* SM_INFO ATTR */ > SM_INFO_RW_ATTR(reclaim_segments, rec_prefree_segments); > @@ -1061,6 +1069,7 @@ static struct attribute *f2fs_attrs[] = { > ATTR_LIST(gc_min_sleep_time), > ATTR_LIST(gc_max_sleep_time), > ATTR_LIST(gc_no_gc_sleep_time), > + ATTR_LIST(gc_io_aware), > ATTR_LIST(gc_idle), > ATTR_LIST(gc_urgent), > ATTR_LIST(reclaim_segments), > -- > 2.43.0 _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9D11C11737 for ; Thu, 8 Feb 2024 00:21:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707351682; cv=none; b=kXFom+CS+dNRhL19qH0gZeeHi5lQoJBg1nxFfgc70vcTegDVuWkHOxHYV8gLUlWrOcVdPeXh7RG2zrpQJrPAtaDy4DzUi2funB1mlr89BakSfl8aOvu7Mz4rC+aOso7GrloWqLK/dOgiCNSkwNZh72n4e2W5Abgn6+LqprHHknc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707351682; c=relaxed/simple; bh=ib1sOSEkwlcTgn6HtFduVrZxSUlkTx/HR6aWFwOhyE4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=eftSOvxf+oEOc2QGi3O6qNzKKdDDwaKKK9dRzjxrPgWX/W/ROCBJaKRouckG6jfwoR9OF2S8l03VsqaHx7sgZANUrK5vaJ5LwJbsgWyMp7CZkl284ejGqyDOmZgDZ6Tr3kcxEtVhiiY9M1BFcnNUFtpxAJmj1BF2hd0mcfZXjxI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=FPv10eP+; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="FPv10eP+" Received: by smtp.kernel.org (Postfix) with ESMTPSA id F3649C433C7; Thu, 8 Feb 2024 00:21:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1707351682; bh=ib1sOSEkwlcTgn6HtFduVrZxSUlkTx/HR6aWFwOhyE4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FPv10eP+1Kuvx108Ty5qI0u4aD60WvLRHAkucg9+OPZuUkqlPpPmLpXubvi0FWpva fog7/UhT9NLyuzCmvXEYdfcwXFX5H/NguG+rNqSBt/KYCZ0f6oB64OqkiEXhVWIcQy FBZzuJI629UqeuxI/e3tNVJROXTaI4pj2y7v2URkLjpVLga1q6DtF8DSyMbkYIKy78 MaXpN3cZUsh3xhwj0QNoqkOefyfu9H2LckwV+UC+rNVr5csHkWjGbSa/+9huTxf6Jz uvIEXxx5QRRvuEO5dfDosh9IsfGRnsm+2Fro14EVV9rUb3jhaza7ChkH85R5L2tKpL y6op9hBt52KwA== Date: Wed, 7 Feb 2024 16:21:20 -0800 From: Jaegeuk Kim To: liujinbao1 Cc: chao@kernel.org, linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, liujinbao1 Subject: Re: [PATCH v3] f2fs: sysfs: support gc_io_aware Message-ID: References: <3b2852b3d404ecbb53d9affa781d12d0e9ea3951.1707022643.git.liujinbao1@xiaomi.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3b2852b3d404ecbb53d9affa781d12d0e9ea3951.1707022643.git.liujinbao1@xiaomi.com> On 02/04, liujinbao1 wrote: > From: liujinbao1 > > Currently, IO can only be ignored when GC_URGENT_HIGH is set, > and the default algorithm used for GC_URGENT_HIGH is greedy. > It gives a way to enable/disable IO aware feature for background > gc, so that we can tune background gc more precisely. e.g. > force to disable IO aware and choose more suitable algorithm > if there are large number of dirty segments. > > Signed-off-by: liujinbao1 > --- > Documentation/ABI/testing/sysfs-fs-f2fs | 6 ++++++ > fs/f2fs/gc.c | 3 ++- > fs/f2fs/gc.h | 1 + > fs/f2fs/sysfs.c | 9 +++++++++ > 4 files changed, 18 insertions(+), 1 deletion(-) > > diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs b/Documentation/ABI/testing/sysfs-fs-f2fs > index 36c3cb547901..47f02fa471fe 100644 > --- a/Documentation/ABI/testing/sysfs-fs-f2fs > +++ b/Documentation/ABI/testing/sysfs-fs-f2fs > @@ -16,6 +16,12 @@ Contact: "Namjae Jeon" > Description: Controls the default sleep time for gc_thread. Time > is in milliseconds. > > +What: /sys/fs/f2fs//gc_io_aware > +Date: January 2024 > +Contact: "Jinbao Liu" > +Description: It controls to enable/disable IO aware feature for background gc. > ++ By default, the value is 1 which indicates IO aware is on. > + > What: /sys/fs/f2fs//gc_idle > Date: July 2013 > Contact: "Namjae Jeon" > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > index 309da3d0faff..0b75d37acc63 100644 > --- a/fs/f2fs/gc.c > +++ b/fs/f2fs/gc.c > @@ -109,7 +109,7 @@ static int gc_thread_func(void *data) > goto next; > } > > - if (!is_idle(sbi, GC_TIME)) { > + if (gc_th->io_aware && !is_idle(sbi, GC_TIME)) { This voids GC_URGENT_HIGH, which looks wrong? > increase_sleep_time(gc_th, &wait_ms); > f2fs_up_write(&sbi->gc_lock); > stat_io_skip_bggc_count(sbi); > @@ -182,6 +182,7 @@ int f2fs_start_gc_thread(struct f2fs_sb_info *sbi) > gc_th->min_sleep_time = DEF_GC_THREAD_MIN_SLEEP_TIME; > gc_th->max_sleep_time = DEF_GC_THREAD_MAX_SLEEP_TIME; > gc_th->no_gc_sleep_time = DEF_GC_THREAD_NOGC_SLEEP_TIME; > + gc_th->io_aware = true; > > gc_th->gc_wake = false; > > diff --git a/fs/f2fs/gc.h b/fs/f2fs/gc.h > index 28a00942802c..51d6ad26b76a 100644 > --- a/fs/f2fs/gc.h > +++ b/fs/f2fs/gc.h > @@ -41,6 +41,7 @@ struct f2fs_gc_kthread { > unsigned int min_sleep_time; > unsigned int max_sleep_time; > unsigned int no_gc_sleep_time; > + bool io_aware; > > /* for changing gc mode */ > bool gc_wake; > diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c > index 417fae96890f..e8d5667cfddd 100644 > --- a/fs/f2fs/sysfs.c > +++ b/fs/f2fs/sysfs.c > @@ -516,6 +516,13 @@ static ssize_t __sbi_store(struct f2fs_attr *a, > return count; > } > > + if (!strcmp(a->attr.name, "gc_io_aware")) { > + if (t > 1) > + return -EINVAL; > + *ui = t ? true : false; > + return count; > + } > + > if (!strcmp(a->attr.name, "migration_granularity")) { > if (t == 0 || t > sbi->segs_per_sec) > return -EINVAL; > @@ -906,6 +913,7 @@ GC_THREAD_RW_ATTR(gc_urgent_sleep_time, urgent_sleep_time); > GC_THREAD_RW_ATTR(gc_min_sleep_time, min_sleep_time); > GC_THREAD_RW_ATTR(gc_max_sleep_time, max_sleep_time); > GC_THREAD_RW_ATTR(gc_no_gc_sleep_time, no_gc_sleep_time); > +GC_THREAD_RW_ATTR(gc_io_aware, io_aware); > > /* SM_INFO ATTR */ > SM_INFO_RW_ATTR(reclaim_segments, rec_prefree_segments); > @@ -1061,6 +1069,7 @@ static struct attribute *f2fs_attrs[] = { > ATTR_LIST(gc_min_sleep_time), > ATTR_LIST(gc_max_sleep_time), > ATTR_LIST(gc_no_gc_sleep_time), > + ATTR_LIST(gc_io_aware), > ATTR_LIST(gc_idle), > ATTR_LIST(gc_urgent), > ATTR_LIST(reclaim_segments), > -- > 2.43.0