<div dir="ltr">For me, it is good :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 9, 2016 at 9:48 AM, Harry Wentland <span dir="ltr"><<a href="mailto:harry.wentland@amd.com" target="_blank">harry.wentland@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This looks good but we should probably do the same for all return paths when reference for port has been acquired.<br>
<br>
Please see attached patch.<br>
<br>
Thanks,<br>
Harry<div class="HOEnZb"><div class="h5"><br>
<br>
On 2016-02-01 11:08 AM, Insu Yun wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In drm_dp_mst_allocate_vcpi, it returns true in two paths,<br>
but in one path, there is no reference couting decrease.<br>
<br>
Signed-off-by: Insu Yun <<a href="mailto:wuninsu@gmail.com" target="_blank">wuninsu@gmail.com</a>><br>
---<br>
  drivers/gpu/drm/drm_dp_mst_topology.c | 1 +<br>
  1 file changed, 1 insertion(+)<br>
<br>
diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c<br>
index 6ed90a2..fe273b6 100644<br>
--- a/drivers/gpu/drm/drm_dp_mst_topology.c<br>
+++ b/drivers/gpu/drm/drm_dp_mst_topology.c<br>
@@ -2446,6 +2446,7 @@ bool drm_dp_mst_allocate_vcpi(struct drm_dp_mst_topology_mgr *mgr, struct drm_dp<br>
                DRM_DEBUG_KMS("payload: vcpi %d already allocated for pbn %d - requested pbn %d\n", port->vcpi.vcpi, port->vcpi.pbn, pbn);<br>
                if (pbn == port->vcpi.pbn) {<br>
                        *slots = port->vcpi.num_slots;<br>
+                       drm_dp_put_port(port);<br>
                        return true;<br>
                }<br>
        }<br>
</blockquote>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div>Regards</div>
<div>Insu Yun</div></div>
</div>