diff options
| author | Keir Fraser <keir.fraser@citrix.com> | 2009-06-03 11:17:00 +0100 |
|---|---|---|
| committer | Keir Fraser <keir.fraser@citrix.com> | 2009-06-03 11:17:00 +0100 |
| commit | 894ea3d395236601cfed1da0fee6eec6318b9e8a (patch) | |
| tree | bf0685c9415bfdf6b61e208f8b13880ed2606b51 /xen/arch/x86/platform_hypercall.c | |
| parent | 4b48ffd60a4b9177e90b069024a26ec31f5e9e6b (diff) | |
| download | xen-894ea3d395236601cfed1da0fee6eec6318b9e8a.tar.gz xen-894ea3d395236601cfed1da0fee6eec6318b9e8a.tar.bz2 xen-894ea3d395236601cfed1da0fee6eec6318b9e8a.zip | |
x86: Fix XENPF_getidletime to correctly modify cpumask.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
Diffstat (limited to 'xen/arch/x86/platform_hypercall.c')
| -rw-r--r-- | xen/arch/x86/platform_hypercall.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/xen/arch/x86/platform_hypercall.c b/xen/arch/x86/platform_hypercall.c index 4bf6777923..bb2f31380d 100644 --- a/xen/arch/x86/platform_hypercall.c +++ b/xen/arch/x86/platform_hypercall.c @@ -313,7 +313,6 @@ ret_t do_platform_op(XEN_GUEST_HANDLE(xen_platform_op_t) u_xenpf_op) { uint32_t cpu; uint64_t idletime, now = NOW(); - struct vcpu *v; struct xenctl_cpumap ctlmap; cpumask_t cpumap; XEN_GUEST_HANDLE(uint8) cpumap_bitmap; @@ -336,7 +335,7 @@ ret_t do_platform_op(XEN_GUEST_HANDLE(xen_platform_op_t) u_xenpf_op) for_each_cpu_mask ( cpu, cpumap ) { - if ( (v = idle_vcpu[cpu]) != NULL ) + if ( idle_vcpu[cpu] == NULL ) cpu_clear(cpu, cpumap); idletime = get_cpu_idle_time(cpu); |