Re: [libvirt] [PATCH] Fix <cpu> element in domain XML schema

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

 



On Tue, Feb 09, 2010 at 03:53:03PM +0100, Jiri Denemark wrote:
> The current schema is more permissive than the XML parsing code in
> libvirt. Precisely, 'match' attribute is optional in schema while in
> reality its use is bound to <model> element:
> 
> - <cpu> element without 'match' attribute is allowed only if <topology>
>   element is the only child element of <cpu>
> 
> - <cpu> element with 'match' attribute requires <model> element to be
>   present; <topology> and <feature> elements are optional
> 
> Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>

  ACK, but there is a limit to this kind of refinements. Basically
the schemas will always have to be more permissive than the internals,
for example some constructs are allowed only for some values of
attribute type on top level element, and except making the schemas
unmaintainable, I don't think it's feasible to have the RNG model
exactly the actual set accepted. It's really 2 different models of
validation.

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
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]