Masaki Kanno
2009-Sep-17 07:32 UTC
[Xen-devel] [PATCH] xend: Unlink VDI instances and VBD instances
Hi, When VBD instances are destroyed by xm delete command, VDI instances keep linking to the VBD instances unilaterally. This patch eliminates the unilateral linking. Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com> Best regards, Kan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Keir Fraser
2009-Sep-18 13:43 UTC
Re: [Xen-devel] [PATCH] xend: Unlink VDI instances and VBD instances
Applied as c/s 20221, but it produces the following backtrace, so I reverted it: Traceback (most recent call last): File "/usr/lib/python2.4/site-packages/xen/xend/XendCheckpoint.py", line 165, in save dominfo.destroy() File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line 2855, in destroy XendDomain.instance().remove_domain(self) File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 468, in remove_domain self._remove_domain(info, domid) File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 489, in _remove_domain info.unlink_xapi_instances() File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line 4054, in unlink_xapi_instances vdi = XendNode.instance().get_vdi_by_uuid(dev_info[''VDI'']) KeyError: ''VDI'' -- Keir On 17/09/2009 08:32, "Masaki Kanno" <kanno.masaki@jp.fujitsu.com> wrote:> Hi, > > When VBD instances are destroyed by xm delete command, VDI instances > keep linking to the VBD instances unilaterally. > This patch eliminates the unilateral linking. > > Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com> > > Best regards, > Kan >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Masaki Kanno
2009-Sep-24 01:32 UTC
Re: [Xen-devel] [PATCH] xend: Unlink VDI instances and VBD instances
Hi Keir, Sorry. I didn''t test the patch on a server to which XenAPI mode was invalid. I will send a new patch. Best regards, Kan Fri, 18 Sep 2009 14:43:55 +0100, Keir Fraser wrote:>Applied as c/s 20221, but it produces the following backtrace, so I reverted >it: >Traceback (most recent call last): > File "/usr/lib/python2.4/site-packages/xen/xend/XendCheckpoint.py", line >165, in save > dominfo.destroy() > File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line >2855, in destroy > XendDomain.instance().remove_domain(self) > File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 468, >in remove_domain > self._remove_domain(info, domid) > File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 489, >in _remove_domain > info.unlink_xapi_instances() > File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line >4054, in unlink_xapi_instances > vdi = XendNode.instance().get_vdi_by_uuid(dev_info[''VDI'']) >KeyError: ''VDI'' > > -- Keir > > >On 17/09/2009 08:32, "Masaki Kanno" <kanno.masaki@jp.fujitsu.com> wrote: > >> Hi, >> >> When VBD instances are destroyed by xm delete command, VDI instances >> keep linking to the VBD instances unilaterally. >> This patch eliminates the unilateral linking. >> >> Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com> >> >> Best regards, >> Kan >> > > > >_______________________________________________ >Xen-devel mailing list >Xen-devel@lists.xensource.com >http://lists.xensource.com/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Masaki Kanno
2009-Sep-24 08:20 UTC
[Xen-devel] [PATCH v2] xend: Unlink VDI instances and VBD instances
Hi Keir, VBD information in xend does not have a VDI value if XenAPI mode is invalid. A new patch confirms the VDI value is valid. So, the new patch cuts off VDI->VBD links if the VDI value is valid. Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com> Best regards, Kan Thu, 24 Sep 2009 10:32:33 +0900, Masaki Kanno wrote:>Hi Keir, > >Sorry. I didn''t test the patch on a server to which XenAPI mode was >invalid. I will send a new patch. > >Best regards, > Kan > >Fri, 18 Sep 2009 14:43:55 +0100, Keir Fraser wrote: > >>Applied as c/s 20221, but it produces the following backtrace, so I reverted >>it: >>Traceback (most recent call last): >> File "/usr/lib/python2.4/site-packages/xen/xend/XendCheckpoint.py", line >>165, in save >> dominfo.destroy() >> File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line >>2855, in destroy >> XendDomain.instance().remove_domain(self) >> File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 468, >>in remove_domain >> self._remove_domain(info, domid) >> File "/usr/lib/python2.4/site-packages/xen/xend/XendDomain.py", line 489, >>in _remove_domain >> info.unlink_xapi_instances() >> File "/usr/lib/python2.4/site-packages/xen/xend/XendDomainInfo.py", line >>4054, in unlink_xapi_instances >> vdi = XendNode.instance().get_vdi_by_uuid(dev_info[''VDI'']) >>KeyError: ''VDI'' >> >> -- Keir >> >> >>On 17/09/2009 08:32, "Masaki Kanno" <kanno.masaki@jp.fujitsu.com> wrote: >> >>> Hi, >>> >>> When VBD instances are destroyed by xm delete command, VDI instances >>> keep linking to the VBD instances unilaterally. >>> This patch eliminates the unilateral linking. >>> >>> Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com> >>> >>> Best regards, >>> Kan >>> >> >> >> >>_______________________________________________ >>Xen-devel mailing list >>Xen-devel@lists.xensource.com >>http://lists.xensource.com/xen-devel > > >_______________________________________________ >Xen-devel mailing list >Xen-devel@lists.xensource.com >http://lists.xensource.com/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel