diff options
author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2012-03-21 11:43:32 -0400 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2012-03-21 12:17:22 -0400 |
commit | 27257fc07c044af99d85400c4bab670342bbc8a5 (patch) | |
tree | 31d3ea8c7a681407d7a226e32cca277056d10b0e | |
parent | 8e6f7c23c135b13f3adf90906fac7edd325bb9af (diff) |
xen/acpi: Remove the WARN's as they just create noise.
When booting the kernel under machines that do not have P-states
we would end up with:
------------[ cut here ]------------
WARNING: at drivers/xen/xen-acpi-processor.c:504
xen_acpi_processor_init+0x286/0
x2e0()
Hardware name: ProLiant BL460c G6
Modules linked in:
Pid: 1, comm: swapper Not tainted 2.6.39-200.0.3.el5uek #1
Call Trace:
[<ffffffff8191d056>] ? xen_acpi_processor_init+0x286/0x2e0
[<ffffffff81068300>] warn_slowpath_common+0x90/0xc0
[<ffffffff8191cdd0>] ? check_acpi_ids+0x1e0/0x1e0
[<ffffffff8106834a>] warn_slowpath_null+0x1a/0x20
[<ffffffff8191d056>] xen_acpi_processor_init+0x286/0x2e0
[<ffffffff8191cdd0>] ? check_acpi_ids+0x1e0/0x1e0
[<ffffffff81002168>] do_one_initcall+0xe8/0x130
.. snip..
Which is OK - the machines do not have P-states, so we fail to register
to process the _PXX states. But there is no need to WARN the user
of it.
Oracle BZ# 13871288
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
-rw-r--r-- | drivers/xen/xen-acpi-processor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/xen/xen-acpi-processor.c b/drivers/xen/xen-acpi-processor.c index 5c2be963aa18..174b5653cd8a 100644 --- a/drivers/xen/xen-acpi-processor.c +++ b/drivers/xen/xen-acpi-processor.c | |||
@@ -501,11 +501,11 @@ static int __init xen_acpi_processor_init(void) | |||
501 | 501 | ||
502 | perf = per_cpu_ptr(acpi_perf_data, i); | 502 | perf = per_cpu_ptr(acpi_perf_data, i); |
503 | rc = acpi_processor_register_performance(perf, i); | 503 | rc = acpi_processor_register_performance(perf, i); |
504 | if (WARN_ON(rc)) | 504 | if (rc) |
505 | goto err_out; | 505 | goto err_out; |
506 | } | 506 | } |
507 | rc = acpi_processor_notify_smm(THIS_MODULE); | 507 | rc = acpi_processor_notify_smm(THIS_MODULE); |
508 | if (WARN_ON(rc)) | 508 | if (rc) |
509 | goto err_unregister; | 509 | goto err_unregister; |
510 | 510 | ||
511 | for_each_possible_cpu(i) { | 511 | for_each_possible_cpu(i) { |