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 60C15C38A02 for ; Fri, 28 Oct 2022 03:46:47 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-1.v29.lw.sourceforge.com) by sfs-ml-1.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from ) id 1ooGKU-0000mk-Gi; Fri, 28 Oct 2022 03:46:46 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-1.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1ooGKT-0000me-0t for linux-f2fs-devel@lists.sourceforge.net; Fri, 28 Oct 2022 03:46:45 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=Content-Transfer-Encoding:Content-Type:In-Reply-To: From:References:Cc:To:Subject:MIME-Version:Date:Message-ID:Sender:Reply-To: 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=ycO/rgbJYLuIG0sgumGEXsQNqWf9r+nzyYU9GhSvzWk=; b=KVGB0DurJfk/jS0wPOrx81DKx0 27N1aWCdkrjsW9cKLpTj6MFkNPE9Ehk1b4UUi0jdeSnpxh6wcjFS2kFiyNtT6UsQrW83h8BFmslBo YdAuSiexX89i6bh5zBtFBnsu+qY0ibfvkGcEDIO3X9Vj8hR7/iTuc8bi7aZe3je+/MFw=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Sender:Reply-To: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=ycO/rgbJYLuIG0sgumGEXsQNqWf9r+nzyYU9GhSvzWk=; b=mxY+9rpPvxgncyJUxZ5AghPTpx tuchxvCU74e8dQriQ/FPytAOLxty5lKQcHb59BpuctgEmH1UXSgG6tKzaK55LtWLwhwPmTNjl/CSb P/jxYCMiy/J2G/qjEctCsu5+xjohUkOCKuP/wLbESLqbnJoWUokT/qhyuNFL47e1OnYc=; Received: from ams.source.kernel.org ([145.40.68.75]) by sfi-mx-1.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.95) id 1ooGKO-002yn0-Vy for linux-f2fs-devel@lists.sourceforge.net; Fri, 28 Oct 2022 03:46:44 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 8A575B81FC3; Fri, 28 Oct 2022 03:46:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0E4FDC433C1; Fri, 28 Oct 2022 03:46:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1666928793; bh=j7KPU/fq3RiXoazWMj0BCPjZoCBqsdxGN8sX2nCt5/M=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=OJRKzsJ/wqUTfN6vYcngeIwiyc+BxQF4y9Q4X1apzABBY75X1ZYtdsCMWnLE9w93s YqufvM2cabCXGh0thxqT1oFkWJxgC6USnqSTW2vioYReQDN4FJEQHnCNtlff3Cx0MB M0z70g+Wy5w0LC/ao2INt4OrEmTIDtIYEv3clLt1qu3Z5YtT2rtcZVvtLiko0N5FIp xTV5EbZbxxerPMmrpI9nh0hDZS1BVfXwe/PvyGtyeV7EaGPWwczbg75uYvSac4j++i BoggPslETiUNo2xvaTc64J23N8XfCzCFc755/+dVpU6fStDZF2YCydS5MjQmHY8DRI Ic8jCNZ1g/InA== Message-ID: <5128189c-a90c-717e-45e1-9c05f51e50a6@kernel.org> Date: Fri, 28 Oct 2022 11:46:30 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.3.1 Content-Language: en-US To: Jaegeuk Kim References: <20221025065025.35017-1-frank.li@vivo.com> <20221025065025.35017-2-frank.li@vivo.com> From: Chao Yu In-Reply-To: X-Headers-End: 1ooGKO-002yn0-Vy Subject: Re: [f2fs-dev] [PATCH v2 2/2] f2fs: introducr gc_urgent_idle_remaining sysfs node and mark gc_urgent_high_remaining node deprecated 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: linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Yangtao Li Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net On 2022/10/28 11:22, Jaegeuk Kim wrote: > On 10/28, Chao Yu wrote: >> On 2022/10/25 14:50, Yangtao Li wrote: >>> Added a new sysfs node called gc_urgent_idle_remaining. >>> The user can set the trial count limit for GC urgent and >>> idle mode with this value. If GC thread gets to the limit, >>> the mode will turn back to GC normal mode finally. >>> >>> This method is not only applicable to gc_urgent_high, >>> but applicable to all gc modes. Also mark >>> gc_urgent_high_remaining as deprecated, so that the node >>> can be removed in the future. >>> >>> Signed-off-by: Yangtao Li >>> --- >>> Documentation/ABI/testing/sysfs-fs-f2fs | 8 ++++++++ >>> fs/f2fs/f2fs.h | 6 ++++-- >>> fs/f2fs/gc.c | 12 ++++++------ >>> fs/f2fs/super.c | 2 +- >>> fs/f2fs/sysfs.c | 14 ++++++++++---- >>> 5 files changed, 29 insertions(+), 13 deletions(-) >>> >>> diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs b/Documentation/ABI/testing/sysfs-fs-f2fs >>> index 483639fb727b..859c4e53a846 100644 >>> --- a/Documentation/ABI/testing/sysfs-fs-f2fs >>> +++ b/Documentation/ABI/testing/sysfs-fs-f2fs >>> @@ -598,6 +598,14 @@ Contact: "Daeho Jeong" >>> Description: You can set the trial count limit for GC urgent high mode with this value. >>> If GC thread gets to the limit, the mode will turn back to GC normal mode. >>> By default, the value is zero, which means there is no limit like before. >>> + >>> + >>> +What: /sys/fs/f2fs//gc_urgent_idle_remaining >>> +Date: October 2022 >>> +Contact: "Yangtao Li" >>> +Description: You can set the trial count limit for GC urgent and idle mode with this value. >>> + If GC thread gets to the limit, the mode will turn back to GC normal mode. >>> + By default, the value is zero, which means there is no limit like before. >>> What: /sys/fs/f2fs//max_roll_forward_node_blocks >>> Date: January 2022 >>> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h >>> index e6355a5683b7..2bad69cf9fd9 100644 >>> --- a/fs/f2fs/f2fs.h >>> +++ b/fs/f2fs/f2fs.h >>> @@ -1734,8 +1734,10 @@ struct f2fs_sb_info { >>> unsigned int cur_victim_sec; /* current victim section num */ >>> unsigned int gc_mode; /* current GC state */ >>> unsigned int next_victim_seg[2]; /* next segment in victim section */ >>> - spinlock_t gc_urgent_high_lock; >>> - unsigned int gc_urgent_high_remaining; /* remaining trial count for GC_URGENT_HIGH */ >>> + spinlock_t gc_urgent_idle_lock; >>> + /* remaining trial count for GC_URGENT_* and GC_IDLE_* */ >>> + unsigned int gc_urgent_idle_remaining; >>> + unsigned int gc_urgent_high_remaining; /* deprecated */ >>> /* for skip statistic */ >>> unsigned long long skipped_gc_rwsem; /* FG_GC only */ >>> diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c >>> index e0ff99c7e3f2..0ed5b3c5922c 100644 >>> --- a/fs/f2fs/gc.c >>> +++ b/fs/f2fs/gc.c >>> @@ -152,14 +152,14 @@ static int gc_thread_func(void *data) >>> /* balancing f2fs's metadata periodically */ >>> f2fs_balance_fs_bg(sbi, true); >>> next: >>> - if (sbi->gc_mode == GC_URGENT_HIGH) { >>> - spin_lock(&sbi->gc_urgent_high_lock); >>> - if (sbi->gc_urgent_high_remaining) { >>> - sbi->gc_urgent_high_remaining--; >>> - if (!sbi->gc_urgent_high_remaining) >>> + if (sbi->gc_mode != GC_NORMAL) { >>> + spin_lock(&sbi->gc_urgent_idle_lock); >>> + if (sbi->gc_urgent_idle_remaining) { >>> + sbi->gc_urgent_idle_remaining--; >>> + if (!sbi->gc_urgent_idle_remaining) >>> sbi->gc_mode = GC_NORMAL; >>> } >>> - spin_unlock(&sbi->gc_urgent_high_lock); >>> + spin_unlock(&sbi->gc_urgent_idle_lock); >>> } >>> sb_end_write(sbi->sb); >>> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c >>> index 3834ead04620..f90a8c0a53ec 100644 >>> --- a/fs/f2fs/super.c >>> +++ b/fs/f2fs/super.c >>> @@ -3616,7 +3616,7 @@ static void init_sb_info(struct f2fs_sb_info *sbi) >>> sbi->seq_file_ra_mul = MIN_RA_MUL; >>> sbi->max_fragment_chunk = DEF_FRAGMENT_SIZE; >>> sbi->max_fragment_hole = DEF_FRAGMENT_SIZE; >>> - spin_lock_init(&sbi->gc_urgent_high_lock); >>> + spin_lock_init(&sbi->gc_urgent_idle_lock); >>> atomic64_set(&sbi->current_atomic_write, 0); >>> sbi->dir_level = DEF_DIR_LEVEL; >>> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c >>> index df27afd71ef4..2b1c653b37cf 100644 >>> --- a/fs/f2fs/sysfs.c >>> +++ b/fs/f2fs/sysfs.c >>> @@ -531,10 +531,14 @@ static ssize_t __sbi_store(struct f2fs_attr *a, >>> return count; >>> } >>> - if (!strcmp(a->attr.name, "gc_urgent_high_remaining")) { >>> - spin_lock(&sbi->gc_urgent_high_lock); >>> - sbi->gc_urgent_high_remaining = t; >>> - spin_unlock(&sbi->gc_urgent_high_lock); >>> + /* deprecated */ >>> + if (!strcmp(a->attr.name, "gc_urgent_high_remaining")) >>> + return -EINVAL; >> >> How about those users who has already used these interface... it breaks >> the usage. >> >> It needs to keep old interface and tag as deprecated, and recommend user >> to use new interface you introduced. > > I feel that this is not a super critical node, so how about just removing it? Only Android is using this inferface, right? Any plan to adjust related code from Android side? Thanks, > >> >> e.g.: >> >> What: /sys/fs/f2fs//features >> Date: July 2017 >> Contact: "Jaegeuk Kim" >> Description: /feature_list/ >> Shows all enabled features in current device. >> Supported features: >> encryption, blkzoned, extra_attr, projquota, inode_checksum, >> flexible_inline_xattr, quota_ino, inode_crtime, lost_found, >> verity, sb_checksum, casefold, readonly, compression, pin_file. >> >> Thanks, >> >>> + >>> + if (!strcmp(a->attr.name, "gc_urgent_idle_remaining")) { >>> + spin_lock(&sbi->gc_urgent_idle_lock); >>> + sbi->gc_urgent_idle_remaining = t; >>> + spin_unlock(&sbi->gc_urgent_idle_lock); >>> return count; >>> } >>> @@ -826,6 +830,7 @@ F2FS_RW_ATTR(FAULT_INFO_TYPE, f2fs_fault_info, inject_type, inject_type); >>> F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, data_io_flag, data_io_flag); >>> F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, node_io_flag, node_io_flag); >>> F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, gc_urgent_high_remaining, gc_urgent_high_remaining); >>> +F2FS_RW_ATTR(F2FS_SBI, f2fs_sb_info, gc_urgent_idle_remaining, gc_urgent_idle_remaining); >>> F2FS_RW_ATTR(CPRC_INFO, ckpt_req_control, ckpt_thread_ioprio, ckpt_thread_ioprio); >>> F2FS_GENERAL_RO_ATTR(dirty_segments); >>> F2FS_GENERAL_RO_ATTR(free_segments); >>> @@ -953,6 +958,7 @@ static struct attribute *f2fs_attrs[] = { >>> ATTR_LIST(data_io_flag), >>> ATTR_LIST(node_io_flag), >>> ATTR_LIST(gc_urgent_high_remaining), >>> + ATTR_LIST(gc_urgent_idle_remaining), >>> ATTR_LIST(ckpt_thread_ioprio), >>> ATTR_LIST(dirty_segments), >>> ATTR_LIST(free_segments), _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel