Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] fix free of event channel in blkfront

To: "Steven Hand" <Steven.Hand@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] fix free of event channel in blkfront
From: "Kasai Takanori" <kasai.takanori@xxxxxxxxxxxxxx>
Date: Tue, 9 Jan 2007 19:48:42 +0900
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 09 Jan 2007 02:49:05 -0800
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>
References: <E1H0LwV-0005UQ-00@xxxxxxxxxxxxxxxxx><023701c73385$dbfd0e50$dab2220a@VF03007L> <071e01c733b6$0845e010$dab2220a@VF03007L>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Hi Steven,

We tested the xm block-attach/detach command.
It repeats block-attach/detach command for DomU and pv-on-hvm on HVM Domain.
(block-attach -> block-detach -> block-attach -> block-detach -> ...)

The block-attach command failed when repeating 256 times.

I can't reproduce this - just tested 500 or so attach/detach pairs in
a tight loop and worked for me. However I was using a regular domU -
is it pv-on-hvm only? Can you give a more detailed repro case (or a
script)?

It already seems to have been corrected.

・xen-unstable.hg : cs 13198
 [PV-ON-HVM] Update evtchn interface to match new PV Linux interfaces.

----------------------------------------------------------------------------------------------------
--- a/unmodified_drivers/linux-2.6/platform-pci/evtchn.c Sat Dec 30 18:23:27 2006 +0000 +++ b/unmodified_drivers/linux-2.6/platform-pci/evtchn.c Sun Dec 31 11:59:47 2006 +0000
@@ -118,8 +144,12 @@ void unbind_from_irqhandler(unsigned int

        mask_evtchn(evtchn);
        evtchns[evtchn].handler = NULL;
+
+       if (evtchns[evtchn].close) {
+               struct evtchn_close close = { .port = evtchn };
+               HYPERVISOR_event_channel_op(EVTCHNOP_close, &close);
+       }
}
-
EXPORT_SYMBOL(unbind_from_irqhandler);

void notify_remote_via_irq(int irq)
----------------------------------------------------------------------------------------------------


However, the following problems remain.
I think that it operates if the following problems can be solved.

It is necessary to correct pv-on-hvm according to the following corrections.

・xen-unstable.hg : cs 13197
 [LINUX] Extend the event-channel interfaces to provide helper methods


I confirmed it with up-to-date xen-unstable.hg. (cs 13282)

It did not reproduce it in DomU.
Because VBD was not able to be used, pv-on-hvm was not able to confirm it.
The following errors occur.

# insmod xen-vbd.ko
xen-vbd: Unknown symbol irq_to_evtchn_port
insmod: error inserting 'xen-vbd.ko': -1 Unknown symbol in module

Thanks,

--
Takanori Kasai


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