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

Re: [Xen-devel] [PATCH]: Fix xm block-detach

To: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH]: Fix xm block-detach
From: Chris Lalancette <clalance@xxxxxxxxxx>
Date: Wed, 03 Dec 2008 16:46:57 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 03 Dec 2008 07:48:20 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <97C954F0AE2572kanno.masaki@xxxxxxxxxxxxxx>
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>
References: <49352971.6020605@xxxxxxxxxx> <96C95486AAA6EEkanno.masaki@xxxxxxxxxxxxxx> <49354440.3080003@xxxxxxxxxx> <49356133.6070004@xxxxxxxxxx> <97C954F0AE2572kanno.masaki@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 2.0.0.18 (X11/20081119)
Masaki Kanno wrote:
> Tue, 02 Dec 2008 17:24:19 +0100, Chris Lalancette wrote:
> 
>> Chris Lalancette wrote:
>>> Masaki Kanno wrote:
>>>> Hi Chris,
>>>>
>>>> I could not reproduce the problem by using the latest xen-unstable.
>>>>
>>>> I also found a problem of block tap devices included by c/s 18562, 
>>>> then I have fixed the problem by c/s 18843.  But the problem occurred 
>>>> by using xm shutdown or xm destroy or etc, not xm block-detach.
>>>>
>>>> Could you try xm block-detach by using the latest xen-unstable?
>>>>
>>>> Best regards,
>>>>  Kan
>>> OK, interesting.  I'll give it a shot, but it's going to take a little 
>>> while
>>> since I have to build from scratch.  I'll report when I'm done.
>> Ah, now I see.  Testing it on xen-unstable does, indeed, show xm block-detach
>> working as expected.  There were some changes made in the meantime that 
>> actually
>> make it work.  That means the first hunk of my changes to DevController.py
>> aren't required.  However, I think the other two hunks are actually "
>> correct",
>> even though we don't see the xm block-detach bug in current xen-unstable.  
>> That
>> is, they move the device section from /vm/UUID/device/tap to
>> /vm/UUID/device/vbd, which seems more right to me.
> 
> Hi Chris,
> 
> I have tried the other two hunks of your changes on the latest 
> xen-unstable.  I have found two issues.  Could you see the attaching 
> file?
> 
> 1. Information of xm list
>   When I tried xm list to an active domain, both "vbd" and "tap" 
>   were shown.  They were same uuid and same uname.
> 
> 2. Double wait in xend
>   I checked xend.log after I started a domain.  Xend was waiting 
>   for both "vdb" and "tap" by using waitForDevices().
> 
> If the other two hunks of your changes are correct, I think that 
> there is some kind of lack in your changes yet.

Yes, I see what you mean (both from the data you sent, and from reproducing it
on my own).  There is probably something missing.  Honestly, I'm not sure how
sxpr's are generated, so I'll have to look into it.  Probably not until tomorrow
though.  Thanks for testing.

-- 
Chris Lalancette

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

<Prev in Thread] Current Thread [Next in Thread>