vcpu: improve cpuset attribute

The <vcpu cpuset=...> attribute has been available since commit
e193b5dd, but without documentation or RNG validation.

* docs/schemas/domain.rng (vcpu): Further validate cpuset.
* docs/formatdomain.html.in: Document it.
* src/conf/domain_conf.c: Fix typos.
This commit is contained in:
Eric Blake 2010-09-27 17:29:13 -06:00
parent 53a2f725ad
commit 62a50a0b80
3 changed files with 21 additions and 6 deletions

View File

@ -194,7 +194,7 @@
&lt;memoryBacking&gt;
&lt;hugepages/&gt;
&lt;/memoryBacking&gt;
&lt;vcpu&gt;1&lt;/vcpu&gt;
&lt;vcpu cpuset="1-4,^3,6"&gt;2&lt;/vcpu&gt;
...</pre>
<dl>
@ -213,7 +213,15 @@
hugepages instead of the normal native page size.</dd>
<dt><code>vcpu</code></dt>
<dd>The content of this element defines the number of virtual
CPUs allocated for the guest OS.</dd>
CPUs allocated for the guest OS, which must be between 1 and
the maximum supported by the hypervisor. <span class="since">Since
0.4.4</span>, this element can contain an optional
<code>cpuset</code> attribute, which is a comma-separated
list of physical CPU numbers that virtual CPUs can be pinned
to. Each element in that list is either a single CPU number,
a range of CPU numbers, or a caret followed by a CPU number to
be excluded from a previous range.
</dd>
</dl>
<h3><a name="elementsCPU">CPU model and topology</a></h3>

View File

@ -302,7 +302,9 @@
<optional>
<element name="vcpu">
<optional>
<attribute name="cpuset"/>
<attribute name="cpuset">
<ref name="cpuset"/>
</attribute>
</optional>
<ref name="countCPU"/>
</element>
@ -1785,6 +1787,11 @@
<param name="pattern">[0-9]+</param>
</data>
</define>
<define name="cpuset">
<data type="string">
<param name="pattern">([0-9]+(-[0-9]+)?|\^[0-9]+)(,([0-9]+(-[0-9]+)?|\^[0-9]+))*</param>
</data>
</define>
<define name="countCPU">
<data type="unsignedShort">
<param name="pattern">[0-9]+</param>

View File

@ -5270,7 +5270,7 @@ virDomainCpuNumberParse(const char **str, int maxcpu)
*
* Serialize the cpuset to a string
*
* Returns the new string NULL in case of error. The string need to be
* Returns the new string NULL in case of error. The string needs to be
* freed by the caller.
*/
char *
@ -5329,8 +5329,8 @@ virDomainCpuSetFormat(char *cpuset, int maxcpu)
* @maxcpu: number of elements available in @cpuset
*
* Parse the cpu set, it will set the value for enabled CPUs in the @cpuset
* to 1, and 0 otherwise. The syntax allows coma separated entries each
* can be either a CPU number, ^N to unset that CPU or N-M for ranges.
* to 1, and 0 otherwise. The syntax allows comma separated entries; each
* can be either a CPU number, ^N to unset that CPU, or N-M for ranges.
*
* Returns the number of CPU found in that set, or -1 in case of error.
* @cpuset is modified accordingly to the value parsed.