diff options
author | Keir Fraser <keir@xen.org> | 2011-02-07 15:04:32 +0000 |
---|---|---|
committer | Keir Fraser <keir@xen.org> | 2011-02-07 15:04:32 +0000 |
commit | f681990e7fd01564a6b9b847cfc979a7274d96cb (patch) | |
tree | eced26686e2902e9982a08c867c79d915d84b8e8 | |
parent | 26f91ef506f5c0f63a399d84624142f318c4f048 (diff) | |
download | xen-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.c | 9 |
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; |