From: Andi Kleen <ak@suse.de>
To: Yasunori Goto <y-goto@jp.fujitsu.com>
Cc: Ingo Molnar <mingo@elte.hu>,
tglx@linutronix.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [5/8] Fix logic error in 64bit memory hotadd
Date: Tue, 12 Feb 2008 12:20:51 +0100 [thread overview]
Message-ID: <200802121220.51755.ak@suse.de> (raw)
In-Reply-To: <20080212190310.F9E8.Y-GOTO@jp.fujitsu.com>
On Tuesday 12 February 2008 11:35:22 Yasunori Goto wrote:
> Hi Ingo-san.
>
> > > Does anyone even
> > > use memory hotplug currently?
> >
> > I don't know.
>
> IBM's powerpc box can memory hot-add/remove by dynamic partitioning.
> And our fujitsu server has memory hot-add feature (Ia-64).
> So, they are concrete user of memory hotplug.
>
> In x86, E8500 chipset has the feature of memory-hotplug.
> (I searched a data-sheet from intel site.)
> http://download.intel.com/design/chipsets/e8500/datashts/30674501.pdf
> (6.3.8 IMI Hot-Plug)
>
> So, it depends on how many server uses it, I think.
With the logic pud error i found and that seems currently will stay
for the forseeable future in the tree I don't think x86-64 64bit hotadds for more
than 1GB have ever worked with the sparsemem method.
Older trees (before 2.6.24) had also a different method that did
preallocate everything based on the SRAT. That one should have worked
for this case.
-Andi
next prev parent reply other threads:[~2008-02-12 11:21 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-11 9:34 [PATCH] [0/8] Various kernel mapping bug fixes Andi Kleen
2008-02-11 9:34 ` [PATCH] [1/8] CPA: Fix gbpages support in try_preserve_lage_page Andi Kleen
2008-02-11 9:45 ` Thomas Gleixner
2008-02-11 10:12 ` Ingo Molnar
2008-02-11 11:01 ` Andi Kleen
2008-02-11 9:34 ` [PATCH] [2/8] CPA: Flush the caches when setting pages not present Andi Kleen
2008-02-11 11:00 ` Ingo Molnar
2008-02-11 12:26 ` Andi Kleen
2008-02-11 9:34 ` [PATCH] [3/8] CPA: Test the correct mapping alias on x86-64 Andi Kleen
2008-02-11 11:49 ` Ingo Molnar
2008-02-11 9:34 ` [PATCH] [4/8] CPA: Fix set_memory_x for ioremap Andi Kleen
2008-02-11 12:27 ` Ingo Molnar
2008-02-11 12:45 ` Andi Kleen
2008-02-11 9:34 ` [PATCH] [5/8] Fix logic error in 64bit memory hotadd Andi Kleen
2008-02-11 12:46 ` Ingo Molnar
2008-02-11 13:05 ` Andi Kleen
2008-02-11 13:33 ` Ingo Molnar
2008-02-11 13:44 ` Andi Kleen
2008-02-12 10:35 ` Yasunori Goto
2008-02-12 11:20 ` Andi Kleen [this message]
2008-02-11 9:34 ` [PATCH] [6/8] Account overlapped mappings in end_pfn_map Andi Kleen
2008-02-11 13:08 ` Ingo Molnar
2008-02-11 13:27 ` Andi Kleen
2008-02-11 13:55 ` Ingo Molnar
2008-02-11 14:16 ` Peter Zijlstra
2008-02-11 14:24 ` Andi Kleen
2008-02-11 14:41 ` Sam Ravnborg
2008-02-11 15:12 ` Arjan van de Ven
2008-02-11 9:34 ` [PATCH] [7/8] Implement true end_pfn_mapped for 32bit Andi Kleen
2008-02-12 19:39 ` Thomas Gleixner
2008-02-12 19:49 ` Andi Kleen
2008-02-12 20:25 ` Thomas Gleixner
2008-02-11 9:34 ` [PATCH] [8/8] RFC: Fix some EFI problems Andi Kleen
2008-02-12 20:04 ` Thomas Gleixner
2008-02-12 20:23 ` Andi Kleen
2008-02-12 20:48 ` Thomas Gleixner
2008-02-13 11:05 ` Andi Kleen
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=200802121220.51755.ak@suse.de \
--to=ak@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
--cc=y-goto@jp.fujitsu.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.