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

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

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