process-metrics 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ca29ce7e69894dd4f9f766caa5ee6055c1a25c06d1b764960ffe72c15dfb1958
-  data.tar.gz: 4723276718e25bc09053ba004357f33a720d3da342b384277f07b5a3eebe898e
+  metadata.gz: 43c70d72392b299b8219b1bb1d5762de8d4ee116568ebd4ada928ba946d3ae96
+  data.tar.gz: d0c5a4ab4a3fd2935e95667fff758c20c87f14165776dec5b908f544cc24a590
 SHA512:
-  metadata.gz: b6548743e1120c91c17cf5aa6255bb14991c68ac7b28861325ea3376061e7a3da18f4d1bfc7762f83eb48e46f2e1ce792e1550e3fc16cae13254f5c8189e9650
-  data.tar.gz: 53cbcbabaef2f8d414e877eeb5b3d1dbc286f87cbb01ce4a6afee588527440e94c9ea144af4e13d5bb7f085971b2da91f373684bbc8076980cafe02c6aa8f7f6
+  metadata.gz: 91961040cacd7a310a28ab42b76ff4f73386bdbd77a36138b7e5e4a3704fed2e6184ad493629ad501e45473c0013bbe675a4c159d74efc723a1cdb138b1a1df7
+  data.tar.gz: 9de09390fbc96a7dc208914fd18aeed8d0380f49fb3bc4ab8bf33ec1e15dba671d48921eb71460a33aba779d409cd4180c7f811f42ff736ddd4bf48d2971958e

data/lib/process/metrics/memory/darwin.rb

@@ -51,18 +51,18 @@ module Process
 			
 			# Capture memory usage for the given process IDs.
 			def self.capture(pid, count: 1, **options)
-				usage = Memory.zero
-				
 				IO.popen(["vmmap", pid.to_s], "r") do |io|
+					usage = Memory.zero
+					
 					io.each_line do |line|
 						if match = LINE.match(line)
+							usage.map_count += 1
+							
 							virtual_size = parse_size(match[:virtual_size])
 							resident_size = parse_size(match[:resident_size])
 							dirty_size = parse_size(match[:dirty_size])
 							swap_size = parse_size(match[:swap_size])
 							
-							# Update counts
-							usage.map_count += 1
 							usage.resident_size += resident_size
 							usage.swap_size += swap_size
 							
@@ -84,13 +84,21 @@ module Process
 							end
 						end
 					end
+					
+					if usage.map_count.zero?
+						# vmap might not fail, but also might not return any data.
+						return nil
+					end
+					
+					# Darwin does not expose proportional memory usage, so we guess based on the number of processes. Yes, this is a terrible hack, but it's the most reasonable thing to do given the constraints:
+					usage.proportional_size = usage.resident_size / count
+					usage.proportional_swap_size = usage.swap_size / count
+					
+					return usage
 				end
-				
-				# Darwin does not expose proportional memory usage, so we guess based on the number of processes. Yes, this is a terrible hack, but it's the most reasonable thing to do given the constraints:
-				usage.proportional_size = usage.resident_size / count
-				usage.proportional_swap_size = usage.swap_size / count
-				
-				return usage
+			rescue Errno::ESRCH
+				# Process doesn't exist.
+				return nil
 			end
 		end
 		

data/lib/process/metrics/memory/linux.rb

@@ -11,20 +11,16 @@ module Process
 			# @parameter pid [Integer] The process ID.
 			# @parameter usage [Memory] The Memory instance to populate with fault counters.
 			def self.capture_faults(pid, usage)
-				begin
-					stat = File.read("/proc/#{pid}/stat")
-					# The comm field can contain spaces and parentheses; find the closing ')':
-					rparen_index = stat.rindex(")")
-					return unless rparen_index
-					fields = stat[(rparen_index+2)..-1].split(/\s+/)
-					# proc(5): field 10=minflt, 12=majflt; our fields array is 0-indexed from field 3.
-					usage.minor_faults = fields[10-3].to_i
-					usage.major_faults = fields[12-3].to_i
-				rescue Errno::ENOENT, Errno::EACCES
-					# The process may have exited or permissions are insufficient; ignore.
-				rescue => error
-					# Be robust to unexpected formats; ignore errors silently.
-				end
+				stat = File.read("/proc/#{pid}/stat")
+				# The comm field can contain spaces and parentheses; find the closing ')':
+				rparen_index = stat.rindex(")")
+				return unless rparen_index
+				fields = stat[(rparen_index+2)..-1].split(/\s+/)
+				# proc(5): field 10=minflt, 12=majflt; our fields array is 0-indexed from field 3.
+				usage.minor_faults = fields[10-3].to_i
+				usage.major_faults = fields[12-3].to_i
+			rescue
+				# Ignore.
 			end
 			
 			# @returns [Numeric] Total memory size in kilobytes.
@@ -58,10 +54,10 @@ module Process
 				
 				# Capture memory usage for the given process IDs.
 				def self.capture(pid, **options)
