1 files changed, 14 insertions, 6 deletions

diff --git a/src/memory.c b/src/memory.c
index ea056bc..d8de483 100644
--- a/src/memory.c
+++ b/src/memory.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012-2025 Robin Haberkorn
+ * Copyright (C) 2012-2026 Robin Haberkorn
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -251,13 +251,19 @@
  * It is of course possible to query the program's RSS via OS APIs.
  * This has long been avoided because it is naturally platform-dependant and
  * some of the APIs have proven to be too slow for frequent polling.
+ * Also, this will only reliably work if malloc_trim(0) does what it's
+ * supposed to do.
  *
  * - Windows has GetProcessMemoryInfo() which is quite slow.
  *   When polled on a separate thread, the slow down is very acceptable.
+ * - POSIX has getrusage().
+ *   __Its performance on different OS is still untested!__
+ *   It reports in different units on different systems, see
+ *   mimalloc/src/prim/unix/prim.c.
+ *   The maxrss field does not shrink even with a working malloc_trim().
  * - OS X has task_info().
  *   __Its performance is still untested!__
  * - FreeBSD has sysctl().
- *   __Its performance is still untested!__
  * - Linux has no APIs but /proc/self/statm.
  *   Reading it is naturally very slow, but at least of constant time.
  *   When polled on a separate thread, the slow down is very acceptable.
@@ -298,6 +304,10 @@ static guint teco_memory_usage = 0;
  */
 #ifdef REPLACE_MALLOC
 
+#ifndef G_ATOMIC_LOCK_FREE
+#warning "malloc() replacement will be very slow!"
+#endif
+
 void * __attribute__((used))
 malloc(size_t size)
 {
@@ -471,10 +481,8 @@ teco_memory_get_usage(void)
 /*
  * Practically only for FreeBSD.
  *
- * The malloc replacement via dlmalloc also works on FreeBSD,
- * but this implementation has been benchmarked to be up to 4 times faster
- * (but only if we poll in a separate thread).
- * On the downside, this will of course be less precise.
+ * Since FreeBSD supports dlmalloc(), which is generally faster than
+ * jemalloc with a poll thread, this is not usually required.
  */
 static gsize
 teco_memory_get_usage(void)