lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] mm: add a new emergency page migratetype.
From
On 06.06.22 05:27, Huanpeng Xin wrote:
> From: xinhuanpeng <xinhuanpeng@xiaomi.com>
>
> add a new page migratetype reserved for
> non-costly non-NOWARN page allocation failure.

Sorry to say, but this patch description is not expressive enough. I
have absolutely no clue what you are trying to achieve and why we should
care.

Especially, why do we care about locally grouping these allocations
(that's what pageblock flags are for after all)?

Your Kconfig option is also not particularly user friendly to read either:

"This enables the migration type MIGRATE_EMERGENCY,which reserves
a small amount of memory for non-costly non-NOWARN page allocation
failure."

Usually we reserve memory via different mechanisms, like atomic
reserves? Why can't something like that be used.

On first sight, defining a new pageblock migratype feels wrong to me.
But then, I have no clue what you are actually trying to achieve.

--
Thanks,

David / dhildenb

\
 
 \ /
  Last update: 2022-06-07 14:56    [W:0.052 / U:0.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site