Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

Show slab usage in memory meter #151

Closed
wants to merge 1 commit into from
Closed

Conversation

ypresto
Copy link

@ypresto ypresto commented Dec 19, 2014

Slab is a kind of kernel memory which includes reclaimable usage. So it should be shown in different color.

#67 (comment)

Why I think slab usage should be separated is:

  • It is just helpful if kernel, special mem usage is distinguished from user land one.
  • Large reclaimable slab usage caused problem in production server monitoring, finally. See newrelic discussion.
  • Latest code of free command from procps doesn't includes slab usage in "used" (actually shown as cache == page cache + slab). (commit)

Many thanks :)

@OmegaPhil
Copy link

@hishamhm
Copy link
Owner

@OmegaPhil, so an ideal PR would be a version of this using SReclaimable instead of Slab?

@OmegaPhil
Copy link

Basically.

There is one outstanding thing I'm not yet fully aware of - conky is also treating Shmem as allocated memory that isn't counted currently - do you know how this works? I did some digging on the conky ticket but its an open question for me. free doesn't specially deal with shmem but then it was completely wrong until recently too.

@hishamhm
Copy link
Owner

do you know how this works?

No, sorry. I haven't kept up with these things lately and each program seems to calculate things differently nowadays, so I'm keen on taking the community's advice. Thank you for looking it up!

@OmegaPhil
Copy link

OK. For reference, at this instant on this machine thats been up ~2d:

Shmem: 331352 kB
Slab: 3970204 kB
SReclaimable: 3788044 kB
SUnreclaim: 182160 kB

Not taking into account SReclaimable is a much bigger problem than the ~324MB Shmem here (although if it is wrong, 324MB wrong is still pretty bad) - I'd recommend making the SReclaimable change now, and if I get a response (or find something out) via conky I'll feed back to you later.

@hishamhm
Copy link
Owner

@OmegaPhil, ok! Will do. I posted a comment on shmem in the other issue.

@OmegaPhil
Copy link

Cheers, sorry I've been stuck on a problem in one of my projects for hours - should be back on this tomorrow.

@ypresto
Copy link
Author

ypresto commented Jul 26, 2016

It seems to be solved in #313

@ypresto ypresto closed this Jul 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants