<div dir="auto"><div>Hi Chris,</div><div dir="auto"><br></div><div dir="auto">Sorry I missed adding you in to list while sending the vblank refcount patch. Hope you saw that.<br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On 25-Oct-2017 9:55 PM, "PrasannaKumar Muralidharan" <<a href="mailto:prasannatsmkumar@gmail.com">prasannatsmkumar@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Warn when refcount > 0 in drm_vblank_cleanup.<br>
<br>
Signed-off-by: PrasannaKumar Muralidharan <<a href="mailto:prasannatsmkumar@gmail.com">prasannatsmkumar@gmail.com</a>><br>
---<br>
Changes in v3:<br>
* Dropped i915 patch that is used for testing this.<br>
<br>
No changes in v2.<br>
<br>
 drivers/gpu/drm/drm_vblank.c | 2 ++<br>
 1 file changed, 2 insertions(+)<br>
<br>
diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c<br>
index 70f2b95..3e61aeb 100644<br>
--- a/drivers/gpu/drm/drm_vblank.c<br>
+++ b/drivers/gpu/drm/drm_vblank.c<br>
@@ -405,6 +405,8 @@ void drm_vblank_cleanup(struct drm_device *dev)<br>
        for (pipe = 0; pipe < dev->num_crtcs; pipe++) {<br>
                struct drm_vblank_crtc *vblank = &dev->vblank[pipe];<br>
<br>
+               WARN_ON(atomic_read(&vblank-><wbr>refcount) > 0);<br>
+<br>
                WARN_ON(READ_ONCE(vblank-><wbr>enabled) &&<br>
                        drm_core_check_feature(dev, DRIVER_MODESET));<br>
<font color="#888888"><br>
--<br>
2.10.0<br>
<br>
</font></blockquote></div><br></div><div class="gmail_extra" dir="auto">As you anticipated Intel CI found that this patch triggers the non zero vblank refcount warning on module unload.</div><div class="gmail_extra" dir="auto"><br></div><div class="gmail_extra" dir="auto">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.</div><div class="gmail_extra" dir="auto"><br></div><div class="gmail_extra" dir="auto">Note: Replying from an Android device. Please forgive typos.</div><div class="gmail_extra" dir="auto"><br></div><div class="gmail_extra" dir="auto">Thanks and regards,</div><div class="gmail_extra" dir="auto">PrasannaKumar</div></div></div>