-					usage = Memory.zero
-					
-					begin
-						File.foreach("/proc/#{pid}/smaps_rollup") do |line|
+					File.open("/proc/#{pid}/smaps_rollup") do |file|
+						usage = Memory.zero
+						
+						file.each_line do |line|
 							if /(?<name>.*?):\s+(?<value>\d+) kB/ =~ line
 								if key = SMAP[name]
 									usage[key] += value.to_i
@@ -72,11 +68,12 @@ module Process
 						usage.map_count += File.readlines("/proc/#{pid}/maps").size
 						# Also capture fault counters:
 						self.capture_faults(pid, usage)
-					rescue Errno::ENOENT => error
-						# Ignore.
+						
+						return usage
 					end
-					
-					return usage
+				rescue Errno::ENOENT, Errno::ESRCH
+					# Process doesn't exist.
+					return nil
 				end
 			elsif File.readable?("/proc/self/smaps")
 				# Whether the memory usage can be captured on this system.
@@ -86,10 +83,10 @@ module Process
 				
 				# Capture memory usage for the given process IDs.
 				def self.capture(pid, **options)
-					usage = Memory.zero
-					
-					begin
-						File.foreach("/proc/#{pid}/smaps") do |line|
+					File.open("/proc/#{pid}/smaps") do |file|
+						usage = Memory.zero
+						
+						file.each_line do |line|
 							# The format of this is fixed according to:
 							# https://github.com/torvalds/linux/blob/351c8a09b00b5c51c8f58b016fffe51f87e2d820/fs/proc/task_mmu.c#L804-L814
 							if /(?<name>.*?):\s+(?<value>\d+) kB/ =~ line
@@ -102,13 +99,15 @@ module Process
 								usage.map_count += 1
 							end
 						end
+						
 						# Also capture fault counters:
 						self.capture_faults(pid, usage)
-					rescue Errno::ENOENT => error
-						# Ignore.
+						
+						return usage
 					end
-					
-					return usage
+				rescue Errno::ENOENT, Errno::ESRCH
+					# Process doesn't exist.
+					return nil
 				end
 			else
 				def self.supported?

data/lib/process/metrics/version.rb

@@ -7,6 +7,6 @@
 module Process
 	# @namespace
 	module Metrics
-		VERSION = "0.6.0"
+		VERSION = "0.7.0"
 	end
 end

data/readme.md

@@ -16,6 +16,14 @@ Please see the [project documentation](https://socketry.github.io/process-metric
 
 Please see the [project releases](https://socketry.github.io/process-metrics/releases/index) for all releases.
 
+### v0.7.0
+
+  - Be more proactive about returning nil if memory capture failed.
+
+### v0.6.1
+
+  - Handle `Errno::ESRCH: No such process @ io_fillbuf - fd:xxx /proc/xxx/smaps_rollup` by ignoring it.
+
 ### v0.6.0
 
   - Add support for major and minor page faults on Linux: `Process::Metrics::Memory#major_faults` and `#minor_faults`. Unfortunately these metrics are not available on Darwin (macOS).
@@ -65,14 +73,6 @@ Please see the [project releases](https://socketry.github.io/process-metrics/rel
   - Fixed process metrics to exclude the `ps` command itself from measurements.
   - Fixed documentation formatting issues.
 
-### v0.1.0
-
-  - Initial release with support for process and memory metrics on Linux and Darwin (macOS).
-  - Support for selecting processes based on PID or PPID (process group).
-  - Implementation of memory statistics collection using `/proc` filesystem on Linux.
-  - Better handling of process hierarchies.
-  - Support for older Ruby versions.
-
 ## Contributing
 
 We welcome contributions to this project.

data/releases.md

@@ -1,5 +1,13 @@
 # Releases
 
+## v0.7.0
+
+  - Be more proactive about returning nil if memory capture failed.
+
+## v0.6.1
+
+  - Handle `Errno::ESRCH: No such process @ io_fillbuf - fd:xxx /proc/xxx/smaps_rollup` by ignoring it.
+
 ## v0.6.0
 
   - Add support for major and minor page faults on Linux: `Process::Metrics::Memory#major_faults` and `#minor_faults`. Unfortunately these metrics are not available on Darwin (macOS).

data.tar.gz.sig

@@ -1,2 +1 @@
-\��1,��5��%��A�F��&|՝\,�J�:����!wA��x��^0`o>��iW�\D|
-�!���i�(�y���B��kJ>*�V��%��Jcy�`�/Uj=���?^��G�.��S���C]�j���ȃ���R%��7�|�VQU�S�p+�/����J+O��&5�4ƺ
+dj\�#c����6B�X;n�k��M���E��Nk[J���t�lN,ԕ��jI��t��s�L�0���P?

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: process-metrics
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Samuel Williams
@@ -122,7 +122,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.7.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Provide detailed OS-specific process metrics.
 test_files: []