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


Re: [Xen-devel] [PATCH 0 of 3] xl: carry domain config around in struct

To: Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 0 of 3] xl: carry domain config around in struct domain_config
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Mon, 26 Jul 2010 15:44:36 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 26 Jul 2010 07:46:00 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1007261539080.22235@kaball-desktop>
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>
Organization: Citrix Systems, Inc.
References: <patchbomb.1280146832@xxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1007261539080.22235@kaball-desktop>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Mon, 2010-07-26 at 15:40 +0100, Stefano Stabellini wrote:
> On Mon, 26 Jul 2010, Ian Campbell wrote:
> > Move more domain configuration items into struct domain_config.
> > 
> > Reduces the number of functions which take large numbers of parameters
> > and generally just keeps configuration stuff in one place.
> Ack the whole series. Kudos to emacs search and replace :)

I for one welcome our Ctrl-Alt-Shift-5 pressing overlords.

On a related note... I was wondering if perhaps it might make sense to
try and refactor some of the xl functionality, e.g. create (and
associated stuff like resume) into separate files, just to try and make
it a little more tractable.


Xen-devel mailing list