Re: [PATCH v5] mm/rmap: do not add fully unmapped large folio to deferred split list

From: Zi Yan
Date: Thu May 02 2024 - 09:20:43 EST


On 2 May 2024, at 9:18, Alexander Gordeev wrote:

> On Wed, May 01, 2024 at 09:38:24AM -0400, Zi Yan wrote:
> Hi Zi,
>> @@ -1557,9 +1557,8 @@ static __always_inline void __folio_remove_rmap(struct folio *folio,
>> * page of the folio is unmapped and at least one page
>> * is still mapped.
>> */
>> - if (folio_test_anon(folio) &&
>> - list_empty(&folio->_deferred_list) &&
>> - partially_mapped)
>> + if (folio_test_anon(folio) && partially_mapped &&
>> + list_empty(&folio->_deferred_list))
>> deferred_split_folio(folio);
>> }
>
> That helps.
>
>> Best Regards,
>> Yan, Zi
>
> Thanks!

Great! I will send a v6.

--
Best Regards,
Yan, Zi

Attachment: signature.asc
Description: OpenPGP digital signature