comments below On 07/17/13 11:29, Wanlong Gao wrote: > Signed-off-by: Wanlong Gao <gaowanl...@cn.fujitsu.com> > --- > qapi-schema.json | 44 ++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 44 insertions(+) > > diff --git a/qapi-schema.json b/qapi-schema.json > index 7b9fef1..f753a35 100644 > --- a/qapi-schema.json > +++ b/qapi-schema.json > @@ -3679,3 +3679,47 @@ > '*cpuid-input-ecx': 'int', > 'cpuid-register': 'X86CPURegister32', > 'features': 'int' } } > + > +## > +# @NumaOptions > +# > +# A discriminated record of NUMA options. > +# > +# Since 1.6 > +## > +{ 'union': 'NumaOptions', > + 'data': { > + 'node': 'NumaNodeOptions', > + 'mem': 'NumaMemOptions' }} > + > +## > +# @NumaNodeOptions > +# > +# Create a guest NUMA node. > +# > +# @nodeid: #optional NUMA node ID > +# > +# @cpus: #optional VCPUs belong to this node > +# > +# Since: 1.6 > +## > +{ 'type': 'NumaNodeOptions', > + 'data': { > + '*nodeid': 'int', > + '*cpus': 'str' }} > +
Should we document the format for "cpus" here too? > +## > +# @NumaMemOptions > +# > +# Set memory information of guest NUMA node. > +# > +# @nodeid: #optional NUMA node ID > +# > +# @size: #optional memory size of this node > +# > +# Since 1.6 > +## > +{ 'type': 'NumaMemOptions', > + 'data': { > + '*nodeid': 'int', > + '*size': 'size' }} > Looks good in general but I'm not sure if hardware tabs are allowed (or usual) in this file. Thanks Laszlo