[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] libxenlight: fix multiple xenstore watches problem



On Thu, 3 Dec 2009, Vincent Hanquez wrote:
> On Wed, Dec 02, 2009 at 03:01:32PM +0000, Stefano Stabellini wrote:
> > Hi all,
> > this patch fixes the multiple xenstore watches problem in libxenlight
> > opening a new xenstore connection to set and read temporary watches on
> > the device state nodes.
> > This way they don't interfere with other long running watches.
> 
> This thing doesn't make sense; you're solving the wrong problem with cloning
> the context. you can open a cheap xenstore connection for temporary watches 
> and
> be done with it.
> 

You are right about that, but Andres found the bug and fixed it with his
"clone context to avoid GC corruption" patch.
We can limit the memory allocated for the new alloc_ptrs to something
much smaller than 256 in libxl_clone_context if you are worried about
memory allocation.


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.