aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKeir Fraser <keir@xen.org>2011-02-07 15:04:32 +0000
committerKeir Fraser <keir@xen.org>2011-02-07 15:04:32 +0000
commitf681990e7fd01564a6b9b847cfc979a7274d96cb (patch)
treeeced26686e2902e9982a08c867c79d915d84b8e8
parent26f91ef506f5c0f63a399d84624142f318c4f048 (diff)
downloadxen-f681990e7fd01564a6b9b847cfc979a7274d96cb.tar.gz
xen-f681990e7fd01564a6b9b847cfc979a7274d96cb.tar.bz2
xen-f681990e7fd01564a6b9b847cfc979a7274d96cb.zip
cpupool: Correct cpupool diag printing
Some of the cpupool_dprintk() calls are using undefined or uninitialized variables. Correct the argument lists to be able to define cpupool_printk as printk. Signed-off-by: Juergen Gross <juergen.gross@ts.fujitsu.com>
-rw-r--r--xen/common/cpupool.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/xen/common/cpupool.c b/xen/common/cpupool.c
index 7fd3ad849b..b90e5c0371 100644
--- a/xen/common/cpupool.c
+++ b/xen/common/cpupool.c
@@ -232,8 +232,8 @@ static long cpupool_unassign_cpu_helper(void *info)
int cpu = cpupool_moving_cpu;
long ret;
- cpupool_dprintk("cpupool_unassign_cpu(pool=%d,cpu=%d) ret %ld\n",
- cpupool_id, cpu, ret);
+ cpupool_dprintk("cpupool_unassign_cpu(pool=%d,cpu=%d)\n",
+ cpupool_cpu_moving->cpupool_id, cpu);
spin_lock(&cpupool_lock);
ret = cpu_disable_scheduler(cpu);
@@ -254,6 +254,7 @@ static long cpupool_unassign_cpu_helper(void *info)
out:
spin_unlock(&cpupool_lock);
+ cpupool_dprintk("cpupool_unassign_cpu ret=%ld\n", ret);
return ret;
}
@@ -331,7 +332,7 @@ int cpupool_unassign_cpu(struct cpupool *c, unsigned int cpu)
out:
spin_unlock(&cpupool_lock);
cpupool_dprintk("cpupool_unassign_cpu(pool=%d,cpu=%d) ret %d\n",
- cpupool_id, cpu, ret);
+ c->cpupool_id, cpu, ret);
return ret;
}
@@ -345,7 +346,7 @@ int cpupool_add_domain(struct domain *d, int poolid)
{
struct cpupool *c;
int rc = 1;
- int n_dom;
+ int n_dom = 0;
if ( poolid == CPUPOOLID_NONE )
return 0;