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] pci device hotplug, race accessing xenstore

To: Simon Horman <horms@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] pci device hotplug, race accessing xenstore
From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Date: Fri, 16 Oct 2009 13:37:11 +0100
Cc: Phung Te Ha <phungte@xxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Fri, 16 Oct 2009 05:37:01 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20091014224918.GD9650@xxxxxxxxxxxx>
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: <f6cf36180910131213x21218f2am7ed55c0a8a381312@xxxxxxxxxxxxxx> <alpine.DEB.2.00.0910141351140.11134@kaball-desktop> <20091014224918.GD9650@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Alpine 2.00 (DEB 1167 2008-08-23)
On Wed, 14 Oct 2009, Simon Horman wrote:
> > 
> > I think we should take this chance to make the pci-insert protocol more
> > reliable.
> > In particular we are missing the following things:
> > 
> > - qemu shouldn't accept any dm-command unless it is in state "running";
> > 
> > - xend should remove the command node on xenstore after reading
> > state "pci-inserted" and before writing state "running"  again.
> > 
> > This way when the second xenstore watch fires the pci-ins command is
> > never executed for a second time because either qemu is not in the right
> > state (pci-inserted instead of running) or the command node doesn't
> > contain any data (it has been removed by xend).
> 
> My memory of that code is a bit hazy, but that sounds like a good idea.

Do you think you'll find the time to fix the first two issues, or
someone else should do it?

I should be able to find a solution for the stubdom coldplug problem
myself.

> > Another problem is that nothing else can happen while xend waits for the
> > device model to be in state running, this also prevents pci coldplug
> > from working with stubdoms.
> > Is it possible to run signalDeviceModel in a new xend Thread?
> 
> I'm interested to hear a comment on what the status of the Ocaml
> replacement for xend is. It seems silly to spend time fixing up the
> python code - there is ample scope for fixing - if a replacement
> is in the wings. In particular, I'm refering to the toolstack.git
> XCI tree.
> 
> 

Surely no replacement is going to be ready for the 3.5 release and I
think if anything is going to happen it will probably be a smooth
transition rather than an abrupt change.


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