Messages in this thread | | | Date | Fri, 9 Apr 2021 10:31:32 +0200 | From | Oscar Salvador <> | Subject | Re: [PATCH 07/10] mm/vmscan: add helper for querying ability to age anonymous pages |
| |
On Wed, Apr 07, 2021 at 11:40:13AM -0700, Wei Xu wrote: > anon_should_be_aged() doesn't really need "lruvec". It essentially > answers whether the pages of the given node can be swapped or demoted. > So it would be clearer and less confusing if anon_should_be_aged() > takes "pgdat" instead of "lruvec" as the argument. The call to > mem_cgroup_lruvec(NULL, pgdat) in age_active_anon() can then be removed > as well.
I tend to agree with this, and I would go one step further with the naming. For me, taking into account the nature of the function that tells us whether we have any means to age those pages, a better fit would be something like anon_can_be_aged(). IIUC, the "should age" part would be inactive_is_low().
-- Oscar Salvador SUSE L3
| |