[Bugs] [Bug 1471137] New: mem-pool: mem_pool_fini() doesn' t release entire memory allocated

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1471137

            Bug ID: 1471137
           Summary: mem-pool: mem_pool_fini() doesn't release entire
                    memory allocated
           Product: GlusterFS
           Version: 3.11
         Component: libgfapi
          Keywords: Triaged
          Severity: high
          Assignee: ndevos at redhat.com
          Reporter: ndevos at redhat.com
        QA Contact: bugs at gluster.org
                CC: bugs at gluster.org
        Depends On: 1470170



+++ This bug was initially created as a clone of Bug #1470170 +++

Description of problem:

At the moment, all the work which mem_pool_fini() does is to exit and cleanup
the sweeper thread. That doesn't ensure that all the memory allocated is
cleaned up as some of those allocations may still be in hot list or if in cold
list, sweeper thread may not have got chance to run through them.
hence we need to iterate though all those per-thread mem-pools and clean them
up as part of fini()

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Niels de Vos on 2017-07-12 20:02:48 CEST ---

I'm looking into this with help from
https://github.com/gluster/gluster-debug-tools/tree/master/gfapi-load-volfile

--- Additional comment from Worker Ant on 2017-07-13 13:58:57 CEST ---

REVIEW: https://review.gluster.org/17768 (mem-pool: free objects from pools on
mem_pools_fini()) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)

--- Additional comment from Worker Ant on 2017-07-14 11:31:22 CEST ---

REVIEW: https://review.gluster.org/17768 (mem-pool: free objects from pools on
mem_pools_fini()) posted (#2) for review on master by Niels de Vos
(ndevos at redhat.com)

--- Additional comment from Worker Ant on 2017-07-14 15:37:50 CEST ---

REVIEW: https://review.gluster.org/17768 (mem-pool: free objects from pools on
mem_pools_fini()) posted (#3) for review on master by Niels de Vos
(ndevos at redhat.com)

--- Additional comment from Worker Ant on 2017-07-14 15:37:54 CEST ---

REVIEW: https://review.gluster.org/17778 (mem-pool: remove references to unused
ctx->mempool_list) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1470170
[Bug 1470170] mem-pool: mem_pool_fini() doesn't release entire memory
allocated
-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.