On Thu, Jan 07, 2016 at 22:49:57 -0500, Cole Robinson wrote:
Seems like the natural fit, since we are already adding other XML
bits
in the PostParse routine.
Previously AddImplicitControllers was only called at the end of XML
parsing, meaning code that builds a DomainDef by hand had to manually
call it. Adding it for those sites causes some test suite churn.
---
[...]
57 files changed, 59 insertions(+), 4 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 2570f94..5b9dab9 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -3854,6 +3854,9 @@ virDomainDefPostParseInternal(virDomainDefPtr def,
if (virDomainDefPostParseTimer(def) < 0)
return -1;
+ if (virDomainDefAddImplicitControllers(def) < 0)
+ return -1;
+
Moving it here makes this called twice in case you use
qemuParseCommandLine or virVMXParseConfig.
/* clean up possibly duplicated metadata entries */
virDomainDefMetadataSanitize(def);
The changes to the test suite look good to me but a more XEN
knowledgeable person could comment on this fact possibly so that we are
sure.