rfjakob / earlyoom

earlyoom - Early OOM Daemon for Linux
MIT License
2.96k stars 157 forks source link

UserMemTotal is not documented, `-M` argument seems to still be used as percentage of *all* total which is confusing #303

Closed mars4science closed 7 months ago

mars4science commented 10 months ago

I've tried to use the app from master branch. Got confused from output of systemctl status earlyoom.service showing avail as % of user mem total, found commit https://github.com/rfjakob/earlyoom/commit/459d76296d3d0a0b59ee1e2e48ad2271429de916. I get the reason, however, IMO in such case -M argument is better be used as part of user memory only.

Also if UserMemTotal is to stay, please somebody document the change in README and man page, explaining AnonPages with no less clarity than now available of free is explained. Thanks.

Edit: created pull request https://github.com/rfjakob/earlyoom/pull/304 to try to cover MANPAGE and code change.

rfjakob commented 9 months ago

Agree, docs need to improve.

The problem with https://github.com/rfjakob/earlyoom/pull/304 is that what earlyoom does depends on how full /tmp was when it was started. This is ugly.