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

Re: [Xen-devel] [PATCH] fix for bug#515

On Fri, Mar 24, 2006 at 01:05:24PM +0000, Harry Butterworth wrote:

> # HG changeset patch
> # User harry@xxxxxxxxxxxxxxxxxxxxx
> # Node ID 78704d769fb2f758bc9c05b0911081375fae018d
> # Parent  8c21c8ea5fff320d1cc4abd18d11c648cdca322e
> Fix bug 515 by adding a global lock around the hotplug scripts in the
> non-udev hotplug case only.
> There are two possible root causes for 515, both of which are only
> problems in the non-udev hotplug case because udev uses udevsend which
> already implements the required serialisation.
> 1) Script concurrency.
> 2) Kernel reordering hotplug events.
> This changeset fixes (1) but not (2).  Since (1) is the problem that
> seems to be happening this is probably OK.
> A fix for (2) for the hotplug case might be to add extra serialisation
> using state changes in the store but this would impact all the drivers
> and the code and extra complexity would be redundant once everyone had
> moved to udev.  This doesn't seem worthwhile unless we actually start to
> see (2) happening.

Applied, thanks Harry.


Xen-devel mailing list



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