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/arch/x86/setup.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/arch/x86/setup.c')
-rw-r--r-- | xen/arch/x86/setup.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c index d6f9e11f59..4bce2f4739 100644 --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -1246,7 +1246,11 @@ void __init __start_xen(unsigned long mbi_p) numa_add_cpu(i); if ( (num_online_cpus() < max_cpus) && !cpu_online(i) ) - cpu_up(i); + { + int ret = cpu_up(i); + if ( ret != 0 ) + printk("Failed to bring up CPU %u (error %d)\n", i, ret); + } } printk("Brought up %ld CPUs\n", (long)num_online_cpus()); |