From: Viktor Rosendahl Date: Thu, 11 Sep 2008 16:20:36 +0000 (+0300) Subject: lowmem: remove an unecessary local variable X-Git-Tag: v2.6.27-omap1~203 X-Git-Url: http://pilppa.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=d0160ff8ee5f7a11f6008e560c9fb7137b069797;p=linux-2.6-omap-h63xx.git lowmem: remove an unecessary local variable This "changed" variable is kind of deprecated. We used to have a spinlock [which was later deemed unecessary] around the first if clause in the *_watermark_state() functions; the variable was needed in order to move the operation in the second if clause out of the lock section. Signed-off-by: Viktor Rosendahl Signed-off-by: Tony Lindgren --- diff --git a/security/lowmem.c b/security/lowmem.c index 2dc0c4a78e0..183c57fb280 100644 --- a/security/lowmem.c +++ b/security/lowmem.c @@ -136,28 +136,18 @@ KERNEL_ATTR_RO(high_watermark); static void low_watermark_state(int new_state) { - int changed = 0; - if (low_watermark_reached != new_state) { low_watermark_reached = new_state; - changed = 1; - } - - if (changed) sysfs_notify(&kernel_subsys.kset.kobj, NULL, "low_watermark"); + } } static void high_watermark_state(int new_state) { - int changed = 0; - if (high_watermark_reached != new_state) { high_watermark_reached = new_state; - changed = 1; - } - - if (changed) sysfs_notify(&kernel_subsys.kset.kobj, NULL, "high_watermark"); + } } static int low_vm_enough_memory(long pages)