diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2008-10-12 13:54:24 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2008-10-13 19:35:27 -0400 |
commit | f5ea64dcbad89875d130596df14c9b25d994a737 (patch) | |
tree | 03bde08104ea5c29edcaafc11576eb5594a7426c /arch/powerpc/mm | |
parent | cd301c7ba4bbb5a0ee6ebf13eb4a304f29b13847 (diff) |
powerpc: Get USE_STRICT_MM_TYPECHECKS working again
The typesafe version of the powerpc pagetable handling (with
USE_STRICT_MM_TYPECHECKS defined) has bitrotted again. This patch
makes a bunch of small fixes to get it back to building status.
It's still not enabled by default as gcc still generates worse
code with it for some reason.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'arch/powerpc/mm')
-rw-r--r-- | arch/powerpc/mm/gup.c | 7 | ||||
-rw-r--r-- | arch/powerpc/mm/hash_utils_64.c | 4 | ||||
-rw-r--r-- | arch/powerpc/mm/init_64.c | 4 |
3 files changed, 8 insertions, 7 deletions
diff --git a/arch/powerpc/mm/gup.c b/arch/powerpc/mm/gup.c index 9fdf4d6335e4..28a114db3ba0 100644 --- a/arch/powerpc/mm/gup.c +++ b/arch/powerpc/mm/gup.c | |||
@@ -41,7 +41,7 @@ static noinline int gup_pte_range(pmd_t pmd, unsigned long addr, | |||
41 | page = pte_page(pte); | 41 | page = pte_page(pte); |
42 | if (!page_cache_get_speculative(page)) | 42 | if (!page_cache_get_speculative(page)) |
43 | return 0; | 43 | return 0; |
44 | if (unlikely(pte != *ptep)) { | 44 | if (unlikely(pte_val(pte) != pte_val(*ptep))) { |
45 | put_page(page); | 45 | put_page(page); |
46 | return 0; | 46 | return 0; |
47 | } | 47 | } |
@@ -92,7 +92,7 @@ static noinline int gup_huge_pte(pte_t *ptep, struct hstate *hstate, | |||
92 | *nr -= refs; | 92 | *nr -= refs; |
93 | return 0; | 93 | return 0; |
94 | } | 94 | } |
95 | if (unlikely(pte != *ptep)) { | 95 | if (unlikely(pte_val(pte) != pte_val(*ptep))) { |
96 | /* Could be optimized better */ | 96 | /* Could be optimized better */ |
97 | while (*nr) { | 97 | while (*nr) { |
98 | put_page(page); | 98 | put_page(page); |
@@ -237,7 +237,8 @@ int get_user_pages_fast(unsigned long start, int nr_pages, int write, | |||
237 | pgd_t pgd = *pgdp; | 237 | pgd_t pgd = *pgdp; |
238 | 238 | ||
239 | VM_BUG_ON(shift != mmu_psize_defs[get_slice_psize(mm, addr)].shift); | 239 | VM_BUG_ON(shift != mmu_psize_defs[get_slice_psize(mm, addr)].shift); |
240 | pr_debug(" %016lx: normal pgd %p\n", addr, (void *)pgd); | 240 | pr_debug(" %016lx: normal pgd %p\n", addr, |
241 | (void *)pgd_val(pgd)); | ||
241 | next = pgd_addr_end(addr, end); | 242 | next = pgd_addr_end(addr, end); |
242 | if (pgd_none(pgd)) | 243 | if (pgd_none(pgd)) |
243 | goto slow; | 244 | goto slow; |
diff --git a/arch/powerpc/mm/hash_utils_64.c b/arch/powerpc/mm/hash_utils_64.c index 09db4efe1921..5c64af174752 100644 --- a/arch/powerpc/mm/hash_utils_64.c +++ b/arch/powerpc/mm/hash_utils_64.c | |||
@@ -541,7 +541,7 @@ static unsigned long __init htab_get_table_size(void) | |||
541 | void create_section_mapping(unsigned long start, unsigned long end) | 541 | void create_section_mapping(unsigned long start, unsigned long end) |
542 | { | 542 | { |
543 | BUG_ON(htab_bolt_mapping(start, end, __pa(start), | 543 | BUG_ON(htab_bolt_mapping(start, end, __pa(start), |
544 | PAGE_KERNEL, mmu_linear_psize, | 544 | pgprot_val(PAGE_KERNEL), mmu_linear_psize, |
545 | mmu_kernel_ssize)); | 545 | mmu_kernel_ssize)); |
546 | } | 546 | } |
547 | 547 | ||
@@ -649,7 +649,7 @@ void __init htab_initialize(void) | |||
649 | mtspr(SPRN_SDR1, _SDR1); | 649 | mtspr(SPRN_SDR1, _SDR1); |
650 | } | 650 | } |
651 | 651 | ||
652 | prot = PAGE_KERNEL; | 652 | prot = pgprot_val(PAGE_KERNEL); |
653 | 653 | ||
654 | #ifdef CONFIG_DEBUG_PAGEALLOC | 654 | #ifdef CONFIG_DEBUG_PAGEALLOC |
655 | linear_map_hash_count = lmb_end_of_DRAM() >> PAGE_SHIFT; | 655 | linear_map_hash_count = lmb_end_of_DRAM() >> PAGE_SHIFT; |
diff --git a/arch/powerpc/mm/init_64.c b/arch/powerpc/mm/init_64.c index 036fe2f10c77..3e6a6543f53a 100644 --- a/arch/powerpc/mm/init_64.c +++ b/arch/powerpc/mm/init_64.c | |||
@@ -228,8 +228,8 @@ int __meminit vmemmap_populate(struct page *start_page, | |||
228 | start, p, __pa(p)); | 228 | start, p, __pa(p)); |
229 | 229 | ||
230 | mapped = htab_bolt_mapping(start, start + page_size, __pa(p), | 230 | mapped = htab_bolt_mapping(start, start + page_size, __pa(p), |
231 | PAGE_KERNEL, mmu_vmemmap_psize, | 231 | pgprot_val(PAGE_KERNEL), |
232 | mmu_kernel_ssize); | 232 | mmu_vmemmap_psize, mmu_kernel_ssize); |
233 | BUG_ON(mapped < 0); | 233 | BUG_ON(mapped < 0); |
234 | } | 234 | } |
235 | 235 | ||