From: "Huang\, Ying" <ying.huang@intel.com>
To: Mel Gorman <mgorman@suse.de>
Cc: Peter Zijlstra <peterz@infradead.org>, <linux-mm@kvack.org>,
<linux-kernel@vger.kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Ingo Molnar <mingo@redhat.com>, Rik van Riel <riel@redhat.com>,
"Johannes Weiner" <hannes@cmpxchg.org>,
"Matthew Wilcox \(Oracle\)" <willy@infradead.org>,
Dave Hansen <dave.hansen@intel.com>,
Andi Kleen <ak@linux.intel.com>, Michal Hocko <mhocko@suse.com>,
David Rientjes <rientjes@google.com>,
ben.widawsky@intel.com
Subject: Re: [PATCH -V2 2/2] autonuma: Migrate on fault among multiple bound nodes
Date: Fri, 06 Nov 2020 15:28:59 +0800 [thread overview]
Message-ID: <87v9ejosec.fsf@yhuang-dev.intel.com> (raw)
In-Reply-To: <20201105112523.GQ3306@suse.de> (Mel Gorman's message of "Thu, 5 Nov 2020 11:25:23 +0000")
Mel Gorman <mgorman@suse.de> writes:
> On Wed, Nov 04, 2020 at 01:36:58PM +0800, Huang, Ying wrote:
>> But from another point of view, I suggest to remove the constraints of
>> MPOL_F_MOF in the future. If the overhead of AutoNUMA isn't acceptable,
>> why not just disable AutoNUMA globally via sysctl knob?
>>
>
> Because it's a double edged sword. NUMA Balancing can make a workload
> faster while still incurring more overhead than it should -- particularly
> when threads are involved rescanning the same or unrelated regions.
> Global disabling only really should happen when an application is running
> that is the only application on the machine and has full NUMA awareness.
Got it. So NUMA Balancing may in generally benefit some workloads but
hurt some other workloads on one machine. So we need a method to
enable/disable NUMA Balancing for one workload. Previously, this is
done via the explicit NUMA policy. If some explicit NUMA policy is
specified, NUMA Balancing is disabled for the memory region or the
thread. And this can be reverted again for a memory region via
MPOL_MF_LAZY. It appears that we lacks MPOL_MF_LAZY for the thread yet.
>> > It might still end up being better but I was not aware of a
>> > *realistic* workload that binds to multiple nodes
>> > deliberately. Generally I expect if an application is binding, it's
>> > binding to one local node.
>>
>> Yes. It's not popular configuration for now. But for the memory
>> tiering system with both DRAM and PMEM, the DRAM and PMEM in one socket
>> will become 2 NUMA nodes. To avoid too much cross-socket memory
>> accessing, but take advantage of both the DRAM and PMEM, the workload
>> can be bound to 2 NUMA nodes (DRAM and PMEM).
>>
>
> Ok, that may lead to unpredictable performance as it'll have variable
> performance with limited control of the "important" applications that
> should use DRAM over PMEM. That's a long road but the step is not
> incompatible with the long-term goal.
Yes. Ben Widawsky is working on a patchset to make it possible to
prefer the remote DRAM instead of the local PMEM as follows,
https://lore.kernel.org/linux-mm/20200630212517.308045-1-ben.widawsky@intel.com/
Best Regards,
Huang, Ying
next prev parent reply other threads:[~2020-11-06 7:29 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-28 2:34 [PATCH -V2 0/2] " Huang Ying
2020-10-28 2:34 ` [PATCH -V2 1/2] mempolicy: Rename MPOL_F_MORON to MPOL_F_MOPRON Huang Ying
2020-10-29 9:04 ` Michal Hocko
2020-10-30 7:27 ` Huang, Ying
2020-10-30 8:25 ` Michal Hocko
2020-11-02 3:12 ` Huang, Ying
2020-10-28 2:34 ` [PATCH -V2 2/2] autonuma: Migrate on fault among multiple bound nodes Huang Ying
2020-11-02 11:17 ` Mel Gorman
2020-11-04 5:36 ` Huang, Ying
2020-11-05 11:25 ` Mel Gorman
2020-11-06 7:28 ` Huang, Ying [this message]
2020-11-06 15:55 ` Ben Widawsky
2020-11-11 6:50 ` 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=87v9ejosec.fsf@yhuang-dev.intel.com \
--to=ying.huang@intel.com \
--cc=ak@linux.intel.com \
--cc=akpm@linux-foundation.org \
--cc=ben.widawsky@intel.com \
--cc=dave.hansen@intel.com \
--cc=hannes@cmpxchg.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mgorman@suse.de \
--cc=mhocko@suse.com \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=riel@redhat.com \
--cc=rientjes@google.com \
--cc=willy@infradead.org \
/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