From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Mel Gorman <mel@skynet.ie>
Cc: Judith Lebzelter <judith@osdl.org>,
linux-kernel@vger.kernel.org, akpm@osdl.org,
Paul Mackerras <paulus@samba.org>
Subject: Re: 2.6.18-rc6-mm2: fix for error compiling ppc/mm/init.c
Date: Wed, 04 Oct 2006 13:22:38 +1000 [thread overview]
Message-ID: <1159932158.8469.12.camel@localhost.localdomain> (raw)
In-Reply-To: <20061003130457.GA17358@skynet.ie>
Patch looks good on a quick look. I will try to test later today at
which point, it should go in asap, current upstream doesn't boot on
CONFIG_HIGHMEM powerpc, which is a lot of them...
Ben.
>
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/i386/kernel/setup.c linux-2.6.18-git19-use_zone_symbolic_names/arch/i386/kernel/setup.c
> --- linux-2.6.18-git19-clean/arch/i386/kernel/setup.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/i386/kernel/setup.c 2006-10-03 11:15:59.000000000 +0100
> @@ -1083,16 +1083,15 @@ static unsigned long __init setup_memory
>
> void __init zone_sizes_init(void)
> {
> + unsigned long max_zone_pfns[MAX_NR_ZONES];
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> + max_zone_pfns[ZONE_DMA] =
> + virt_to_phys((char *)MAX_DMA_ADDRESS) >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_NORMAL] = max_low_pfn;
> #ifdef CONFIG_HIGHMEM
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = {
> - virt_to_phys((char *)MAX_DMA_ADDRESS) >> PAGE_SHIFT,
> - max_low_pfn,
> - highend_pfn};
> + max_zone_pfns[ZONE_HIGHMEM] = highend_pfn;
> add_active_range(0, 0, highend_pfn);
> #else
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = {
> - virt_to_phys((char *)MAX_DMA_ADDRESS) >> PAGE_SHIFT,
> - max_low_pfn};
> add_active_range(0, 0, max_low_pfn);
> #endif
>
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/i386/mm/discontig.c linux-2.6.18-git19-use_zone_symbolic_names/arch/i386/mm/discontig.c
> --- linux-2.6.18-git19-clean/arch/i386/mm/discontig.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/i386/mm/discontig.c 2006-10-03 11:17:27.000000000 +0100
> @@ -357,11 +357,12 @@ void __init numa_kva_reserve(void)
> void __init zone_sizes_init(void)
> {
> int nid;
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = {
> - virt_to_phys((char *)MAX_DMA_ADDRESS) >> PAGE_SHIFT,
> - max_low_pfn,
> - highend_pfn
> - };
> + unsigned long max_zone_pfns[MAX_NR_ZONES];
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> + max_zone_pfns[ZONE_DMA] =
> + virt_to_phys((char *)MAX_DMA_ADDRESS) >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_NORMAL] = max_low_pfn;
> + max_zone_pfns[ZONE_HIGHMEM] = highend_pfn;
>
> /* If SRAT has not registered memory, register it now */
> if (find_max_pfn_with_active_regions() == 0) {
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/ia64/mm/contig.c linux-2.6.18-git19-use_zone_symbolic_names/arch/ia64/mm/contig.c
> --- linux-2.6.18-git19-clean/arch/ia64/mm/contig.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/ia64/mm/contig.c 2006-10-03 11:21:09.000000000 +0100
> @@ -233,6 +233,7 @@ paging_init (void)
> efi_memmap_walk(count_pages, &num_physpages);
>
> max_dma = virt_to_phys((void *) MAX_DMA_ADDRESS) >> PAGE_SHIFT;
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> max_zone_pfns[ZONE_DMA] = max_dma;
> max_zone_pfns[ZONE_NORMAL] = max_low_pfn;
>
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/ia64/mm/discontig.c linux-2.6.18-git19-use_zone_symbolic_names/arch/ia64/mm/discontig.c
> --- linux-2.6.18-git19-clean/arch/ia64/mm/discontig.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/ia64/mm/discontig.c 2006-10-03 11:21:54.000000000 +0100
> @@ -709,6 +709,7 @@ void __init paging_init(void)
> max_pfn = mem_data[node].max_pfn;
> }
>
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> max_zone_pfns[ZONE_DMA] = max_dma;
> max_zone_pfns[ZONE_NORMAL] = max_pfn;
> free_area_init_nodes(max_zone_pfns);
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/powerpc/mm/mem.c linux-2.6.18-git19-use_zone_symbolic_names/arch/powerpc/mm/mem.c
> --- linux-2.6.18-git19-clean/arch/powerpc/mm/mem.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/powerpc/mm/mem.c 2006-10-03 11:10:57.000000000 +0100
> @@ -307,11 +307,12 @@ void __init paging_init(void)
> top_of_ram, total_ram);
> printk(KERN_DEBUG "Memory hole size: %ldMB\n",
> (top_of_ram - total_ram) >> 20);
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> #ifdef CONFIG_HIGHMEM
> - max_zone_pfns[0] = total_lowmem >> PAGE_SHIFT;
> - max_zone_pfns[1] = top_of_ram >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_DMA] = total_lowmem >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_HIGHMEM] = top_of_ram >> PAGE_SHIFT;
> #else
> - max_zone_pfns[0] = top_of_ram >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_DMA] = top_of_ram >> PAGE_SHIFT;
> #endif
> free_area_init_nodes(max_zone_pfns);
> }
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/powerpc/mm/numa.c linux-2.6.18-git19-use_zone_symbolic_names/arch/powerpc/mm/numa.c
> --- linux-2.6.18-git19-clean/arch/powerpc/mm/numa.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/powerpc/mm/numa.c 2006-10-03 11:12:01.000000000 +0100
> @@ -617,9 +617,9 @@ void __init do_init_bootmem(void)
>
> void __init paging_init(void)
> {
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = {
> - lmb_end_of_DRAM() >> PAGE_SHIFT
> - };
> + unsigned long max_zone_pfns[MAX_NR_ZONES];
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> + max_zone_pfns[ZONE_DMA] = lmb_end_of_DRAM() >> PAGE_SHIFT;
> free_area_init_nodes(max_zone_pfns);
> }
>
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/ppc/mm/init.c linux-2.6.18-git19-use_zone_symbolic_names/arch/ppc/mm/init.c
> --- linux-2.6.18-git19-clean/arch/ppc/mm/init.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/ppc/mm/init.c 2006-10-03 11:13:46.000000000 +0100
> @@ -374,11 +374,12 @@ void __init paging_init(void)
> end_pfn = start_pfn + (total_memory >> PAGE_SHIFT);
> add_active_range(0, start_pfn, end_pfn);
>
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> #ifdef CONFIG_HIGHMEM
> - max_zone_pfns[0] = total_lowmem >> PAGE_SHIFT;
> - max_zone_pfns[1] = total_memory >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_DMA] = total_lowmem >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_HIGHMEM] = total_memory >> PAGE_SHIFT;
> #else
> - max_zone_pfns[0] = total_memory >> PAGE_SHIFT;
> + max_zone_pfns[ZONE_DMA] = total_memory >> PAGE_SHIFT;
> #endif /* CONFIG_HIGHMEM */
> free_area_init_nodes(max_zone_pfns);
> }
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/x86_64/mm/init.c linux-2.6.18-git19-use_zone_symbolic_names/arch/x86_64/mm/init.c
> --- linux-2.6.18-git19-clean/arch/x86_64/mm/init.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/x86_64/mm/init.c 2006-10-03 11:18:54.000000000 +0100
> @@ -406,9 +406,12 @@ void __cpuinit zap_low_mappings(int cpu)
> #ifndef CONFIG_NUMA
> void __init paging_init(void)
> {
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = {MAX_DMA_PFN,
> - MAX_DMA32_PFN,
> - end_pfn};
> + unsigned long max_zone_pfns[MAX_NR_ZONES];
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> + max_zone_pfns[ZONE_DMA] = MAX_DMA_PFN;
> + max_zone_pfns[ZONE_DMA32] = MAX_DMA32_PFN;
> + max_zone_pfns[ZONE_NORMAL] = end_pfn;
> +
> memory_present(0, 0, end_pfn);
> sparse_init();
> free_area_init_nodes(max_zone_pfns);
> diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.18-git19-clean/arch/x86_64/mm/numa.c linux-2.6.18-git19-use_zone_symbolic_names/arch/x86_64/mm/numa.c
> --- linux-2.6.18-git19-clean/arch/x86_64/mm/numa.c 2006-10-03 09:12:56.000000000 +0100
> +++ linux-2.6.18-git19-use_zone_symbolic_names/arch/x86_64/mm/numa.c 2006-10-03 11:19:47.000000000 +0100
> @@ -338,9 +338,11 @@ static void __init arch_sparse_init(void
> void __init paging_init(void)
> {
> int i;
> - unsigned long max_zone_pfns[MAX_NR_ZONES] = { MAX_DMA_PFN,
> - MAX_DMA32_PFN,
> - end_pfn};
> + unsigned long max_zone_pfns[MAX_NR_ZONES];
> + memset(max_zone_pfns, 0, sizeof(max_zone_pfns));
> + max_zone_pfns[ZONE_DMA] = MAX_DMA_PFN;
> + max_zone_pfns[ZONE_DMA32] = MAX_DMA32_PFN;
> + max_zone_pfns[ZONE_NORMAL] = end_pfn;
>
> arch_sparse_init();
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2006-10-04 4:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-14 17:37 2.6.18-rc6-mm2: fix for error compiling ppc/mm/init.c Judith Lebzelter
2006-09-14 18:11 ` Mel Gorman
2006-10-03 4:24 ` Benjamin Herrenschmidt
2006-10-03 8:51 ` Mel Gorman
2006-10-03 9:19 ` Benjamin Herrenschmidt
2006-10-03 9:34 ` Mel Gorman
2006-10-03 12:12 ` Benjamin Herrenschmidt
2006-10-03 13:04 ` Mel Gorman
2006-10-04 3:22 ` Benjamin Herrenschmidt [this message]
2006-10-04 14:12 ` Mel Gorman
2006-10-05 9:09 ` Benjamin Herrenschmidt
2006-10-09 23:25 ` Paul Mackerras
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=1159932158.8469.12.camel@localhost.localdomain \
--to=benh@kernel.crashing.org \
--cc=akpm@osdl.org \
--cc=judith@osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mel@skynet.ie \
--cc=paulus@samba.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