From: Michal Hocko <mhocko@suse.com>
To: Dave Hansen <dave.hansen@linux.intel.com>
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org,
yang.shi@linux.alibaba.com, rientjes@google.com,
ying.huang@intel.com, dan.j.williams@intel.com, david@redhat.com,
osalvador@suse.de, weixugc@google.com
Subject: Re: [PATCH 00/10] [v7][RESEND] Migrate Pages in lieu of discard
Date: Fri, 16 Apr 2021 14:35:39 +0200 [thread overview]
Message-ID: <YHmEm/yHpaqO6khp@dhcp22.suse.cz> (raw)
In-Reply-To: <20210401183216.443C4443@viggo.jf.intel.com>
Hi,
I am really sorry to jump into this train sooo late. I have quickly
glanced through the series and I have some questions/concerns. Let me
express them here rather than in specific patches.
First of all I do think that demotion is a useful way to balance the
memory in general. And that is not really bound to PMEM equipped
systems. There are larger NUMA machines which are not trivial to
partition and our existing NUMA APIs are far from ideal to help with
that.
I do appreciate that the whole thing is an opt in because this might
break workloads which are careful with the placement. I am not sure
there is a way to handle constrains in an optimal way if that is
possible at all in some cases (e.g. do we have a way to track page to
its cpuset resp. task mempolicy in all cases?).
The cover letter is focusing on usecases but it doesn't really provide
so let me try to lay it down here (let's see whether I missed something
important).
- order for demontion defines a very simple fallback to a single node
based on the proximity but cycles are not allowed in the fallback
mask.
I have to confess that I haven't grasped the initialization
completely. There is a nice comment explaining a 2 socket system with
3 different NUMA nodes attached to it with one node being terminal.
This is OK if the terminal node is PMEM but how that fits into usual
NUMA setups. E.g.
4 nodes each with its set of CPUs
node distances:
node 0 1 2 3
0: 10 20 20 20
1: 20 10 20 20
2: 20 20 10 20
3: 20 20 20 10
Do I get it right that Node 3 would be terminal?
- The demotion is controlled by node_reclaim_mode but unlike other modes
it applies to both direct and kswapd reclaims.
I do not see that explained anywhere though.
- The demotion is implemented at shrink_page_list level which migrates
pages in the first round and then falls back to the regular reclaim
when migration fails. This means that the reclaim context
(PF_MEMALLOC) will allocate memory so it has access to full memory
reserves. Btw. I do not __GFP_NO_MEMALLOC anywhere in the allocation
mask which looks like a bug rather than an intention. Btw. using
GFP_NOWAIT in the allocation callback would make more things clear
IMO.
- Memcg reclaim is excluded from all this because it is not NUMA aware
which makes sense to me.
- Anonymous pages are bit tricky because they can be demoted even when
they cannot be reclaimed due to no (or no available) swap storage.
Unless I have missed something the second round will try to reclaim
them even the later is true and I am not sure this is completely OK.
I hope I've captured all important parts. There are some more details
but they do not seem that important.
I am still trying to digest the whole thing but at least jamming
node_reclaim logic into kswapd seems strange to me. Need to think more
about that though.
Btw. do you have any numbers from running this with some real work
workload?
--
Michal Hocko
SUSE Labs
next prev parent reply other threads:[~2021-04-16 12:35 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-01 18:32 [PATCH 00/10] [v7][RESEND] Migrate Pages in lieu of discard Dave Hansen
2021-04-01 18:32 ` [PATCH 01/10] mm/numa: node demotion data structure and lookup Dave Hansen
2021-04-08 8:03 ` Oscar Salvador
2021-04-08 21:29 ` Dave Hansen
2021-04-09 5:32 ` Wei Xu
2021-04-01 18:32 ` [PATCH 02/10] mm/numa: automatically generate node migration order Dave Hansen
2021-04-08 8:26 ` Oscar Salvador
2021-04-08 21:51 ` Dave Hansen
2021-04-09 8:17 ` Oscar Salvador
2021-04-10 3:07 ` Wei Xu
2021-04-14 8:08 ` Oscar Salvador
2021-04-14 8:11 ` Oscar Salvador
2021-04-14 8:12 ` David Hildenbrand
2021-04-14 8:14 ` Oscar Salvador
2021-04-14 8:20 ` David Hildenbrand
2021-04-15 4:07 ` Wei Xu
2021-04-15 15:35 ` Dave Hansen
2021-04-15 20:25 ` Wei Xu
2021-04-01 18:32 ` [PATCH 03/10] mm/migrate: update node demotion order during on hotplug events Dave Hansen
2021-04-08 9:52 ` Oscar Salvador
2021-04-09 10:14 ` Oscar Salvador
2021-04-09 10:15 ` Oscar Salvador
2021-04-09 18:59 ` David Hildenbrand
2021-04-12 7:19 ` Oscar Salvador
2021-04-12 9:19 ` David Hildenbrand
2021-04-01 18:32 ` [PATCH 04/10] mm/migrate: make migrate_pages() return nr_succeeded Dave Hansen
2021-04-01 22:35 ` Wei Xu
2021-04-01 23:21 ` Dave Hansen
2021-04-01 22:39 ` Wei Xu
2021-04-08 10:14 ` Oscar Salvador
2021-04-08 17:26 ` Yang Shi
2021-04-08 18:17 ` Oscar Salvador
2021-04-08 18:21 ` Oscar Salvador
2021-04-08 20:40 ` Yang Shi
2021-04-09 5:06 ` Oscar Salvador
2021-04-09 5:43 ` Wei Xu
2021-04-09 15:43 ` Yang Shi
2021-04-09 15:50 ` Dave Hansen
2021-04-09 18:47 ` Wei Xu
2021-04-09 20:10 ` Yang Shi
2021-04-01 18:32 ` [PATCH 05/10] mm/migrate: demote pages during reclaim Dave Hansen
2021-04-01 20:01 ` Yang Shi
2021-04-01 22:58 ` Dave Hansen
2021-04-08 10:47 ` Oscar Salvador
2021-04-10 3:35 ` Wei Xu
2021-04-01 18:32 ` [PATCH 06/10] mm/vmscan: add page demotion counter Dave Hansen
2021-04-10 3:40 ` Wei Xu
2021-04-01 18:32 ` [PATCH 07/10] mm/vmscan: add helper for querying ability to age anonymous pages Dave Hansen
2021-04-07 18:40 ` Wei Xu
2021-04-09 8:31 ` Oscar Salvador
2021-04-01 18:32 ` [PATCH 08/10] mm/vmscan: Consider anonymous pages without swap Dave Hansen
2021-04-02 0:55 ` Wei Xu
2021-04-01 18:32 ` [PATCH 09/10] mm/vmscan: never demote for memcg reclaim Dave Hansen
2021-04-02 0:18 ` Wei Xu
2021-04-01 18:32 ` [PATCH 10/10] mm/migrate: new zone_reclaim_mode to enable reclaim migration Dave Hansen
2021-04-01 20:06 ` Yang Shi
2021-04-10 4:10 ` Wei Xu
2021-04-16 12:35 ` Michal Hocko [this message]
2021-04-16 14:26 ` [PATCH 00/10] [v7][RESEND] Migrate Pages in lieu of discard Dave Hansen
2021-04-16 15:02 ` Michal Hocko
2021-04-21 2:39 ` Huang, Ying
2021-05-07 6:14 ` Huang, Ying
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YHmEm/yHpaqO6khp@dhcp22.suse.cz \
--to=mhocko@suse.com \
--cc=dan.j.williams@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=david@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=osalvador@suse.de \
--cc=rientjes@google.com \
--cc=weixugc@google.com \
--cc=yang.shi@linux.alibaba.com \
--cc=ying.huang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).