Stefan Berger
2007-Apr-24 16:21 UTC
Fw: [Xen-devel] [PATCH] [Xen] [ACM] Code restructuring
Stefan Berger/Watson/IBM wrote on 04/24/2007 12:15:11 PM:> Keir Fraser <keir@xensource.com> wrote on 04/24/2007 11:55:51 AM: > > > Much nicer, thank you. I''ve applied it for 3.0.5. However, given theeffort> > 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 isactually> > destroyed? This would correspond to putting the destructor call in > > domain_destroy() or, even better, complete_domain_destroy(). Bear inmind> > that a domain may still be accessible and have resources allocated toit> > even after domain_kill() finishes executing. So stripping its resourcelabel> > 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@us.ibm.com> wrote: > > > > > This patch restructures part of the ACM code in Xen. When a domainis> > > created, the function acm_domain_create() in domain_create() iscalled> > > that does what previously the pre- and post_domain_create functionswere> > > doing. Similarly there''s a function acm_domain_destroy() in > > > domain_kill() that reverts changes to state when destroying adomain.> > > 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@us.ibm.com> > >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel