Em Qua, 2016-09-14 às 09:40 +0100, chris@xxxxxxxxxxxxxxxxxx escreveu: > On Tue, Sep 13, 2016 at 03:21:48PM +0300, Mika Kuoppala wrote: > > > > Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> writes: > > > > > > "Zanoni, Paulo R" <paulo.r.zanoni@xxxxxxxxx> writes: > > > > > > > > > > > > > > +#if IS_ENABLED(CONFIG_LOCKDEP) > > > > > + GEM_BUG_ON(!!lockdep_is_held(&req->i915- > > > > > >drm.struct_mutex) > > > > > != > > > > > + !!(flags & I915_WAIT_LOCKED)); > > > > > +#endif > > > > > > > > I'm hitting this on my SKL. It usually happens right after the > > > > login, > > > > when I click the terminal icon on the toolbar in order to open > > > > it (on > > > > Cinnamon). When it doesn't happen at that time, I just open a > > > > browser > > > > and browse for like 30 seconds until it happens. > > > > > > > > I did manual reverts of this series up to this patch and > > > > obviously the > > > > problem goes away (no more GEM_BUG_ON to hit). I didn't try to > > > > do a > > > > real bisect to check if this is the first bad commit or if it's > > > > something later on the series. It could even be an older > > > > problem that > > > > just got exposed by the new BUG(). I'm available to test > > > > patches, just > > > > send them to me. > > > > > > No patches yet but there is a comment on > > > intel_prepare_plane_fb() that says that it must be called with > > > mutex > > > held. > > > > > > Quite likely the new GEM_BUG_ON catches this not being true. > > > > > > > As Chris pointed out in irc, its about the reverse. Waiting > > with mutex when you shouldn't. > > Fwiw, the fix is now on the list. Can you please tell us which patch it is? I see we still have this problem. Thanks, Paulo > -Chris > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
