* PATCH:
@ 2000-07-12 16:24 Juan J. Quintela
0 siblings, 0 replies; only message in thread
From: Juan J. Quintela @ 2000-07-12 16:24 UTC (permalink / raw)
To: lkml, linux-mm, linus
Hi
somebody pointed out that mm->rss is defined as an unsigned
long, I think this patch is needed to do the desired effect.
I have not founded other invalid uses of mm->rss. Only that
in someplaces it is tested against (mm->rss <= 0) instead of
(mm->rss == 0). I think that the last checks are harmless.
Later, Juan.
diff -urN --exclude-from=/home/lfcia/quintela/work/kernel/exclude base/mm/memory.c working/mm/memory.c
--- base/mm/memory.c Mon May 15 21:00:33 2000
+++ working/mm/memory.c Wed Jul 12 03:55:11 2000
@@ -373,12 +373,12 @@
spin_unlock(&mm->page_table_lock);
/*
* Update rss for the mm_struct (not necessarily current->mm)
+ * Notice that rss is an unsigned long.
*/
- if (mm->rss > 0) {
+ if (mm->rss > freed)
mm->rss -= freed;
- if (mm->rss < 0)
- mm->rss = 0;
- }
+ else
+ mm->rss = 0;
}
--
In theory, practice and theory are the same, but in practice they
are different -- Larry McVoy
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux.eu.org/Linux-MM/
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2000-07-12 16:24 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-07-12 16:24 PATCH: Juan J. Quintela
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox