On 8/13/25 15:52, Jakub Kicinski wrote:
On Wed, 13 Aug 2025 12:18:56 +0100 Pavel Begunkov wrote:
It should go to net, there will be enough of conflicts otherwise.
mm maintainers, do you like it as a shared branch or can it just
go through the net tree?
Looks like this is 100% in mm, and the work is not urgent at all.
There is a slight dependency in rc1, but we should be able to
massage it to be mm only.
So I'm happy for Andrew to take this, and dependent patches (if any)
can come in the next cycle.
Yeah, good option. It'd be a good idea to cut the diff down to
avoid removing the relevant mm page state checks until the next
cycle.
>> @@ -1379,9 +1376,11 @@ __always_inline bool free_pages_prepare(struct page *page,
mod_mthp_stat(order, MTHP_STAT_NR_ANON, -1);
folio->mapping = NULL;
}
- if (unlikely(page_has_type(page)))
+ if (unlikely(page_has_type(page))) {
+ WARN_ON_ONCE(PageNet_pp(page));
I guess my ask to add a comment here got ignored?
It's an old patch attached as a point of reference. Any actual submission
surely will need to follow up on the reviews.
--
Pavel Begunkov
