Re: [Intel-gfx] [PATCH 08/13] drm/fb-helper: Stop using mode_config.mutex for internals

2017-07-04 Thread Daniel Vetter
On Tue, Jul 4, 2017 at 5:40 PM, Ville Syrjälä wrote: >> + mutex_lock(_helper->dev->mode_config.mutex); >> drm_enable_connectors(fb_helper, enabled); >> + if (drm_fb_helper_probe_connector_modes(fb_helper, width, height) == 0) > > This changes the order

Re: [Intel-gfx] [PATCH 08/13] drm/fb-helper: Stop using mode_config.mutex for internals

2017-07-04 Thread Ville Syrjälä
On Tue, Jul 04, 2017 at 05:18:28PM +0200, Daniel Vetter wrote: > Those are now all protected using fb_helper->lock. > > v2: We still need to hold mode_config.mutex right around calling > connector->fill_modes. > > v3: I forgot to hold mode_config.mutex while looking at > connector->status and

[Intel-gfx] [PATCH 08/13] drm/fb-helper: Stop using mode_config.mutex for internals

2017-07-04 Thread Daniel Vetter
Those are now all protected using fb_helper->lock. v2: We still need to hold mode_config.mutex right around calling connector->fill_modes. v3: I forgot to hold mode_config.mutex while looking at connector->status and the mode list. Also, we need to patch up the i915 ->initial_config callback to

[Intel-gfx] [PATCH 08/13] drm/fb-helper: Stop using mode_config.mutex for internals

2017-06-27 Thread Daniel Vetter
Those are now all protected using fb_helper->lock. v2: We still need to hold mode_config.mutex right around calling connector->fill_modes. v3: I forgot to hold mode_config.mutex while looking at connector->status and the mode list. Also, we need to patch up the i915 ->initial_config callback to