<div dir="ltr">Hi Rommer,<br>comments inlined.<br><br><div class="gmail_quote">On Mon, Oct 20, 2008 at 8:25 PM, Rommer <span dir="ltr"><<a href="mailto:rommer@active.by">rommer@active.by</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Sun, 19 Oct 2008 09:12:26 +0530<br>
"Basavanagowda Kanur" <<a href="mailto:gowda@zresearch.com">gowda@zresearch.com</a>> wrote:<br>
<br>
> Rommer,<br>
> It is not a memory leak.<br>
> pl_forget() is not called by protocol/server, pl_forget() is called<br>
> by __inode_destroy() after inode->ref becomes 0 (zero).<br>
><br>
<br>
</div>Thanks.<br>
One more question:<br>
server_open_resume() makes new fd by fd_create(), but server_open_cbk()<br>
in protocol/server-protocol.c does not fd_destroy(fd), if op_ret < 0.<br>
Where fd is destroing if open() failed?</blockquote><div><br>fd is unrefed (and possibly destroyed) during free_state () called in protocol_server_reply().<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<div><div></div><div class="Wj3C7c"><br>
<br>
_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@nongnu.org">Gluster-devel@nongnu.org</a><br>
<a href="http://lists.nongnu.org/mailman/listinfo/gluster-devel" target="_blank">http://lists.nongnu.org/mailman/listinfo/gluster-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Raghavendra G<br><br>
</div>