| Message ID | 20180709083650.23549-6-daniel.vetter@ffwll.ch (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
LGTM:
Reviewed-by: Pavel Tatashin <pasha.tatashin@oracle.com>
diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h index 32699b2dc52a..1bd5f4c72c8b 100644 --- a/include/linux/mmzone.h +++ b/include/linux/mmzone.h @@ -940,9 +940,7 @@ extern struct zone *next_zone(struct zone *zone); for (zone = (first_online_pgdat())->node_zones; \ zone; \ zone = next_zone(zone)) \ - if (!populated_zone(zone)) \ - ; /* do nothing */ \ - else + for_each_if (populated_zone(zone)) static inline struct zone *zonelist_zone(struct zoneref *zoneref) {
Avoids the inverted condition of the open-coded version. Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Michal Hocko <mhocko@suse.com> Cc: Vlastimil Babka <vbabka@suse.cz> Cc: Mel Gorman <mgorman@techsingularity.net> Cc: David Rientjes <rientjes@google.com> Cc: Kemi Wang <kemi.wang@intel.com> Cc: Pavel Tatashin <pasha.tatashin@oracle.com> Cc: Petr Tesarik <ptesarik@suse.com> Cc: YASUAKI ISHIMATSU <yasu.isimatu@gmail.com> Cc: Andrey Ryabinin <aryabinin@virtuozzo.com> Cc: Nikolay Borisov <nborisov@suse.com> Cc: linux-mm@kvack.org --- include/linux/mmzone.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)