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

Re: [Xen-devel] [PATCH] Documentation: Include cpupool example in instal

To: André Przywara <andre.przywara@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Documentation: Include cpupool example in install
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Tue, 15 Feb 2011 15:14:00 +0000
Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 15 Feb 2011 07:15:27 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D59C89B.6060207@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>
Newsgroups: chiark.mail.xen.devel
References: <e66f078e198dde642fd3.1297699182@elijah> <4D59C89B.6060207@xxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
André Przywara writes ("Re: [Xen-devel] [PATCH] Documentation: Include cpupool 
example in install"):
> Is this just a limit in the implementation or is the requirement for a 
> regular on-disk file there for a reason?

It's a limit in the implementation.  We try to read the config file
into a char array in memory, and that's most easily done by
stat/malloc/fread.

For a domain, the config file is used both immediately when creating
the domain (being parsed by being fed to the flex bytearray scanner
entrypoint) but also stored for use when the domain is migrated/saved.

> I could easily create a patch to do away with these restrictions (after 
> the release), are there any arguments against this?

No, that would be fine.  The right answer is probably somewhere around
libxl_read_file_contents.  Perhaps we need libxl_read_fd_contents
which reallocs.

Ian.

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