public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Yinghai Lu <yinghai@kernel.org>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Ingo Molnar <mingo@redhat.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	the arch/x86 maintainers <x86@kernel.org>
Subject: Re: memblock vs early_res
Date: Fri, 17 Sep 2010 15:47:18 -0700	[thread overview]
Message-ID: <4C93EFF6.5050806@kernel.org> (raw)
In-Reply-To: <4C93D3F0.3030307@goop.org>

On 09/17/2010 01:47 PM, Jeremy Fitzhardinge wrote:
>  Hi Yinghai,
> 
> I have the patch below floating around in my tree to make sure that
> early-reserved highmem is honoured when freeing unreserved memory.  I
> was trying to rebase it to current linux-next and noticed that all the
> early_res stuff has been replaced with memblock.
> 
> Is this still an issue?  What would the memblock version of this patch
> look like?
> 

Not sure why this patch is needed.

For the early reserve ranges, that could be overlapped with high pages, is "KVA RAM",
but We do remove those range in active ranges array. [ in calculate_numa_remap_pages() ].
[
...
                memblock_x86_reserve_range(node_kva_final,
                              node_kva_final+(((u64)size)<<PAGE_SHIFT),
                              "KVA RAM");

                node_remap_start_pfn[nid] = node_kva_final>>PAGE_SHIFT;
                remove_active_range(nid, node_remap_start_pfn[nid],
                                         node_remap_start_pfn[nid] + size);
...
]

Can you check if Gianluca Guida still can duplicate the problem that will need his patch ?

Thanks

Yinghai Lu

  reply	other threads:[~2010-09-17 22:48 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-17 20:47 memblock vs early_res Jeremy Fitzhardinge
2010-09-17 22:47 ` Yinghai Lu [this message]
2010-09-17 23:11   ` Jeremy Fitzhardinge
2010-09-18  6:10     ` Yinghai Lu
2010-09-18  6:21       ` Jeremy Fitzhardinge
2010-09-22  0:09       ` Jeremy Fitzhardinge

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=4C93EFF6.5050806@kernel.org \
    --to=yinghai@kernel.org \
    --cc=jeremy@goop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=x86@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