On 2024/4/8 1:42, Alexander H Duyck wrote: > On Sun, 2024-04-07 at 21:08 +0800, Yunsheng Lin wrote: >> Inspired by [1], but use free_unref_page() to replace free_the_page() >> instead of __free_pages(), use VM_BUG_ON() to catch that we can use >> free_unref_page() directly, also add its own header file. > > Instead of doing this all as one patch it would be better to split this > into 2. Make the refactor first, and then move the code. Don't do it > all in one patch. Sure, will split this into 2 patches, thanks.