On 9 Oct 2024, at 5:54, Kirill A. Shutemov wrote: > On Tue, Oct 08, 2024 at 06:37:47PM -0400, Zi Yan wrote: >> mm/huge_memory.c | 648 ++++++++++++++++++++++++++++++++++++++++++++++- >> 1 file changed, 647 insertions(+), 1 deletion(-) > > The idea is sane, but I think it would require a lot more ground work > before getting it upstream. I don't think we can afford two parallel split > implementations. folio_split() and split_huge_page*() should share the same > implementation internally. Otherwise it is going to be pain to maintain > them in-sync. The goal is to replace split_huge_page*() with folio_split(). But for now, split_huge_page*() is still needed for swap cached anon folios and shmem. And this might take quite a while until we have a better swap system. I think it is possible to use the same internal implementation for both folio_split() and split_huge_page*(). I can give it a try in the next version. -- Best Regards, Yan, Zi
Attachment:
signature.asc
Description: OpenPGP digital signature
