From 83f633a562891e9117b9916665add430d1296b7e Mon Sep 17 00:00:00 2001 From: dailz Date: Fri, 5 Jun 2026 13:34:01 +0800 Subject: [PATCH] fix(bench): make can_reset_vm_hwm side-effect-free with open probe (closes #34) --- crates/bench/src/metrics.rs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/crates/bench/src/metrics.rs b/crates/bench/src/metrics.rs index f51d9ca..29e7f80 100644 --- a/crates/bench/src/metrics.rs +++ b/crates/bench/src/metrics.rs @@ -80,9 +80,12 @@ impl MetricsCollector { }) } - /// Check if we can reset VmHWM (i.e., have root) + /// Check if we can reset VmHWM (i.e., can open `/proc/self/clear_refs` for writing) pub fn can_reset_vm_hwm() -> bool { - fs::write("/proc/self/clear_refs", "5").is_ok() + std::fs::OpenOptions::new() + .write(true) + .open("/proc/self/clear_refs") + .is_ok() } /// Get file inode number