[systemd-devel] [PATCH] readahead-common: fix total memory size detection

Michal Schmidt mschmidt at redhat.com
Fri Jun 10 05:37:21 PDT 2011


sysinfo returns the total memory size in multiples of mem_unit bytes.
As long as the size in bytes fits into unsigned long, the kernel uses
mem_unit = 1, but this is not true on i386 with more than 4 GB RAM.

https://bugzilla.redhat.com/show_bug.cgi?id=712341
---

 src/readahead-common.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/readahead-common.c b/src/readahead-common.c
index 8a75b2e..f0d57b4 100644
--- a/src/readahead-common.c
+++ b/src/readahead-common.c
@@ -154,9 +154,8 @@ bool enough_ram(void) {
 
         assert_se(sysinfo(&si) >= 0);
 
-        return si.totalram > 127 * 1024*1024; /* Enable readahead only
-                                               * with at least 128MB
-                                               * memory */
+        /* Enable readahead only with at least 128MB memory */
+        return si.totalram > 127 * 1024*1024 / si.mem_unit;
 }
 
 int open_inotify(void) {



More information about the systemd-devel mailing list