At Thu, 19 Apr 2012 20:11:53 +0200, Takashi Iwai wrote: > > At Thu, 19 Apr 2012 13:55:04 -0400, > Adam Jackson wrote: > > > > On Thu, 2012-04-19 at 18:10 +0200, Takashi Iwai wrote: > > > > > This patch adds a flag to disable the hotplug during PM operation for > > > avoiding such a race. > > > > > > Cc: <stable at kernel.org> > > > Signed-off-by: Takashi Iwai <tiwai at suse.de> > > > > This seems simpler (untested): > > This looks promising. I'll ask a test with your patch. Tester reported a positive feedback. But he also experienced with a blank screen after a couple of S4 resumes. Now it's being checked whether it's a regression by the patch or not. The machine has a problem with S3 anyway, also showing a blank screen after S3 resume. So, the problem is likely irrelevant from your patch, but just to be sure. thanks, Takashi > > > thanks, > > Takashi > > > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > > index 1a7559b..1cd77a4 100644 > > --- a/drivers/gpu/drm/i915/i915_drv.c > > +++ b/drivers/gpu/drm/i915/i915_drv.c > > @@ -530,11 +530,12 @@ static int i915_drm_thaw(struct drm_device *dev) > > ironlake_init_pch_refclk(dev); > > > > drm_mode_config_reset(dev); > > - drm_irq_install(dev); > > > > /* Resume the modeset for every activated CRTC */ > > drm_helper_resume_force_mode(dev); > > > > + drm_irq_install(dev); > > + > > if (IS_IRONLAKE_M(dev)) > > ironlake_enable_rc6(dev); > > > > - ajax > _______________________________________________ > Intel-gfx mailing list > Intel-gfx at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx >
