Description of problem: We have moved to newer implementation of mem-pools. But statedump doesn't print the information about them. Github issue [1] captures upstream effort on this. But preliminary reading shows the work still has some parts to be completed. So, the effort includes: * backporting https://review.gluster.org/18076 * reviewing whether the information printed by above patch is complete from the perspective of debugging memory consumption issues. [1] https://github.com/gluster/glusterfs/issues/307 Version-Release number of selected component (if applicable): 3.4.0 How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
Patch is already available upstream.
(In reply to Amar Tumballi from comment #4) > Patch is already available upstream. If you are speaking about https://review.gluster.org/18076, we still need to evaluate whether it covers all aspects of newer memory pools. So, the work may not be complete
https://review.gluster.org/#/c/glusterfs/+/18075/ is needed too.
commit msg of one of patches [1] say - This only captures the information from 'struct mem_pool', and not from the global 'struct mem_pool_shared' or the pool_sweeper thread. The current details help with detecting memory leaks. so we need to find out what is left out if we don't dump those fields and preferably add those two fields if important. [1] https://review.gluster.org/18076
Based on the points discussed in #comment28,#comment30 and #comment31 , moving this bug to verified.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:0263