[PATCH 2/2] drm/fb-helper: improve drm_fb_helper_initial_config locking

2014-03-21 Thread Thierry Reding
On Thu, Mar 20, 2014 at 02:26:35PM +0100, Daniel Vetter wrote: [...] > Hence the right fix is to grab the mode_config mutex, but only that > and only right around the probe calls. > > It seems to be sufficient to shut up all the locking WARNINGs I see on > i915 and nouveau in

[PATCH 2/2] drm/fb-helper: improve drm_fb_helper_initial_config locking

2014-03-20 Thread Daniel Vetter
The locking in drm_fb_helper_initial_config is a bit troublesome for a few reasons: - We can't just wrap the entire function up into modeset locks since the fbdev registration might call down into fbcon code, which then through our ->set_par implementation needs to be able to grab all

[PATCH 2/2] drm/fb-helper: improve drm_fb_helper_initial_config locking

2014-03-20 Thread Daniel Vetter
The locking in drm_fb_helper_initial_config is a bit troublesome for a few reasons: - We can't just wrap the entire function up into modeset locks since the fbdev registration might call down into fbcon code, which then through our ->set_par implementation needs to be able to grab all