[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20121019105546.9704.93446.stgit@srivatsabhat.in.ibm.com>
Date: Fri, 19 Oct 2012 16:25:47 +0530
From: "Srivatsa S. Bhat" <srivatsa.bhat@...ux.vnet.ibm.com>
To: linux-kernel@...r.kernel.org
Cc: linux-mm@...ck.org, kosaki.motohiro@...il.com,
akpm@...ux-foundation.org, hannes@...xchg.org,
srivatsa.bhat@...ux.vnet.ibm.com
Subject: [PATCH] mm: Simplify for_each_populated_zone()
Move the check for populated_zone() to the control statement of the
'for' loop and get rid of the odd looking if/else block.
Signed-off-by: Srivatsa S. Bhat <srivatsa.bhat@...ux.vnet.ibm.com>
---
include/linux/mmzone.h | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
index 50aaca8..5bdf02e 100644
--- a/include/linux/mmzone.h
+++ b/include/linux/mmzone.h
@@ -913,11 +913,8 @@ extern struct zone *next_zone(struct zone *zone);
#define for_each_populated_zone(zone) \
for (zone = (first_online_pgdat())->node_zones; \
- zone; \
- zone = next_zone(zone)) \
- if (!populated_zone(zone)) \
- ; /* do nothing */ \
- else
+ zone && populated_zone(zone); \
+ zone = next_zone(zone))
static inline struct zone *zonelist_zone(struct zoneref *zoneref)
{
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists