Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix incorrect freemem report in docker containers #377

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

durigon
Copy link

@durigon durigon commented Jul 22, 2020

When using the Erlang agent to monitor OS, I found that the free memory collection in the docker container was incorrect.

We use 'MemAvailable' to get a free memory in the linux system when using the erlang agent.

MemAvailable %lu (since Linux 3.14)
An estimate of how much memory is available for starting new applications, without swapping.

In a container, MemAvailable should be the sum of MemFree, Cached and Buffers, but it is the same as MemFree.
After all, we can be mistaken for a memory leak because it does not include Cached/Buffers memory.

docker-container# egrep "MemFree|MemAvail|^Cached|Buffers" /proc/meminfo
MemFree: 93009720 kB
MemAvailable: 93009720 kB
Buffers: 0 kB
Cached: 7712588 kB

docker-container# uname -r
4.19.55

In a Host machine, MemAvailable is MemFree + Buffers + Cached.

host-machine# egrep "MemFree|MemAvail|^Cached|Buffers" /proc/meminfo
MemFree: 339513028 kB
MemAvailable: 381096504 kB
Buffers: 15072 kB
Cached: 37029112 kB

host-machine# uname -r
4.19.55

[How to improve]
Use the sum of MemFree, Cached, and Buffers as real available memory instead of MemAvailable
to improve the absence of MemAvailable field in old linux kernel version(prior to 3.14)
or to improve the incorrect MemAvailable in docker containers.

This PR solves issue #376

to improve the absence of MemAvailable field in old linux kernel version(prior to 3.14)
or to improve the incorrect MemAvailable in docker containers.
This PR solves issue processone#376
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant