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

Re: [Xen-devel] [PATCH] fix race condition between libvirtd event handling and libxl fd deregister



Ian Campbell writes ("Re: [PATCH] fix race condition between libvirtd event 
handling and libxl fd deregister"):
> I took Bamvor's most recent response to mean that a per-event lock was
> already in place in libvirt and inferred that this was the reason why
> the originally proposed one line fix worked for them. Perhaps I
> misunderstood?

Yes, I think that's what Bamvor meant but I don't think it's correct
that such a lock eliminates the race.  libvirt has to release that
lock before making the callback (to follow the libxl locking rules
which are necessary to avoid deadlock).

I'm not surprised that the original patch makes Bamvor's symptoms go
away.  Bamvor had one of the possible races (the fd-related one) but
not the other.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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