Re: [libvirt] [RFC PATCH v2 02/12] domain_conf: introduce cpu def helpers

On Wed, Feb 04, 2015 at 05:18:20PM +0800, Zhu Guihua wrote: > virDomainCPUDefFree - free memory allocated > virDomainCPUDefParseXML - parse job type > virDomainCPUDefFormat - output job type > > Signed-off-by: Zhu Guihua <zhugh.fnst(a)cn.fujitsu.com&gt; > --- > docs/formatdomain.html.in | 28 +++++++++++++++ > docs/schemas/domaincommon.rng | 3 ++ > src/conf/domain_conf.c | 82 +++++++++++++++++++++++++++++++++++++++++++ > src/conf/domain_conf.h | 1 + > src/libvirt_private.syms | 1 + > 5 files changed, 115 insertions(+) > > diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in > index f8d5f89..10cbd29 100644 > --- a/docs/formatdomain.html.in > +++ b/docs/formatdomain.html.in > @@ -5830,6 +5830,34 @@ qemu-kvm -net nic,model=? /dev/null > </dd> > </dl> > > + <h4><a name="elementsCpu">CPU device</a></h4> > + <p> > + CPU device allows to be hot added to the guest. > + <span class="since">Since 1.2.12, QEMU and KVM only</span> > + </p> > + > +<pre> > + ... > + &lt;devices&gt; > + &lt;cpu match='exact'&gt; > + &lt;model fallback='allow'&gt;core2duo&lt;/model&gt; > + &lt;vendor&gt;Intel&lt;/vendor&gt; > + &lt;topology sockets='1' cores='2' threads='1'/&gt; > + &lt;feature policy='disable' name='lahf_lm'/&gt; > + &lt;/cpu&gt; > + &lt;devices&gt; NACK, it is just madness to duplicate the CPU info we already have in a different place in the XML.

Regards, Daniel