On Thu, 19 Oct 2023 21:42:10 +0800 Zhiguo Jiang <justinjiang@xxxxxxxx> wrote: > In the shrink_folio_list() the sources of the file dirty folio include > two ways below: > 1. The dirty folio is from the incoming parameter folio_list, > which is the inactive file lru. > 2. The dirty folio is from the PTE dirty bit transferred by > the try_to_unmap(). > > For the first source of the dirty folio, if the dirty folio does not > support pageout, the dirty folio can skip unmap in advance to reduce > recyling time. > This patch does an amount of code movement and it implements a functional change. Is it possible to split these? The first patch moves code around but has no runtime effect, the second patch implements the functional change. Also, the patch doesn't apply to current code so please redo it against Linus's latest tree? Thanks.