RE: [Xen-devel] [PATCH][XEND]Fix checkname so that it detects duplicate domains.

["Petersson, Mats" <Mats.Petersson@xxxxxxx>]

> -----Original Message-----
> From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx] 
> Sent: 25 May 2007 10:21
> To: Petersson, Mats; xen-devel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH][XEND]Fix checkname so that 
> it detects duplicate domains.
> 
> On 24/5/07 10:57, "Petersson, Mats" <Mats.Petersson@xxxxxxx> wrote:
> 
> > 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
> > place. 
> 
> If we don't enforce UUID uniqueness, what is the point of 
> having a UUID?
> Also, don't we store some VM information in /vm/<uuid> in 
> xenstore: how does
> that work out if we have multiple domains with the same UUID?

All very good points. I don't actually know how this is meant to work,
I'm just fixing an apparent bug, which is that if the UUID is
duplicated, you can have two domains with the same name, which isn't
what is supposed to happen. There should probably ALSO be a "_checkuuid"
function to verify that the UUID is unique. Not sure it's my place to
fix that, tho'?

I still think my fix is valid, although, of course, if the UUID is
always guaranteed to be unique even during save/restore/migration within
the same machine, then we could of course use the UUID instead of the
DOMID. 

--
Mats
> 
>  -- Keir
> 
> 
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel