diff options
author | Keir Fraser <keir@xen.org> | 2011-01-14 09:52:02 +0000 |
---|---|---|
committer | Keir Fraser <keir@xen.org> | 2011-01-14 09:52:02 +0000 |
commit | 2d87af888ff25608e5a09d6d6eb3f1c292051a89 (patch) | |
tree | 382999eceb51aedf54247df22574503e8a327d7b /xen/common/cpu.c | |
parent | 835e8cf959f4a6537374c3cf9cd43ff9fb35e500 (diff) | |
download | xen-2d87af888ff25608e5a09d6d6eb3f1c292051a89.tar.gz xen-2d87af888ff25608e5a09d6d6eb3f1c292051a89.tar.bz2 xen-2d87af888ff25608e5a09d6d6eb3f1c292051a89.zip |
cpu hotplug: Core functions are quiet on failure.
This was already inconsistent, so make them consistently quiet and
leave it to callers to log an error. Add suitable error logging to the
arch-specific CPU bringup loops,
In particular this avoids printing error on EBUSY, in which case
caller may want a silent retry loop.
Signed-off-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/common/cpu.c')
-rw-r--r-- | xen/common/cpu.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/xen/common/cpu.c b/xen/common/cpu.c index 2a248275a2..5cdfd0fb10 100644 --- a/xen/common/cpu.c +++ b/xen/common/cpu.c @@ -108,7 +108,6 @@ int cpu_down(unsigned int cpu) fail: notifier_rc = notifier_call_chain(&cpu_chain, CPU_DOWN_FAILED, hcpu, &nb); BUG_ON(notifier_rc != NOTIFY_DONE); - printk("Failed to take down CPU %u (error %d)\n", cpu, err); cpu_hotplug_done(); return err; } @@ -150,7 +149,6 @@ int cpu_up(unsigned int cpu) fail: notifier_rc = notifier_call_chain(&cpu_chain, CPU_UP_CANCELED, hcpu, &nb); BUG_ON(notifier_rc != NOTIFY_DONE); - printk("Failed to bring up CPU %u (error %d)\n", cpu, err); cpu_hotplug_done(); return err; } |