[Xen-devel] Re: [Xen-ia64-devel] [PATCH] Close nvram file when rebooting HVM domain

[Alex Williamson <alex.williamson@xxxxxx>]

On Tue, 2007-12-11 at 11:11 +0900, Masaki Kanno wrote:
> 
> Hi,
> 
> On ia64, when I rebooted a HVM domain, a nvram file for the HVM domain 
> was not closed.  When I repeated rebooting the HVM domain, a lot of the 
> nvram files have still opened as follows. 
> 
> # ps aux | grep xend
> root      4215  0.0  0.2  81600 11840 ?        S    10:00   0:00 python 
> /usr/sbin/xend start
> root      4221  0.3  0.7 223664 29136 ?        Sl   10:00   0:03 python 
> /usr/sbin/xend start
> root      5189  0.0  0.0  60432  1824 pts/0    S+   10:17   0:00 grep xend
> # ls -l /proc/4221/fd | grep nvram
> lrwx------ 1 root root 64 Dec 11 10:05 23 -> 
> /var/lib/xen/nvram/nvram_HVMdomain.1
> lrwx------ 1 root root 64 Dec 11 10:14 24 -> 
> /var/lib/xen/nvram/nvram_HVMdomain.1
> lrwx------ 1 root root 64 Dec 11 10:14 25 -> 
> /var/lib/xen/nvram/nvram_HVMdomain.1
> lrwx------ 1 root root 64 Dec 11 10:16 26 -> 
> /var/lib/xen/nvram/nvram_HVMdomain.1
> lrwx------ 1 root root 64 Dec 11 10:08 4 -> 
> /var/lib/xen/nvram/nvram_HVMdomain.1
> 
> This patch closes a nvram file for a HVM domain when the HVM domain 
> rebooted.
> 
> 
> Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>

Keir,

   This would be a nice fix to get in for 3.2.  Thanks

        Alex

Acked-by: Alex Williamson <alex.williamson@xxxxxx>
> 
> diff -r 4054cd60895b tools/python/xen/xend/XendDomainInfo.py
> --- a/tools/python/xen/xend/XendDomainInfo.py   Mon Dec 10 13:49:22
> 2007 +0000
> +++ b/tools/python/xen/xend/XendDomainInfo.py   Mon Dec 10 18:44:37
> 2007 +0900
> @@ -1880,8 +1880,6 @@ class XendDomainInfo:
>  
>          self._cleanupVm()
>          if self.dompath is not None:
> -            if self.domid is not None:
> -                xc.domain_destroy_hook(self.domid)
>              self.destroyDomain()
>  
>          self._cleanup_phantom_devs(paths)
> @@ -1899,6 +1897,7 @@ class XendDomainInfo:
>  
>          try:
>              if self.domid is not None:
> +                xc.domain_destroy_hook(self.domid)
>                  xc.domain_destroy(self.domid)
>                  for state in DOM_STATES_OLD:
>                      self.info[state] = 0
> 
-- 
Alex Williamson                             HP Open Source & Linux Org.


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