linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Tang Chen <tangchen@cn.fujitsu.com>
To: tglx@linutronix.de, mingo@elte.hu, hpa@zytor.com,
	akpm@linux-foundation.org, tj@kernel.org, trenn@suse.de,
	yinghai@kernel.org, jiang.liu@huawei.com, wency@cn.fujitsu.com,
	laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com,
	mgorman@suse.de, minchan@kernel.org, mina86@mina86.com,
	gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com,
	lwoodman@redhat.com, riel@redhat.com, jweiner@redhat.com,
	prarit@redhat.com
Cc: x86@kernel.org, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: [Part3 PATCH v2 0/4] Support hot-remove local pagetable pages.
Date: Thu, 13 Jun 2013 21:03:52 +0800	[thread overview]
Message-ID: <1371128636-9027-1-git-send-email-tangchen@cn.fujitsu.com> (raw)

The following patch-set from Yinghai allocates pagetables to local nodes.
v1: https://lkml.org/lkml/2013/3/7/642
v2: https://lkml.org/lkml/2013/3/10/47
v3: https://lkml.org/lkml/2013/4/4/639
v4: https://lkml.org/lkml/2013/4/11/829

Since pagetable pages are used by the kernel, they cannot be offlined.
As a result, they cannot be hot-remove.

This patch fix this problem with the following solution:

     1.   Introduce a new bootmem type LOCAL_NODE_DATAL, and register local
          pagetable pages as LOCAL_NODE_DATAL by setting page->lru.next to
          LOCAL_NODE_DATAL, just like we register SECTION_INFO pages.

     2.   Skip LOCAL_NODE_DATAL pages in offline/online procedures. When the
          whole memory block they reside in is offlined, the kernel can
          still access the pagetables.
          (This changes the semantics of offline/online a little bit.)

     3.   Do not free LOCAL_NODE_DATAL pages to buddy system because they
          were skipped when in offline/online procedures. The memory block
          they reside in could have been offlined.

Anyway, this problem should be fixed. Any better idea is welcome.

Change log:
v1 -> v2:
        patch2: As suggested by Wu Jianguo, define a macro to check if a page
                cantains local node data.
        patch4: As suggested by Wu Jianguo, prevent freeing LOCAL_NODE_DATA
                pages in free_pagetable() instead of in put_page_bootmem().

Tang Chen (4):
  bootmem, mem-hotplug: Register local pagetable pages with
    LOCAL_NODE_DATA when freeing bootmem.
  mem-hotplug: Skip LOCAL_NODE_DATA pages in memory offline procedure.
  mem-hotplug: Skip LOCAL_NODE_DATA pages in memory online procedure.
  mem-hotplug: Do not free LOCAL_NODE_DATA pages to buddy system in
    hot-remove procedure.

 arch/x86/mm/init_64.c          |   10 +++++++-
 include/linux/memblock.h       |   22 +++++++++++++++++
 include/linux/memory_hotplug.h |   20 +++++++++++++--
 mm/memblock.c                  |   52 ++++++++++++++++++++++++++++++++++++++++
 mm/memory_hotplug.c            |   31 +++++++++++++++++++++++
 mm/page_alloc.c                |   15 ++++++++++-
 mm/page_isolation.c            |    5 ++++
 7 files changed, 149 insertions(+), 6 deletions(-)

--
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:[~2013-06-13 13:27 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-13 13:03 Tang Chen [this message]
2013-06-13 13:03 ` [Part3 PATCH v2 1/4] bootmem, mem-hotplug: Register local pagetable pages with LOCAL_NODE_DATA when freeing bootmem Tang Chen
2013-06-13 14:16   ` Michal Nazarewicz
2013-06-14  5:37     ` Tang Chen
2013-06-13 13:03 ` [Part3 PATCH v2 2/4] mem-hotplug: Skip LOCAL_NODE_DATA pages in memory offline procedure Tang Chen
2013-06-13 17:17   ` Dave Hansen
2013-06-14  5:45     ` Tang Chen
2013-06-13 13:03 ` [Part3 PATCH v2 3/4] mem-hotplug: Skip LOCAL_NODE_DATA pages in memory online procedure Tang Chen
2013-06-13 13:03 ` [Part3 PATCH v2 4/4] mem-hotplug: Do not free LOCAL_NODE_DATA pages to buddy system in hot-remove procedure Tang Chen
2013-06-17  1:58 ` [Part3 PATCH v2 0/4] Support hot-remove local pagetable pages Jianguo Wu
2013-06-17  2:07   ` Tang Chen
2013-06-18 17:05 ` Vasilis Liaskovitis
2013-06-18 23:59   ` Toshi Kani
2013-06-19  2:58     ` Yasuaki Ishimatsu
2013-06-19 15:32       ` Toshi Kani
2013-06-19  7:29   ` Tang Chen
2013-06-19 10:00     ` Vasilis Liaskovitis
2013-06-19 15:18     ` Toshi Kani

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=1371128636-9027-1-git-send-email-tangchen@cn.fujitsu.com \
    --to=tangchen@cn.fujitsu.com \
    --cc=akpm@linux-foundation.org \
    --cc=gong.chen@linux.intel.com \
    --cc=hpa@zytor.com \
    --cc=isimatu.yasuaki@jp.fujitsu.com \
    --cc=jiang.liu@huawei.com \
    --cc=jweiner@redhat.com \
    --cc=laijs@cn.fujitsu.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=lwoodman@redhat.com \
    --cc=mgorman@suse.de \
    --cc=mina86@mina86.com \
    --cc=minchan@kernel.org \
    --cc=mingo@elte.hu \
    --cc=prarit@redhat.com \
    --cc=riel@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=tj@kernel.org \
    --cc=trenn@suse.de \
    --cc=vasilis.liaskovitis@profitbricks.com \
    --cc=wency@cn.fujitsu.com \
    --cc=x86@kernel.org \
    --cc=yinghai@kernel.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