linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Hongbo Zhong <bocui107@gmail.com>
Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] mm: Remove the highmem zones' memmap in the highmem zone
Date: Mon, 1 Dec 2014 16:13:00 -0800	[thread overview]
Message-ID: <20141201161300.8d370b30b181e988065ec71d@linux-foundation.org> (raw)
In-Reply-To: <1417085194-17042-1-git-send-email-bocui107@gmail.com>

On Thu, 27 Nov 2014 18:46:34 +0800 Hongbo Zhong <bocui107@gmail.com> wrote:

> From: Zhong Hongbo <bocui107@gmail.com>
> 
> Since the commit 01cefaef40c4 ("mm: provide more accurate estimation
> of pages occupied by memmap") allocate the pages from lowmem for the
> highmem zones' memmap. So It is not need to reserver the memmap's for
> the highmem.

Looks right.

> A 2G DDR3 for the arm platform:
> On node 0 totalpages: 524288
> free_area_init_node: node 0, pgdat 80ccd380, node_mem_map 80d38000
>   DMA zone: 3568 pages used for memmap
>   DMA zone: 0 pages reserved
>   DMA zone: 456704 pages, LIFO batch:31
>   HighMem zone: 528 pages used for memmap
>   HighMem zone: 67584 pages, LIFO batch:15
> 
> On node 0 totalpages: 524288
> free_area_init_node: node 0, pgdat 80cd6f40, node_mem_map 80d42000
>   DMA zone: 3568 pages used for memmap
>   DMA zone: 0 pages reserved
>   DMA zone: 456704 pages, LIFO batch:31
>   HighMem zone: 67584 pages, LIFO batch:15

So nothing changed.  Maybe it would have if the machine had more
highmem.

I'm trying to work out what effect this patch actually has.  AFAICT it
provides more accurate values for zone->min_unmapped_pages and
zone->min_slab_pages on NUMA.  Anything else?


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

      reply	other threads:[~2014-12-02  0:13 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-27 10:46 Hongbo Zhong
2014-12-02  0:13 ` Andrew Morton [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141201161300.8d370b30b181e988065ec71d@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=bocui107@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox