diff options
| author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2011-01-14 17:55:44 -0500 |
|---|---|---|
| committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2011-03-14 11:17:13 -0400 |
| commit | 146c4e511717e581065800938537b276173d8548 (patch) | |
| tree | 7894b82558b3c5c3fd6e939a73bd99bc3d16468b | |
| parent | fc25151d9ac7d809239fe68de0a1490b504bb94a (diff) | |
xen/m2p: No need to catch exceptions when we know that there is no RAM
.. beyound what we think is the end of memory. However there might
be more System RAM - but assigned to a guest. Hence jump to the
M2P override check and consult.
[v1: Added Review-by tag]
Reviewed-by: Ian Campbell <ian.campbell@citrix.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
| -rw-r--r-- | arch/x86/include/asm/xen/page.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/arch/x86/include/asm/xen/page.h b/arch/x86/include/asm/xen/page.h index 78ebbeb88d9c..195707060493 100644 --- a/arch/x86/include/asm/xen/page.h +++ b/arch/x86/include/asm/xen/page.h | |||
| @@ -85,6 +85,10 @@ static inline unsigned long mfn_to_pfn(unsigned long mfn) | |||
| 85 | if (xen_feature(XENFEAT_auto_translated_physmap)) | 85 | if (xen_feature(XENFEAT_auto_translated_physmap)) |
| 86 | return mfn; | 86 | return mfn; |
| 87 | 87 | ||
| 88 | if (unlikely((mfn >> machine_to_phys_order) != 0)) { | ||
| 89 | pfn = ~0; | ||
| 90 | goto try_override; | ||
| 91 | } | ||
| 88 | pfn = 0; | 92 | pfn = 0; |
| 89 | /* | 93 | /* |
| 90 | * The array access can fail (e.g., device space beyond end of RAM). | 94 | * The array access can fail (e.g., device space beyond end of RAM). |
| @@ -92,7 +96,7 @@ static inline unsigned long mfn_to_pfn(unsigned long mfn) | |||
| 92 | * but we must handle the fault without crashing! | 96 | * but we must handle the fault without crashing! |
| 93 | */ | 97 | */ |
| 94 | __get_user(pfn, &machine_to_phys_mapping[mfn]); | 98 | __get_user(pfn, &machine_to_phys_mapping[mfn]); |
| 95 | 99 | try_override: | |
| 96 | /* | 100 | /* |
| 97 | * If this appears to be a foreign mfn (because the pfn | 101 | * If this appears to be a foreign mfn (because the pfn |
| 98 | * doesn't map back to the mfn), then check the local override | 102 | * doesn't map back to the mfn), then check the local override |
