On Mon, Oct 4, 2010 at 6:49 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: > On Sun, 3 Oct 2010 19:36:26 -0700, Keith Packard <keithp@xxxxxxxxxx> wrote: >> Cancel the output polling work proc before acquiring the struct mutex >> to avoid acquiring the work proc mutex with the struct mutex >> held. This avoids inverting the lock order seen when the work proc >> runs. > > I thought this was part of Daniel's reordering to avoid race conditions > between interrupts and module unload. Fortunately not. The lack of locking > during hotplug is worrisome should we ever do dynamic connectors. We don't do dynamic connectors now, so adding locking with no way of actually really testing it would just mean you'd probably have just as much pain when you do add dyanamic connectors. I also tried to think of a reason to support dynamic connectors, maybe displayport daisychaining, but none of the userspace drivers are ready for it. Dave. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
