From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Michal Privoznik <mprivozn@redhat.com>
Cc: peter.maydell@linaro.org, ehabkost@redhat.com,
libvir-list@redhat.com, qemu-devel@nongnu.org,
Markus Armbruster <armbru@redhat.com>,
qemu-arm@nongnu.org, qemu-ppc@nongnu.org, pbonzini@redhat.com,
Igor Mammedov <imammedo@redhat.com>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
david@gibson.dropbear.id.au
Subject: Re: [Qemu-arm] [Qemu-devel] [libvirt] [PATCH 1/2] numa: deprecate 'mem' parameter of '-numa node' option
Date: Mon, 4 Mar 2019 16:27:23 +0000 [thread overview]
Message-ID: <20190304162723.GP4239@redhat.com> (raw)
In-Reply-To: <ae0335fb-9201-e09d-80ad-5e497365267b@redhat.com>
On Mon, Mar 04, 2019 at 05:12:40PM +0100, Michal Privoznik wrote:
> On 3/4/19 4:19 PM, Igor Mammedov wrote:
>
> > Then I'd guess that most VMs end up with default '-numa node,mem'
> > which by design can produce only fake NUMA without ability to manage
> > guest RAM on host side. So such VMs aren't getting performance benefits
> > or worse run with performance regression (due to wrong sched/mm decisions
> > as guest kernel assumes NUMA topology is valid one).
>
> Specifying NUMA distances in libvirt XML makes it generate the modern cmd
> line.
AFAIK, specifying any guest NUMA -> Host NUMA affinity makes it use the
modern cmd line. eg I just modified a plain 8 CPU / 2 GB RAM guest
with this:
<numatune>
<memnode cellid='0' mode='strict' nodeset='0'/>
<memnode cellid='1' mode='strict' nodeset='1'/>
</numatune>
<cpu mode='host-model'>
<numa>
<cell id='0' cpus='0-3' memory='1024000' unit='KiB'/>
<cell id='1' cpus='4-7' memory='1024000' unit='KiB'/>
</numa>
</cpu>
and I can see libvirt decided to use memdev
-object memory-backend-ram,id=ram-node0,size=1048576000,host-nodes=0,policy=bind
-numa node,nodeid=0,cpus=0-3,memdev=ram-node0
-object memory-backend-ram,id=ram-node1,size=1048576000,host-nodes=1,policy=bind
-numa node,nodeid=1,cpus=4-7,memdev=ram-node1
So unless I'm missing something, we aren't suffering from the problem
described by Igor above even today.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
WARNING: multiple messages have this Message-ID (diff)
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Michal Privoznik <mprivozn@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>,
peter.maydell@linaro.org, ehabkost@redhat.com,
libvir-list@redhat.com, qemu-devel@nongnu.org,
Markus Armbruster <armbru@redhat.com>,
qemu-arm@nongnu.org, qemu-ppc@nongnu.org, pbonzini@redhat.com,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
david@gibson.dropbear.id.au
Subject: Re: [Qemu-devel] [libvirt] [PATCH 1/2] numa: deprecate 'mem' parameter of '-numa node' option
Date: Mon, 4 Mar 2019 16:27:23 +0000 [thread overview]
Message-ID: <20190304162723.GP4239@redhat.com> (raw)
In-Reply-To: <ae0335fb-9201-e09d-80ad-5e497365267b@redhat.com>
On Mon, Mar 04, 2019 at 05:12:40PM +0100, Michal Privoznik wrote:
> On 3/4/19 4:19 PM, Igor Mammedov wrote:
>
> > Then I'd guess that most VMs end up with default '-numa node,mem'
> > which by design can produce only fake NUMA without ability to manage
> > guest RAM on host side. So such VMs aren't getting performance benefits
> > or worse run with performance regression (due to wrong sched/mm decisions
> > as guest kernel assumes NUMA topology is valid one).
>
> Specifying NUMA distances in libvirt XML makes it generate the modern cmd
> line.
AFAIK, specifying any guest NUMA -> Host NUMA affinity makes it use the
modern cmd line. eg I just modified a plain 8 CPU / 2 GB RAM guest
with this:
<numatune>
<memnode cellid='0' mode='strict' nodeset='0'/>
<memnode cellid='1' mode='strict' nodeset='1'/>
</numatune>
<cpu mode='host-model'>
<numa>
<cell id='0' cpus='0-3' memory='1024000' unit='KiB'/>
<cell id='1' cpus='4-7' memory='1024000' unit='KiB'/>
</numa>
</cpu>
and I can see libvirt decided to use memdev
-object memory-backend-ram,id=ram-node0,size=1048576000,host-nodes=0,policy=bind
-numa node,nodeid=0,cpus=0-3,memdev=ram-node0
-object memory-backend-ram,id=ram-node1,size=1048576000,host-nodes=1,policy=bind
-numa node,nodeid=1,cpus=4-7,memdev=ram-node1
So unless I'm missing something, we aren't suffering from the problem
described by Igor above even today.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2019-03-04 16:27 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-01 15:42 [Qemu-devel] [PATCH 0/2] numa: deprecate -numa node, mem and default memory distribution Igor Mammedov
2019-03-01 15:42 ` Igor Mammedov
2019-03-01 15:42 ` [Qemu-arm] [PATCH 1/2] numa: deprecate 'mem' parameter of '-numa node' option Igor Mammedov
2019-03-01 15:42 ` [Qemu-devel] " Igor Mammedov
2019-03-01 15:49 ` [Qemu-devel] [libvirt] " Daniel P. Berrangé
2019-03-01 15:49 ` Daniel P. Berrangé
2019-03-01 17:33 ` [Qemu-arm] " Igor Mammedov
2019-03-01 17:33 ` Igor Mammedov
2019-03-01 17:48 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-01 17:48 ` Daniel P. Berrangé
2019-03-04 7:13 ` [Qemu-arm] " Markus Armbruster
2019-03-04 7:13 ` Markus Armbruster
2019-03-04 10:19 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-04 10:19 ` Daniel P. Berrangé
2019-03-04 11:45 ` [Qemu-arm] " Markus Armbruster
2019-03-04 11:45 ` Markus Armbruster
2019-03-04 15:28 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-04 15:28 ` Daniel P. Berrangé
2019-03-04 15:46 ` [Qemu-arm] " Igor Mammedov
2019-03-04 15:46 ` Igor Mammedov
2019-03-10 10:14 ` [Qemu-arm] " Markus Armbruster
2019-03-10 10:14 ` Markus Armbruster
2019-03-19 14:17 ` [Qemu-arm] " Igor Mammedov
2019-03-04 14:24 ` Michal Privoznik
2019-03-04 14:24 ` Michal Privoznik
2019-03-04 15:03 ` [Qemu-arm] [libvirt] [Qemu-devel] " Igor Mammedov
2019-03-04 15:03 ` [Qemu-devel] [libvirt] " Igor Mammedov
2019-03-04 12:25 ` [Qemu-arm] " Igor Mammedov
2019-03-04 12:25 ` Igor Mammedov
2019-03-04 12:39 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-04 12:39 ` Daniel P. Berrangé
2019-03-04 14:16 ` [Qemu-arm] " Igor Mammedov
2019-03-04 14:16 ` Igor Mammedov
2019-03-04 14:24 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-04 14:24 ` Daniel P. Berrangé
2019-03-04 15:19 ` [Qemu-arm] " Igor Mammedov
2019-03-04 15:19 ` Igor Mammedov
2019-03-04 16:12 ` [Qemu-arm] " Michal Privoznik
2019-03-04 16:12 ` Michal Privoznik
2019-03-04 16:27 ` Daniel P. Berrangé [this message]
2019-03-04 16:27 ` Daniel P. Berrangé
2019-03-04 16:20 ` Michal Privoznik
2019-03-04 16:20 ` Michal Privoznik
2019-03-04 16:31 ` Dr. David Alan Gilbert
2019-03-04 16:31 ` Dr. David Alan Gilbert
2019-03-04 16:35 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-04 16:35 ` Daniel P. Berrangé
2019-03-06 19:03 ` [Qemu-arm] " Igor Mammedov
2019-03-06 19:03 ` Igor Mammedov
2019-03-07 9:59 ` [Qemu-arm] " Daniel P. Berrangé
2019-03-07 9:59 ` Daniel P. Berrangé
2019-03-10 10:16 ` [Qemu-arm] " Markus Armbruster
2019-03-10 10:16 ` Markus Armbruster
2019-03-14 14:52 ` Igor Mammedov
2019-03-06 19:56 ` [Qemu-arm] " Igor Mammedov
2019-03-06 19:56 ` Igor Mammedov
2019-03-04 14:34 ` Michal Privoznik
2019-03-04 14:34 ` Michal Privoznik
2019-03-04 8:11 ` [Qemu-arm] [Qemu-ppc] " Thomas Huth
2019-03-04 8:11 ` [Qemu-devel] [Qemu-ppc] " Thomas Huth
2019-03-04 13:55 ` [Qemu-arm] [Qemu-ppc] [Qemu-devel] " Igor Mammedov
2019-03-04 13:55 ` [Qemu-devel] [Qemu-ppc] " Igor Mammedov
2019-03-04 13:59 ` [Qemu-arm] [Qemu-ppc] [Qemu-devel] " Daniel P. Berrangé
2019-03-04 13:59 ` [Qemu-devel] [Qemu-ppc] " Daniel P. Berrangé
2019-03-04 14:54 ` [Qemu-arm] [Qemu-ppc] [Qemu-devel] " Igor Mammedov
2019-03-04 14:54 ` [Qemu-devel] [Qemu-ppc] " Igor Mammedov
2019-03-04 15:02 ` [Qemu-arm] [Qemu-ppc] [Qemu-devel] " Daniel P. Berrangé
2019-03-04 15:02 ` [Qemu-devel] [Qemu-ppc] " Daniel P. Berrangé
2019-03-04 16:45 ` [Qemu-arm] [Qemu-ppc] [Qemu-devel] " Igor Mammedov
2019-03-04 16:45 ` [Qemu-devel] [Qemu-ppc] " Igor Mammedov
2019-03-01 18:01 ` [Qemu-arm] [Qemu-devel] " Dr. David Alan Gilbert
2019-03-01 18:01 ` Dr. David Alan Gilbert
2019-03-04 13:52 ` [Qemu-arm] " Igor Mammedov
2019-03-04 13:52 ` Igor Mammedov
2019-03-18 16:44 ` [Qemu-arm] " Igor Mammedov
2019-03-01 15:42 ` [Qemu-arm] [PATCH 2/2] numa: deprecate implict memory distribution between nodes Igor Mammedov
2019-03-01 15:42 ` [Qemu-devel] " Igor Mammedov
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=20190304162723.GP4239@redhat.com \
--to=berrange@redhat.com \
--cc=armbru@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=dgilbert@redhat.com \
--cc=ehabkost@redhat.com \
--cc=imammedo@redhat.com \
--cc=libvir-list@redhat.com \
--cc=mprivozn@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.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 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.