Re: [PATCH] drm/i915: Sync the hotplug work when device suspending

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Daniel Vetter [mailto:daniel.vetter@xxxxxxxx] On Behalf Of Daniel Vetter
> Sent: Monday, August 05, 2013 1:42 PM
> To: Chris Wilson; Liu, Chuansheng; daniel.vetter@xxxxxxxx; airlied@xxxxxxxx;
> intel-gfx@xxxxxxxxxxxxxxxxxxxxx; Li, Fei; dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Subject: Re:  [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@xxxxxxxxxxxxxxxxxx
> >   http://lists.freedesktop.org/archives/intel-gfx/2012-July/019144.html
> >
> > which is in response to the bug raised here
> >
> >   s5hlilr4s9h.wl%tiwai@xxxxxxx
> >   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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux