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


Groups > linux.kernel > #1524565

Re: [patch 1/2] mm, zone: track number of pages in free area by migratetype

From Vlastimil Babka <vbabka@suse.cz>
Newsgroups linux.kernel
Subject Re: [patch 1/2] mm, zone: track number of pages in free area by migratetype
Date 2016-11-17 18:30 +0100
Message-ID <sEyFB-3CT-83@gated-at.bofh.it> (permalink)
References <sEjQe-2jO-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 11/17/2016 02:32 AM, David Rientjes wrote:
> Each zone's free_area tracks the number of free pages for all free lists.
> This does not allow the number of free pages for a specific migratetype
> to be determined without iterating its free list.
> 
> An upcoming change will use this information to preclude doing async
> memory compaction when the number of MIGRATE_UNMOVABLE pageblocks is
> below a certain threshold.
> 
> The total number of free pages is still tracked, however, to not make
> zone_watermark_ok() more expensive.  Reading /proc/pagetypeinfo, however,
> is faster.

Yeah I've already seen a case with /proc/pagetypeinfo causing soft
lockups due to high number of iterations...

> This patch introduces no functional change and increases the amount of
> per-zone metadata at worst by 48 bytes per memory zone (when CONFIG_CMA
> and CONFIG_MEMORY_ISOLATION are enabled).

Isn't it 48 bytes per zone and order?

> Signed-off-by: David Rientjes <rientjes@google.com>

I'd be for this if there are no performance regressions. It affects hot
paths and increases cache footprint. I think at least some allocator
intensive microbenchmark should be used.

Vlastimil

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


Thread

[patch 1/2] mm, zone: track number of pages in free area by  migratetype David Rientjes <rientjes@google.com> - 2016-11-17 02:40 +0100
  Re: [patch 1/2] mm, zone: track number of pages in free area by  migratetype Vlastimil Babka <vbabka@suse.cz> - 2016-11-17 18:30 +0100
    Re: [patch 1/2] mm, zone: track number of pages in free area by  migratetype David Rientjes <rientjes@google.com> - 2016-11-17 23:20 +0100
      Re: [patch 1/2] mm, zone: track number of pages in free area by  migratetype Vlastimil Babka <vbabka@suse.cz> - 2016-11-18 22:00 +0100

csiph-web