From d0160ff8ee5f7a11f6008e560c9fb7137b069797 Mon Sep 17 00:00:00 2001 From: Viktor Rosendahl Date: Thu, 11 Sep 2008 19:20:36 +0300 Subject: [PATCH] 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 --- security/lowmem.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) 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) -- 2.41.1