Hi Chris, Sorry I missed adding you in to list while sending the vblank refcount patch. Hope you saw that.
On 25-Oct-2017 9:55 PM, "PrasannaKumar Muralidharan" < prasannatsmku...@gmail.com> wrote: Warn when refcount > 0 in drm_vblank_cleanup. Signed-off-by: PrasannaKumar Muralidharan <prasannatsmku...@gmail.com> --- Changes in v3: * Dropped i915 patch that is used for testing this. No changes in v2. drivers/gpu/drm/drm_vblank.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c index 70f2b95..3e61aeb 100644 --- a/drivers/gpu/drm/drm_vblank.c +++ b/drivers/gpu/drm/drm_vblank.c @@ -405,6 +405,8 @@ void drm_vblank_cleanup(struct drm_device *dev) for (pipe = 0; pipe < dev->num_crtcs; pipe++) { struct drm_vblank_crtc *vblank = &dev->vblank[pipe]; + WARN_ON(atomic_read(&vblank->refcount) > 0); + WARN_ON(READ_ONCE(vblank->enabled) && drm_core_check_feature(dev, DRIVER_MODESET)); -- 2.10.0 As you anticipated Intel CI found that this patch triggers the non zero vblank refcount warning on module unload. Please advice on what to do next. Should I don't think I have to do something for this patch. If that's not the case please let me know what to do from my side for this patch. Note: Replying from an Android device. Please forgive typos. Thanks and regards, PrasannaKumar
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel