Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751732AbdFSNyl (ORCPT ); Mon, 19 Jun 2017 09:54:41 -0400 Received: from mail-pf0-f195.google.com ([209.85.192.195]:36382 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750878AbdFSNyj (ORCPT ); Mon, 19 Jun 2017 09:54:39 -0400 From: Hao Lee To: akpm@linux-foundation.org Cc: vbabka@suse.cz, mgorman@techsingularity.net, mhocko@suse.com, hannes@cmpxchg.org, iamjoonsoo.kim@lge.com, minchan@kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Hao Lee Subject: [PATCH] mm: remove a redundant condition in the for loop Date: Mon, 19 Jun 2017 21:54:18 +0800 Message-Id: <20170619135418.8580-1-haolee.swjtu@gmail.com> X-Mailer: git-send-email 2.9.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 935 Lines: 26 The variable current_order decreases from MAX_ORDER-1 to order, so the condition current_order <= MAX_ORDER-1 is always true. Signed-off-by: Hao Lee --- mm/page_alloc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 2302f25..9120c2b 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -2215,9 +2215,8 @@ __rmqueue_fallback(struct zone *zone, unsigned int order, int start_migratetype) bool can_steal; /* Find the largest possible block of pages in the other list */ - for (current_order = MAX_ORDER-1; - current_order >= order && current_order <= MAX_ORDER-1; - --current_order) { + for (current_order = MAX_ORDER-1; current_order >= order; + --current_order) { area = &(zone->free_area[current_order]); fallback_mt = find_suitable_fallback(area, current_order, start_migratetype, false, &can_steal); -- 2.9.3