Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1445615

Re: [PATCH 1/3] mm, vmscan: Remove redundant check in shrink_zones()

From Johannes Weiner <hannes@cmpxchg.org>
Newsgroups linux.kernel
Subject Re: [PATCH 1/3] mm, vmscan: Remove redundant check in shrink_zones()
Date 2016-07-18 18:20 +0200
Message-ID <rWjqW-8vS-21@gated-at.bofh.it> (permalink)
References <rWibv-7xJ-13@gated-at.bofh.it> <rWibv-7xJ-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Mon, Jul 18, 2016 at 03:50:24PM +0100, Mel Gorman wrote:
> As pointed out by Minchan Kim, shrink_zones() checks for populated
> zones in a zonelist but a zonelist can never contain unpopulated
> zones. While it's not related to the node-lru series, it can be
> cleaned up now.
> 
> Suggested-by: Minchan Kim <minchan@kernel.org>
> Signed-off-by: Mel Gorman <mgorman@techsingularity.net>

Ha, I didn't know that. But yeah, the zonelist building code excludes
unpopulated zones from the start. Neat.

Acked-by: Johannes Weiner <hannes@cmpxchg.org>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/3] mm, vmscan: Remove redundant check in shrink_zones() Mel Gorman <mgorman@techsingularity.net> - 2016-07-18 17:00 +0200
  Re: [PATCH 1/3] mm, vmscan: Remove redundant check in shrink_zones() Johannes Weiner <hannes@cmpxchg.org> - 2016-07-18 18:20 +0200
  Re: [PATCH 1/3] mm, vmscan: Remove redundant check in shrink_zones() Minchan Kim <minchan@kernel.org> - 2016-07-19 02:00 +0200

csiph-web