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

swap_memory() tries /proc/stat if /proc/vmstat missing #727

Closed
wants to merge 4 commits into from

Conversation

dpavlenkov
Copy link

In joyent's SmartOS LX-brand zones, /proc/vmstat is missing but /proc/stat is available.

@dpavlenkov dpavlenkov closed this Jan 14, 2016
@dpavlenkov dpavlenkov reopened this Jan 14, 2016
@giampaolo
Copy link
Owner

Mmmm.. on Ubuntu 14.04 that's not available:

~/svn/psutil {master}$ cat /proc/stat | grep swap
~/svn/psutil {master}$

@dpavlenkov
Copy link
Author

/proc/stat -> swap is available in Joyent's LX zones. On real ubuntu /proc/vmstat is available. This doesn't affect real linux behavior, but it fixes LX zones.

@giampaolo
Copy link
Owner

I guess I'm completely ignorant about this SmartOS / LX zones. Can you elaborate on what this patch is trying to fix/address and on what operating systems?

@dpavlenkov
Copy link
Author

https://docs.joyent.com/public-cloud/instances/infrastructure/images
https://wiki.smartos.org/display/DOC/LX+Branded+Zones

Lx branded zones are linux containers on illumos. They should work like native linux, but that's work in progress. Some features are missing like /proc/vmstat . So, at first swap_memory barfed. You fixed that. Now this PR is about getting useful numbers for sin sout. They are available in /proc/stat.

@dpavlenkov
Copy link
Author

Let's hold this thought. I haven't seen non-zero values for sin/sout. If I can't get numbers, then this patch is moot.

@dpavlenkov
Copy link
Author

Closing PR because swap in/out are not available on this platform.

@dpavlenkov dpavlenkov closed this Jan 14, 2016
@dpavlenkov dpavlenkov deleted the issue722 branch January 14, 2016 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants