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

Fw: [Xen-devel] [PATCH] [Xen] [ACM] Code restructuring

To: Keir Fraser <keir@xxxxxxxxxxxxx>
Subject: Fw: [Xen-devel] [PATCH] [Xen] [ACM] Code restructuring
From: Stefan Berger <stefanb@xxxxxxxxxx>
Date: Tue, 24 Apr 2007 12:21:19 -0400
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 24 Apr 2007 09:19:54 -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

Stefan Berger/Watson/IBM wrote on 04/24/2007 12:15:11 PM:

> Keir Fraser <keir@xxxxxxxxxxxxx> wrote on 04/24/2007 11:55:51 AM:
>
> > Much nicer, thank you. I've applied it for 3.0.5. However, given the effort
> > put in to labelling a domain at the very start of its lifetime, wouldn't it
> > be a good idea to leave that label attached until the domain is actually
> > destroyed? This would correspond to putting the destructor call in
> > domain_destroy() or, even better, complete_domain_destroy(). Bear in mind
> > that a domain may still be accessible and have resources allocated to it
> > even after domain_kill() finishes executing. So stripping its resource label
> > at that point in time is a bit worrying.

>
> Right. Can you move this one call into complete_domain_destroy()?


And thanks for pointing this out.

  Stefan

>
>    Stefan
> >
> >  -- Keir
> >
> > On 24/4/07 17:17, "Stefan Berger" <stefanb@xxxxxxxxxx> wrote:
> >
> > > This patch restructures part of the ACM code in Xen. When a domain is
> > > created, the function acm_domain_create() in domain_create() is called
> > > that does what previously the pre- and post_domain_create functions were
> > > doing. Similarly there's a function acm_domain_destroy() in
> > > domain_kill() that reverts changes to state when destroying a domain.
> > > There's no more separate initialization necessary for domain-0, but
> > > domain_create takes one more parameter, the ssidref. It is usually
> > > passed through the hypercall when a domain is created.
> > >
> > > Signed-off-by: Stefan Berger <stefanb@xxxxxxxxxx>
> >
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>