]> granicus.if.org Git - procps-ng/commit
top: finally circumvent that minor libnuma memory leak
authorJim Warner <james.warner@comcast.net>
Mon, 12 Sep 2016 21:16:16 +0000 (16:16 -0500)
committerCraig Small <csmall@enc.com.au>
Sun, 18 Sep 2016 10:39:12 +0000 (20:39 +1000)
commit18b1a887b781e94391909fc7a8f7021309a8ebec
tree41ec8a076e365e927bd68f5fea6bb9cff9a36e80
parent24fd2605c51fccc375ab0287cec33aa767f06718
top: finally circumvent that minor libnuma memory leak

Still unhappy with a minor memory leak associated with
libnuma, I experimented with omitting the dlclose that
was issued at module's end. For some reason which will
remain a mystery, the valgrind leak then went bye-bye.

So this patch just omits one use of dlclose and relies
on whatever kernel magic is at work to free the memory
when each process ends. We kept, however, the original
code (now commented-out) to serve as a future caution.

There remains one potential (but unlikely) dlclose use
near the original dlopen. But there will be no leak as
that 'numa_node_of_cpu' will not yet have been called.
This seems to be the culprit that triggers such leaks.

None of this libnuma shit would likely have come close
to hitting our fan had the numa developers provided us
with 'new' and 'unref' functions like our newlib does.

[ this commit parallels a patch in our newlib branch ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
NEWS
top/top.c