[Windows] psutil.swap_memory() show swap instead of committed memory

Signed-off-by: David Knaack <davidkna@users.noreply.github.com>
giampaolo · Mar 14, 2021 · 1a66d57 · 1a66d57
1 parent c3e63b4
commit 1a66d57
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/CREDITS b/CREDITS
@@ -740,3 +740,7 @@ I: 1851
 N: guille
 W: https://github.com/guille
 I: 1913
+
+N: David Knaack
+W: https://github.com/davidkna
+I: 1921
diff --git a/HISTORY.rst b/HISTORY.rst
@@ -22,6 +22,7 @@ XXXX-XX-XX
   called after sprintf().  (patch by alxchk)
 - 1874_: [Solaris] swap output error due to incorrect range.
 - 1913_: [Linux] wait_procs seemingly ignoring timeout, TimeoutExpired thrown
+- 1921_: [Windows] psutil.swap_memory() shows committed memory instead of swap
 
 5.8.0
 =====

diff --git a/psutil/_pswindows.py b/psutil/_pswindows.py
@@ -241,8 +241,16 @@ def virtual_memory():
 def swap_memory():
     """Swap system memory as a (total, used, free, sin, sout) tuple."""
     mem = cext.virtual_mem()
-    total = mem[2]
-    free = mem[3]
+
+    total_phys = mem[0]
+    free_phys = mem[1]
+    total_system = mem[2]
+    free_system = mem[3]
+
+    # Despite the name PageFile refers to total system memory here
+    # thus physical memory values need to be substracted to get swap values
+    total = total_system - total_phys
+    free = min(total, free_system - free_phys)
     used = total - free
     percent = usage_percent(used, total, round_=1)
     return _common.sswap(total, used, free, percent, 0, 0)