Re: drm_atomic_helper_resume() locking changes in v4.12

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

 



On Mon, May 29, 2017 at 05:29:55PM +0300, Jyri Sarha wrote:
> Hi,
> I have "WARN_ON(!drm_modeset_is_locked(&crtc->mutex))" in tilcdc crtc
> enable and disable callbacks. Now those warnings are firing when
> resuming from suspend and I see that "drm_modeset_lock_all(dev)" has
> been removed from "drm_atomic_helper_resume()".
> 
> I guess I should remove those warnings from tilcdc since no other driver
> has such warnings in place, but it still seems odd to me that the crtc
> enable or disable maybe called without the crtc modeset lock.
> 
> Is there something wrong with drm_atomic_helper_resume() locking or am
> just confused about how the locking in drm works? (The later is probably
> true in any case)

Oversight in the conversion, i.e. thanks for reporting this bug I'll type
the patch to fix it asap :-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux