Re: [Intel-gfx] [PATCH 2/4] drm/i915: cancel hotplug and dig_port work during suspend and unload

2014-08-12 Thread Ville Syrjälä
On Mon, Aug 11, 2014 at 09:54:15PM +0300, Imre Deak wrote: Make sure these work handlers don't run after we system suspend or unload the driver. Note that we don't cancel the handlers during runtime suspend. That could lead to a lockup, since we take a runtime PM ref from the handlers

Re: [Intel-gfx] [PATCH 2/4] drm/i915: cancel hotplug and dig_port work during suspend and unload

2014-08-12 Thread Imre Deak
On Tue, 2014-08-12 at 15:13 +0300, Ville Syrjälä wrote: On Mon, Aug 11, 2014 at 09:54:15PM +0300, Imre Deak wrote: Make sure these work handlers don't run after we system suspend or unload the driver. Note that we don't cancel the handlers during runtime suspend. That could lead to a

Re: [Intel-gfx] [PATCH 2/4] drm/i915: cancel hotplug and dig_port work during suspend and unload

2014-08-12 Thread Ville Syrjälä
On Tue, Aug 12, 2014 at 03:36:01PM +0300, Imre Deak wrote: On Tue, 2014-08-12 at 15:13 +0300, Ville Syrjälä wrote: On Mon, Aug 11, 2014 at 09:54:15PM +0300, Imre Deak wrote: Make sure these work handlers don't run after we system suspend or unload the driver. Note that we don't cancel the

Re: [Intel-gfx] [PATCH 2/4] drm/i915: cancel hotplug and dig_port work during suspend and unload

2014-08-12 Thread Imre Deak
On Tue, 2014-08-12 at 15:53 +0300, Ville Syrjälä wrote: On Tue, Aug 12, 2014 at 03:36:01PM +0300, Imre Deak wrote: On Tue, 2014-08-12 at 15:13 +0300, Ville Syrjälä wrote: On Mon, Aug 11, 2014 at 09:54:15PM +0300, Imre Deak wrote: Make sure these work handlers don't run after we system

[Intel-gfx] [PATCH 2/4] drm/i915: cancel hotplug and dig_port work during suspend and unload

2014-08-11 Thread Imre Deak
Make sure these work handlers don't run after we system suspend or unload the driver. Note that we don't cancel the handlers during runtime suspend. That could lead to a lockup, since we take a runtime PM ref from the handlers themselves. Fortunaltely canceling there is not needed since the RPM