diff options
author | David Vrabel <david.vrabel@citrix.com> | 2014-01-07 06:36:53 -0500 |
---|---|---|
committer | David Vrabel <david.vrabel@citrix.com> | 2014-05-15 11:15:01 -0400 |
commit | 2dcc9a3de1d7f77bb4dbc108358a75776328e887 (patch) | |
tree | 74909541226457ce58d59a53f4ae60a583fbed06 /arch/x86/xen | |
parent | 3cb83e46d032505016ab2565f067e24c8cba9a9d (diff) |
x86/xen: only warn once if bad MFNs are found during setup
In xen_add_extra_mem(), if the WARN() checks for bad MFNs trigger it is
likely that they will trigger at lot, spamming the log.
Use WARN_ONCE() instead.
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Tested-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Diffstat (limited to 'arch/x86/xen')
-rw-r--r-- | arch/x86/xen/setup.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c index 0982233b9b84..2afe55e21d59 100644 --- a/arch/x86/xen/setup.c +++ b/arch/x86/xen/setup.c | |||
@@ -89,10 +89,10 @@ static void __init xen_add_extra_mem(u64 start, u64 size) | |||
89 | for (pfn = PFN_DOWN(start); pfn < xen_max_p2m_pfn; pfn++) { | 89 | for (pfn = PFN_DOWN(start); pfn < xen_max_p2m_pfn; pfn++) { |
90 | unsigned long mfn = pfn_to_mfn(pfn); | 90 | unsigned long mfn = pfn_to_mfn(pfn); |
91 | 91 | ||
92 | if (WARN(mfn == pfn, "Trying to over-write 1-1 mapping (pfn: %lx)\n", pfn)) | 92 | if (WARN_ONCE(mfn == pfn, "Trying to over-write 1-1 mapping (pfn: %lx)\n", pfn)) |
93 | continue; | 93 | continue; |
94 | WARN(mfn != INVALID_P2M_ENTRY, "Trying to remove %lx which has %lx mfn!\n", | 94 | WARN_ONCE(mfn != INVALID_P2M_ENTRY, "Trying to remove %lx which has %lx mfn!\n", |
95 | pfn, mfn); | 95 | pfn, mfn); |
96 | 96 | ||
97 | __set_phys_to_machine(pfn, INVALID_P2M_ENTRY); | 97 | __set_phys_to_machine(pfn, INVALID_P2M_ENTRY); |
98 | } | 98 | } |