Re: [libvirt] [PATCH] schemas: capability: Add ostype 'aix'

On Tue, Apr 21, 2015 at 02:34:09PM -0600, Eric Blake wrote: > On 04/21/2015 02:17 PM, Cole Robinson wrote: >> --- >> docs/schemas/capability.rng | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/docs/schemas/capability.rng b/docs/schemas/capability.rng >> index 5f3ec70..3868ee2 100644 >> --- a/docs/schemas/capability.rng >> +++ b/docs/schemas/capability.rng >> @@ -266,6 +266,7 @@ >> <value>hvm</value> <!-- unmodified OS --> >> <value>exe</value> <!-- For container based virt --> >> <value>uml</value> <!-- user mode linux --> >> + <value>aix</value> <!-- used by phyp driver --> > > ACK Actually I don't think this is correct - I think it is another case of the phyp driver being rather confused. The phyp driver doesn't use the def->os.type anywhere in its code as far as I can tell. The place where 'aix' is used is bizarrely in the phypDomainAttachDevice method where it calls virDomainDefNew() and then sets os.type to 'aix' before calling virDomainDeviceDefParse. This is all just bizarre, but the 'aix' type doesn't appear to ever leak out to the end user in the XML unless I'm missing something. So I don't think we should add this to the RNG.