All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: aliguori@us.ibm.com, ehabkost@redhat.com, hutao@cn.fujitsu.com,
	qemu-devel@nongnu.org, lcapitulino@redhat.com, bsd@redhat.com,
	y-goto@jp.fujitsu.com, peter.huangpeng@huawei.com,
	lersek@redhat.com, afaerber@suse.de, gaowanlong@cn.fujitsu.com
Subject: Re: [Qemu-devel] [PATCH V8 01/11] NUMA: add NumaOptions, NumaNodeOptions and NumaMemOptions
Date: Thu, 22 Aug 2013 21:21:54 +0200	[thread overview]
Message-ID: <521664D2.9030900@redhat.com> (raw)
In-Reply-To: <52157770.7030803@redhat.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Il 22/08/2013 04:29, Eric Blake ha scritto:
>>>>> If everything is optional, then what defaults are used if
>>>>> I specify nothing?  Should nodeid be mandatory (here, and
>>>>> in NumaNodeOptions)?
>>> 
>>> The defaults are all consistent with original behaviour. If 
>>> nodeid is omitted, the option will be assigned node by node 
>>> from node0.
> What will be assigned?  If I omit both nodeid and size, there's 
> nothing left in the object I'm passing.  Just because HMP can do 
> sane defaults doesn't mean that QMP needs to mark all fields as 
> optional.

Even though this is for the command-line, I agree that both should be
mandatory.  The default (splitting memory equally across nodes) can be
specified by not having "-numa mem" at all, so there is no need to
have short versions of the "-numa mem" command line.

Paolo
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJSFmTSAAoJEBvWZb6bTYbyzUsP/3BhPGBHfzNsFHdwPL6aiBW4
67TJJ/X9GB4wjav6yjtKtBGuDNOGmEO8BkyJZVNUzw051t74uPpYzDIgd7QvrsU/
nMFRFX6q3vRUxX7k9L5RP3OkJbcAgaq8YQ135XNh59bOEd/VOeTrrEtgqyVPo45h
6tt0JSlGDPF9qc+jGoqgQN8522nvV8mcF/XhJdQ7DqttpAP75ECxViKFKezXwrsh
H3dX6v0SbWBMAlihldxEMte8hlKNB/Asb3CDQz5L2ySaPGHcwOI06wSVKGD9slbv
4SSiGg2XMdPx0VKh0ozlKmXE46+hlDJkxi7JLEuscSmZL9gZwFpenz2+29OBk2MM
g8fwxKkm36jJE7cnn/z3TPiTVZ7sJm8LvtRW2183CC5I9LpTIUrCZxxP/06ZOREq
2fH41A3PyxDOvhOwp8B3XDAnOkmklSr86hiZR+s3I+59uPa9KzOY2WFeG/6w90uq
6EmZNbmkLrZYfns7wk24YRhH9DAgTFP9GmceaJu0lEbk41INeZggswugwPvrFNI3
bPsXacwK5Y4T4ZkRs4vyaw4Dn2B732D2CMvmPJ+EpKeVIc+8tKMXNtuFKybiraQi
zrlHk4zGjeUZwW8+cx8xkHe3qyKUwbYFuRvDqSKXYRrQMzzJNMf/rNGb4JcDHiH6
BmVejHjaYKPJKA+LF3hB
=Oqhm
-----END PGP SIGNATURE-----

  parent reply	other threads:[~2013-08-22 19:22 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-20  1:07 [Qemu-devel] [PATCH V8 00/11] Add support for binding guest numa nodes to host numa nodes Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 01/11] NUMA: add NumaOptions, NumaNodeOptions and NumaMemOptions Wanlong Gao
2013-08-21 20:59   ` Eric Blake
2013-08-22  1:12     ` Wanlong Gao
2013-08-22  2:29       ` Eric Blake
2013-08-22  3:16         ` Wanlong Gao
2013-08-22  8:46           ` Laszlo Ersek
2013-08-22 16:14             ` Eric Blake
2013-08-22 16:36               ` Laszlo Ersek
2013-08-22 19:21         ` Paolo Bonzini [this message]
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 02/11] NUMA: split -numa option Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 03/11] NUMA: move numa related code to numa.c Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 04/11] NUMA: Add numa_info structure to contain numa nodes info Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 05/11] NUMA: Add Linux libnuma detection Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 06/11] NUMA: parse guest numa nodes memory policy Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 07/11] NUMA: set " Wanlong Gao
2013-08-20 13:41   ` Andrew Jones
2013-08-21  2:43     ` Wanlong Gao
2013-08-21  7:15       ` Andrew Jones
2013-08-21  7:23         ` Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 08/11] NUMA: add qmp command set-mem-policy to set memory policy for NUMA node Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 09/11] NUMA: add hmp command set-mem-policy Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 10/11] NUMA: add qmp command query-numa Wanlong Gao
2013-08-20  1:07 ` [Qemu-devel] [PATCH V8 11/11] NUMA: convert hmp command info_numa to use qmp command query_numa Wanlong Gao
2013-08-20 13:43 ` [Qemu-devel] [PATCH V8 00/11] Add support for binding guest numa nodes to host numa nodes Paolo Bonzini
2013-08-21  1:22   ` Wanlong Gao
2013-08-21  9:00     ` Paolo Bonzini
2013-08-21  9:08       ` Wanlong Gao
2013-08-21  9:22         ` Paolo Bonzini
2013-08-21  9:34           ` Wanlong Gao
2013-08-21 10:00             ` Paolo Bonzini
2013-08-21  9:33 ` Laszlo Ersek
2013-08-21  9:37   ` Wanlong Gao

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=521664D2.9030900@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=afaerber@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=bsd@redhat.com \
    --cc=eblake@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=gaowanlong@cn.fujitsu.com \
    --cc=hutao@cn.fujitsu.com \
    --cc=lcapitulino@redhat.com \
    --cc=lersek@redhat.com \
    --cc=peter.huangpeng@huawei.com \
    --cc=qemu-devel@nongnu.org \
    --cc=y-goto@jp.fujitsu.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.