This patch moves duplicate name checking out of XendDomainInfo (the item class), and places it in XendDomain (the container class). For the restore case, I moved the code that extracts the config information from a state image into its own function so that the container class can validate the info before initiating the restore. This was done in an attempt to remove yet another mutual dependency between different levels of abstraction. Additionally, I could not find in the existing code the point at which a domain was added to the XendDomain list after restore. Several attempts to restore would result in positive log messages, but the restored domain would not show up in "xm list". This patch includes a call to _add_domain(), which results in restore working for me. Comments are welcomed, especially related to the last point. Signed-off-by: Dan Smith <danms@us.ibm.com> -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms@us.ibm.com _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel