On 07/03/2012 06:18 PM, Michal Privoznik wrote:
On 29.06.2012 17:02, Viktor Mihajlovski wrote:
Added s390-virtio machine type to the XML schema for domains in order
to not fail the domain schema tests.

Signed-off-by: Viktor Mihajlovski<mihaj...@linux.vnet.ibm.com>
---
  docs/schemas/domaincommon.rng |   20 ++++++++++++++++++++
  1 files changed, 20 insertions(+), 0 deletions(-)

diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng
index 912a1a2..70c7d16 100644
--- a/docs/schemas/domaincommon.rng
+++ b/docs/schemas/domaincommon.rng
@@ -283,6 +283,7 @@
            <ref name="hvmsparc"/>
            <ref name="hvmppc"/>
            <ref name="hvmppc64"/>
+<ref name="hvms390"/>
          </choice>
        </optional>
        <value>hvm</value>
@@ -369,6 +370,25 @@
        </optional>
      </group>
    </define>
+<define name="hvms390">
+<group>
+<optional>
+<attribute name="arch">
+<choice>
+<value>s390</value>
+<value>s390x</value>
+</choice>
+</attribute>
+</optional>
+<optional>
+<attribute name="machine">
+<choice>
+<value>s390-virtio</value>
+</choice>
+</attribute>
+</optional>
+</group>
+</define>
    <define name="osexe">
      <element name="os">
        <element name="type">


Sorry cannot ACK this one until you update the documentation as well.

Michal


Hi Michal,

actually I was pondering about a doc update when preparing the patches. I only wasn't clear where to put it. The only place where possible arch/machine values are mentioned seems to be in formatcaps.html.in. Would you expect me to add a sample output of the capabilities XML for s390 with some comments in there, or did you have something else in mind?

Thanks.

--

Mit freundlichen Grüßen/Kind Regards
   Viktor Mihajlovski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to