linux-kernel - 答复: [PATCH] f2fs: introduce gc_urgent_mid

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <SEZPR06MB52693451F34B9A5CB60E2851E82A9@SEZPR06MB5269.apcprd06.prod.outlook.com>
Date:   Thu, 20 Oct 2022 08:12:28 +0000
From:   李扬韬 <frank.li@...o.com>
To:     Chao Yu <chao@...nel.org>,
        "jaegeuk@...nel.org" <jaegeuk@...nel.org>
CC:     "linux-f2fs-devel@...ts.sourceforge.net" 
        <linux-f2fs-devel@...ts.sourceforge.net>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: 答复: [PATCH] f2fs: introduce gc_urgent_mid_remaining sysfs node

Chao,

> Not sure, we will add gc_urgent_low_remaining later...

> Can we share the same interface for all gc_mode? since each mode is exclusive.

> Thoughts?

Both GC urgent mid mode and GC urgent high mode run using the urgent_sleep_time interval. If the user program does not switch back to normal mode in an abnormal state, it will cause the gc to run rapidly.

Therefore, there needs to be a mechanism to ensure that the GC urgent mid mode and GC urgent high mode can be switched back to the normal mode. Since there is currently a gc_urgent_high_remaining node, I added the gc_urgent_mid_remaining node.

If gc_urgent_high_remaining can act on three modes, it is not in line with the meaning of the name. Not sure if the existing sys node can still change the name, if it can be changed, obviously using gc_urgent_remaining is a better way. Can an existing node still be renamed?

If the name cannot be changed, it is necessary to increase gc_urgent_mid_remaining. Perhaps one more gc_urgent_low_remaining could be added?