All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <juergen.gross@ts.fujitsu.com>
To: Andre Przywara <andre.przywara@amd.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: RFC: automatic NUMA placement
Date: Tue, 28 Sep 2010 06:48:45 +0200	[thread overview]
Message-ID: <4CA173AD.1070302@ts.fujitsu.com> (raw)
In-Reply-To: <4CA110D3.5050000@amd.com>

Hi Andre,

thanks for your thoughts.

On 09/27/10 23:46, Andre Przywara wrote:
> Juergen Gross wrote:
>> Hi,
>>
>> I just stumbled upon the automatic pinning of vcpus on domain creation in
>> case of NUMA.
>> This behaviour is questionable IMO, as it breaks correct handling of
>> scheduling weights on NUMA machines.
>> I would suggest to switch this feature off per default and make it a
>> configuration option of xend. It would make sense, however, to change
>> cpu pool
>> processor allocation to be NUMA-aware.
>> Switching NUMA off via boot option would remove NUMA-optimized memory
>> allocation, which would be sub-optimal :-)
> Hi Jürgen,
>
> stumbled over your mail just now, so sorry for the delay.
> First: Don't turn off automatic NUMA placement ;-)
> In my tests it helped a lot to preserve performance on NUMA machines.
>
> I was just browsing through the ML archive to find your original CPU
> pools description from April, and it seems to fit the requirements in
> NUMA machines quite well.
> I haven't done any experiments with Cpupools nor haven't looked at the
> code yet, but just a quick idea:
> What about if we marry static NUMA placement and Cpupools?
>
> I'd suggest to introduce static NUMA pools, one for each node. The CPUs
> assigned to each pool are fixed and cannot be removed nor added (because
> the NUMA topology is fixed).
> Is that possible? Can we assign one physical CPUs to multiple pools (to
> Pool-0 and to NUMA-0?) Or are they exclusive or hierarchical like the
> Linux' cpusets?

A cpu is always member of only one pool.

> We could introduce magic names for each NUMA pool, so that people just
> say cpupool="NUMA-2" and get their domain pinned to that pool. Without
> any explicit assignment the system would pick a NUMA node (like it does
> today) and would just use the respective Cpupool. I think that is very
> similar to what it does today, only that the pinning nature is more
> evident to the user (as it uses the Cpupool name space).
> Also it would allow for users to override the pinning by specifying a
> different Cpupool explicitly (like Pool-0).
>
> Just tell me what you think about this and whether I am wrong with my
> thinking ;-)

With your proposal it isn't possible to start a domU with more vcpus than
cpus in a node without changing cpu pools.

I would suggest to do it the following way:
- use automatic NUMA placement only in Pool-0 (this won't change anything
   for users not using cpu pools), perhaps with an option to switch it off
- change the cpu allocation for pools to be NUMA aware
- optionally add a xl and/or xm command to create one cpu pool per NUMA node


Juergen

-- 
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@ts.fujitsu.com
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

      reply	other threads:[~2010-09-28  4:48 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-16 13:38 RFC: automatic NUMA placement Juergen Gross
2010-09-16 15:52 ` Dan Magenheimer
2010-09-17  6:26   ` Juergen Gross
2010-09-27 21:46 ` Andre Przywara
2010-09-28  4:48   ` Juergen Gross [this message]

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=4CA173AD.1070302@ts.fujitsu.com \
    --to=juergen.gross@ts.fujitsu.com \
    --cc=andre.przywara@amd.com \
    --cc=xen-devel@lists.xensource.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.