[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds

2015-05-19 Thread Dave Airlie
On 19 May 2015 at 12:27, Michel Dänzer wrote: > On 19.05.2015 01:24, Alex Deucher wrote: >> >> @@ -96,10 +98,12 @@ static void radeon_dp_work_func(struct work_struct *work) >> struct drm_connector *connector; >> >> /* this should take a mutex */ >> + mutex_lock(&mode_config->mutex

[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds (v2)

2015-05-19 Thread Alex Deucher
Since we are messing with state in the worker. v2: drop the changes in the mst worker Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/radeon_irq_kms.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/radeon/radeon_irq_kms.c b/drivers/gp

[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds

2015-05-19 Thread Michel Dänzer
On 19.05.2015 01:24, Alex Deucher wrote: > > @@ -96,10 +98,12 @@ static void radeon_dp_work_func(struct work_struct *work) > struct drm_connector *connector; > > /* this should take a mutex */ > + mutex_lock(&mode_config->mutex); This comment can be removed? -- Earthling Mich

[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds

2015-05-19 Thread Daniel Vetter
On Mon, May 18, 2015 at 11:28:47PM -0400, Alex Deucher wrote: > On Mon, May 18, 2015 at 11:24 PM, Dave Airlie wrote: > > On 19 May 2015 at 12:27, Michel Dänzer wrote: > >> On 19.05.2015 01:24, Alex Deucher wrote: > >>> > >>> @@ -96,10 +98,12 @@ static void radeon_dp_work_func(struct work_struct

[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds

2015-05-18 Thread Alex Deucher
On Mon, May 18, 2015 at 11:24 PM, Dave Airlie wrote: > On 19 May 2015 at 12:27, Michel Dänzer wrote: >> On 19.05.2015 01:24, Alex Deucher wrote: >>> >>> @@ -96,10 +98,12 @@ static void radeon_dp_work_func(struct work_struct >>> *work) >>> struct drm_connector *connector; >>> >>> /*

[PATCH] drm/radeon: take the mode_config mutex when dealing with hpds

2015-05-18 Thread Alex Deucher
Since we are messing with state in the worker. Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/radeon_irq_kms.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/radeon/radeon_irq_kms.c b/drivers/gpu/drm/radeon/radeon_irq_kms.c index 71