Re: [PATCH 1/2] schema: Fix the definition of SEV in domain capabilities schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 14, 2018 at 02:28:02PM +0200, Erik Skultety wrote:
The whole <sev> element was optional which it shouldn't be as if the
platform doesn't support SEV we should format <sev supported='no'/>.

Signed-off-by: Erik Skultety <eskultet@xxxxxxxxxx>
---
docs/schemas/domaincaps.rng | 19 ++++++++++---------
1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/docs/schemas/domaincaps.rng b/docs/schemas/domaincaps.rng
index e25201fc68..ed29823548 100644
--- a/docs/schemas/domaincaps.rng
+++ b/docs/schemas/domaincaps.rng
@@ -185,9 +185,7 @@
        <ref name='gic'/>
        <ref name='vmcoreinfo'/>
        <ref name='vmgenid'/>
-        <optional>
-          <ref name='sev'/>
-        </optional>
+        <ref name='sev'/>
      </interleave>
    </element>
  </define>
@@ -213,12 +211,15 @@

  <define name='sev'>
    <element name='sev'>
-      <element name='cbitpos'>
-        <data type='unsignedInt'/>
-      </element>
-      <element name='reducedPhysBits'>
-        <data type='unsignedInt'/>
-      </element>
+      <ref name='supported'/>
+      <optional>
+        <element name='cbitpos'>
+          <data type='unsignedInt'/>
+        </element>
+        <element name='reduced-phys-bits'>

Actually, s/reduced-phys-bits/reducedPhysBits/

+          <data type='unsignedInt'/>
+        </element>
+      </optional>
    </element>
  </define>

--
2.14.4

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

Attachment: signature.asc
Description: Digital signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux