Re: [PATCH v1 17/29] mm/page_isolation: drop __folio_test_movable() check for large folios
From: David Hildenbrand <hidden>
Date: 2025-07-01 12:33:01
Also in:
linux-doc, linux-fsdevel, linux-mm, lkml, virtualization
On 01.07.25 13:03, Lorenzo Stoakes wrote:
On Mon, Jun 30, 2025 at 02:59:58PM +0200, David Hildenbrand wrote:quoted
Currently, we only support migration of individual movable_ops pages, so we can not run into that. Reviewed-by: Zi Yan <ziy@nvidia.com> Signed-off-by: David Hildenbrand <redacted>Seems sensible, so: Reviewed-by: Lorenzo Stoakes <redacted> Maybe worth adding a VM_WARN_ON_ONCE() just in case? Or do you think not worth it?
Not for now I think. Whoever wants to support compound pages has to fixup a bunch of other stuff first, before running into that one here. So a full audit of all paths that handle page_has_movable_ops() is required either way. -- Cheers, David / dhildenb