Re: [REGRESSION] system hang on ILK/SNB/IVB

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

 



Hello Lukas,

The problem with the results in your link is that there is no HSW, ILK, IVB or 
SNB results. This might give the impression that everything is well.

Most damning is lack of HSW-gt2 and SNB-dellxps: those machines hang on to 
APC, and have run quite stably for every Patchwork run. The case isn't strong 
enough yet that series should fail if either of those won't run, but it might 
be so in future.

Tomi



On Wednesday 30 March 2016 23:35:08 Lukas Wunner wrote:
> Hi Gabriel,
> 
> On Wed, Mar 30, 2016 at 08:20:26PM +0300, Gabriel Feceoru wrote:
> > This commit causes a hang while running kms suspend tests
> > (kms_pipe_crc_basic@suspend-read-crc-pipe-*) on ILK/SNB/IVB, affecting CI.
> 
> This happened with v1 but not with v2 of the patch.
> Please check if somehow v1 ended up in your tree.
> 
> v2 passed CI fine, save for one warning not caused by the patch:
> https://patchwork.freedesktop.org/series/4068/
> 
> For comparison, this was v1:
> https://patchwork.freedesktop.org/patch/75840/
> 
> Best regards,
> 
> Lukas
> 
> > Probably the same problem with the one in v2, but on older HW.
> > 
> > 
> > commit a7442b93cf32c1e1ddb721a26cd1f92302e2a222
> > Author: Lukas Wunner <lukas@xxxxxxxxx>
> > Date:   Wed Mar 9 12:52:53 2016 +0100
> > 
> >     drm/i915: Fix races on fbdev
> >     
> >     The ->lastclose callback invokes intel_fbdev_restore_mode() and has
> >     been witnessed to run before intel_fbdev_initial_config_async()
> >     has finished.
> >     
> >     We might likewise receive hotplug events before we've had a chance to
> >     fully set up the fbdev.
> >     
> >     Fix by waiting for the asynchronous thread to finish.
> >     
> >     v2:
> >     An async_synchronize_full() was also added to
> >     intel_fbdev_set_suspend()
> >     in v1 which turned out to be entirely gratuitous. It caused a deadlock
> >     on suspend (discovered by CI, thanks to Damien Lespiau and Tomi
> >     Sarvela
> >     for CI support) and was unnecessary since a device is never suspended
> >     until its ->probe callback (and all asynchronous tasks it scheduled)
> >     have finished. See dpm_prepare(), which calls wait_for_device_probe(),
> >     which calls async_synchronize_full().
> >     
> >     Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93580
> >     Reported-by: Gustav Fägerlind <gustav.fagerlind@xxxxxxxxx>
> >     Reported-by: "Li, Weinan Z" <weinan.z.li@xxxxxxxxx>
> >     Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> >     Cc: stable@xxxxxxxxxxxxxxx
> >     Signed-off-by: Lukas Wunner <lukas@xxxxxxxxx>
> >     Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> >     Link:
> >     http://patchwork.freedesktop.org/patch/msgid/20160309115147.67B2B6E0D
> >     3@xxxxxxxxxxxxxxxxxxxx> 
> > Regards,
> > Gabriel

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://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