On Thursday, June 30, 2011 15:42:28 Becky Ligon wrote: > http://www.beowulf-underground.org/pipermail/pvfs2-developers/attachments/2 > 0101020/068fd427/pint-mgmt-opid-cleanups-0001.obj > > Bart/Ben: > > Above is a link to the patch that was in the pvfs2-developers archive. It > has alot more stuff in it than the patch that you just sent me. In > addition, Sam made a comment about the pint-mgmt.c change indicating that > memory would not be released unless other changes were made. So, I'm > thinking that the changes I applied to OrangeFS included everything that > Sam intended and not just a quick solution. Please take a look at this > link and see if your code has these same changes. These changes may be > contributing to your memory problem, but we also think there still may be > something else as well. > > Becky
Becky, The referenced pint-mgmt-opid-cleanups-0001.obj patch above is the exact same commit that I've identified as causing the memory leak. If we revert that change from the OrangeFS code, the memory leak ceases. So, it appears that something isn't right with the changes Sam made. When you said that Sam made a comment about the pint-mgmt.c change, was he talking in reference to this pint-mgmt-opid-cleanups-0001.obj change or the one Bart sent you that was just an early return on error. Also, we are indeed testing with both the 2.8.2 branch and the latest OrangeFS code and both code bases have this change in them. -- Benjamin Severs _______________________________________________ Pvfs2-developers mailing list [email protected] http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
