[Intel-gfx] [PATCH] drm/i915: Sync the hotplug work when device suspending

Liu, Chuansheng chuansheng.liu at intel.com
Mon Aug 5 18:00:20 PDT 2013



> -----Original Message-----
> From: Daniel Vetter [mailto:daniel.vetter at ffwll.ch] On Behalf Of Daniel Vetter
> Sent: Monday, August 05, 2013 1:42 PM
> To: Chris Wilson; Liu, Chuansheng; daniel.vetter at ffwll.ch; airlied at linux.ie;
> intel-gfx at lists.freedesktop.org; Li, Fei; dri-devel at lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] drm/i915: Sync the hotplug work when device
> suspending
> 
> On Sat, Jul 27, 2013 at 10:40:14AM +0100, Chris Wilson wrote:
> > On Sun, Jul 28, 2013 at 01:43:02AM +0800, Chuansheng Liu wrote:
> > >
> > > It is possible that during i915 device suspending with one pending
> > > hotplug work, one of cases is the device resume/suspend quickly.
> > >
> > > At this case, the hotplug work will be executed even after device
> > > is OFF, in Intel Android platform, it will cause system hang.
> >
> > See
> >
> >   1343070574-23917-1-git-send-email-chris at chris-wilson.co.uk
> >   http://lists.freedesktop.org/archives/intel-gfx/2012-July/019144.html
> >
> > which is in response to the bug raised here
> >
> >   s5hlilr4s9h.wl%tiwai at suse.de
> >   http://lists.freedesktop.org/archives/intel-gfx/2012-April/016738.html
> 
> Oops, I fail. And since your patch also has the required
> flush_scheduled_work (this will annoy Jesse since we flush system-wide,
> but alas) and is older ... can you please resubmit with References: for
> this new thread and the above two added and Chuanshen Lui cc'ed?
Yes, we really need this fix otherwise will cause some system hanging issue.
Thanks.

> 
> Thanks, Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the dri-devel mailing list