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

memory: expose virtual totals / avail values on windows #1588

Closed
wants to merge 1 commit into from

Conversation

sni
Copy link

@sni sni commented Jan 26, 2024

There are 2 more memory values i'd like to use. They are there already, just not exposed by the module yet.

sni added a commit to ConSol-Monitoring/snclient that referenced this pull request Jan 26, 2024
pull request openened to expose those values:
shirou/gopsutil#1588
@shirou
Copy link
Owner

shirou commented Jan 27, 2024

Thank you for your contribute. However, we don't want to add platform specific fields anymore. We have a v4 plan which introduces platform specific structs. We will be able to add fields on it, but not developed yet.

There are 2 more memory values i'd like to use. They are there
already, just not exposed by the module.

- https://learn.microsoft.com/en-us/windows/win32/api/sysinfoapi/ns-sysinfoapi-memorystatusex

Signed-off-by: Sven Nierlein <[email protected]>
@sni sni force-pushed the win_virt_memory branch from 202353d to 7019c70 Compare January 29, 2024 12:42
shirou added a commit that referenced this pull request Feb 24, 2024
shirou added a commit that referenced this pull request Feb 24, 2024
shirou added a commit that referenced this pull request Feb 24, 2024
shirou added a commit that referenced this pull request Mar 10, 2024
shirou added a commit that referenced this pull request Apr 22, 2024
@shirou shirou closed this in 3caf443 Jun 1, 2024
@sni
Copy link
Author

sni commented Jun 1, 2024

that's great, thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants