This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] [PATCH][XEND]Fix checkname so that it detects duplicate doma

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH][XEND]Fix checkname so that it detects duplicate domains.
From: "Petersson, Mats" <Mats.Petersson@xxxxxxx>
Date: Thu, 24 May 2007 11:57:20 +0200
Delivery-date: Thu, 24 May 2007 02:56:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Aced6etScrTUhsnqQ+CeH6LEai4TSA==
Thread-topic: [PATCH][XEND]Fix checkname so that it detects duplicate domains.
When using "xm restore" or "xm create" it's possible to create multiple
instances of the same domain. 

With this patch, it is not allowed to have multiple instances with the
same name. This should still work for live migration to localhost, as
the migration-path changes the name of the original domain-name, so the
original domain will not name-clash with the migration-target-domain. 

The previous behaviour was to check that the UUID is different, but for
domains that either have UUID in the config file, or that are the result
of restore, the UUID is definitely going to be the same for multiple
instances, so I believe this isn't a correct behaviour in the first

There is still a bit of a problem that "xm restore" doesn't report the
error correctly - it does say "Restore failed", but not explicitly that
it was caused by using an already existing name.

Signed off by: Mats Petersson (mats.petersson@xxxxxxx)


Attachment: patch.fix_checkname
Description: patch.fix_checkname

Xen-devel mailing list