WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH,v2] xl: randomly generate UUID's

To: Christoph Egger <Christoph.Egger@xxxxxxx>
Subject: [Xen-devel] Re: [PATCH,v2] xl: randomly generate UUID's
From: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
Date: Fri, 20 Aug 2010 16:09:59 +0100
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, Xen Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Fri, 20 Aug 2010 08:15:34 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <201008161705.29426.Christoph.Egger@xxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1281970136.18490.395.camel@xxxxxxxxxxxxxxxxxxxxxx> <201008161705.29426.Christoph.Egger@xxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Mon, 2010-08-16 at 16:05 +0100, Christoph Egger wrote:
> On Monday 16 August 2010 16:48:56 Gianni Tedesco wrote:
> > This patch converts xl to randomly generate UUID's rather than using a
> > dodgy time-seeded PRNG. I have incorporated Cristoph Eggers suggestions
> > wrt. portability so I would be grateful for an ACK on the NetBSD side of
> > things. This also folds in the "v2: xl: make libxl_uuid2string internal
> > to libxenlight" patch since it couldn't be avoided in fixing up xl
> > printfs() which grok around in the uuid type directly.
> >
> > I have ignored various suggestions so far on auto-generation of MAC
> > addresses and left it as a topic for a future patch to solve. In other
> > words the behaviour stays the same it's just using a true random source.
> >
> > This will probably conflict horribly with "libxl: autogenerate type
> > definitions and destructor functions" so if it's acked in principle I
> > can re-base and re-send on top of that.
> 
> Please do. The patch looks almost good to me - see line comments -
> I would like to compile-test it at least on NetBSD before acknowledging
> the patch.
> 
> Christoph
> 
> >
> > Signed-off-by: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
> >
> > diff -r 85bd0f6e8fed tools/libxl/Makefile
> > --- a/tools/libxl/Makefile      Fri Aug 13 13:55:51 2010 +0100
> > +++ b/tools/libxl/Makefile      Mon Aug 16 15:39:13 2010 +0100
> > @@ -16,6 +16,9 @@ CFLAGS += -I. -fPIC
> >  CFLAGS += $(CFLAGS_libxenctrl) $(CFLAGS_libxenguest) $(CFLAGS_libxenstore)
> > $(CFLAGS_libblktapctl)
> >
> >  LIBS = $(LDLIBS_libxenctrl) $(LDLIBS_libxenguest) $(LDLIBS_libxenstore)
> > $(LDLIBS_libblktapctl) $(UTIL_LIBS)
> > +ifeq ($(CONFIG_Linux),y)
> > +LIBS += -luuid
> > +endif
> 
> This is not needed because $(UTIL_LIBS) already covers that case.

Not sure what you mean by this, if I omit that linen the I get link
errors. That's the way it's done in tools/blktap2/ and I just copied
that.

Am about to re-base and re-send this patch so please clarify if this is
indeed wrong for some reason.


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

<Prev in Thread] Current Thread [Next in Thread>