diff options
author | Dave Hansen <dave@linux.vnet.ibm.com> | 2013-01-22 16:24:32 -0500 |
---|---|---|
committer | H. Peter Anvin <hpa@linux.intel.com> | 2013-01-25 19:33:23 -0500 |
commit | f3c4fbb68e93b10c781c0cc462a9d80770244da6 (patch) | |
tree | 6a56b1d5688f23361c3f305c2902053bdcd098ab | |
parent | 4cbeb51b860c57ba8b2ae50c4016ee7a41f5fbd5 (diff) |
x86, mm: Use new pagetable helpers in try_preserve_large_page()
try_preserve_large_page() can be slightly simplified by using
the new page_level_*() helpers. This also moves the 'level'
over to the new pg_level enum type.
Signed-off-by: Dave Hansen <dave@linux.vnet.ibm.com>
Link: http://lkml.kernel.org/r/20130122212432.14F3D993@kernel.stglabs.ibm.com
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
-rw-r--r-- | arch/x86/mm/pageattr.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c index 40f92f3aea54..2a5c9ab710b9 100644 --- a/arch/x86/mm/pageattr.c +++ b/arch/x86/mm/pageattr.c | |||
@@ -396,7 +396,7 @@ try_preserve_large_page(pte_t *kpte, unsigned long address, | |||
396 | pte_t new_pte, old_pte, *tmp; | 396 | pte_t new_pte, old_pte, *tmp; |
397 | pgprot_t old_prot, new_prot, req_prot; | 397 | pgprot_t old_prot, new_prot, req_prot; |
398 | int i, do_split = 1; | 398 | int i, do_split = 1; |
399 | unsigned int level; | 399 | enum pg_level level; |
400 | 400 | ||
401 | if (cpa->force_split) | 401 | if (cpa->force_split) |
402 | return 1; | 402 | return 1; |
@@ -412,15 +412,12 @@ try_preserve_large_page(pte_t *kpte, unsigned long address, | |||
412 | 412 | ||
413 | switch (level) { | 413 | switch (level) { |
414 | case PG_LEVEL_2M: | 414 | case PG_LEVEL_2M: |
415 | psize = PMD_PAGE_SIZE; | ||
416 | pmask = PMD_PAGE_MASK; | ||
417 | break; | ||
418 | #ifdef CONFIG_X86_64 | 415 | #ifdef CONFIG_X86_64 |
419 | case PG_LEVEL_1G: | 416 | case PG_LEVEL_1G: |
420 | psize = PUD_PAGE_SIZE; | ||
421 | pmask = PUD_PAGE_MASK; | ||
422 | break; | ||
423 | #endif | 417 | #endif |
418 | psize = page_level_size(level); | ||
419 | pmask = page_level_mask(level); | ||
420 | break; | ||
424 | default: | 421 | default: |
425 | do_split = -EINVAL; | 422 | do_split = -EINVAL; |
426 | goto out_unlock; | 423 | goto out_unlock; |