diff options
author | Jan Beulich <jbeulich@suse.com> | 2013-08-27 11:13:50 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2013-08-27 11:13:50 +0200 |
commit | 784ce3fd05476ffe46bf54579f3927c777eb2c3b (patch) | |
tree | 525f48dc4d1e27693edabb167207178ef3943e3d /xen/arch/x86/hvm/vmx | |
parent | d838ac2539cf1987bea6e15662fd6a80a58fe26d (diff) | |
download | xen-784ce3fd05476ffe46bf54579f3927c777eb2c3b.tar.gz xen-784ce3fd05476ffe46bf54579f3927c777eb2c3b.tar.bz2 xen-784ce3fd05476ffe46bf54579f3927c777eb2c3b.zip |
VMX: convert EOI exit bitmap to a proper bitmap
... allowing bitmap operations to be used on it, making things
consistent with struct pi_desc's pir field, and shrinking overall
source code size.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/arch/x86/hvm/vmx')
-rw-r--r-- | xen/arch/x86/hvm/vmx/intr.c | 16 | ||||
-rw-r--r-- | xen/arch/x86/hvm/vmx/vmcs.c | 35 |
2 files changed, 19 insertions, 32 deletions
diff --git a/xen/arch/x86/hvm/vmx/intr.c b/xen/arch/x86/hvm/vmx/intr.c index 45942ab43a..e6d5b46c64 100644 --- a/xen/arch/x86/hvm/vmx/intr.c +++ b/xen/arch/x86/hvm/vmx/intr.c @@ -285,6 +285,7 @@ void vmx_intr_assist(void) intack.source != hvm_intsrc_vector ) { unsigned long status = __vmread(GUEST_INTR_STATUS); + unsigned int i, n; /* * Set eoi_exit_bitmap for periodic timer interrup to cause EOI-induced VM @@ -299,14 +300,13 @@ void vmx_intr_assist(void) status |= VMX_GUEST_INTR_STATUS_SUBFIELD_BITMASK & intack.vector; __vmwrite(GUEST_INTR_STATUS, status); - if (v->arch.hvm_vmx.eoi_exitmap_changed) { -#define UPDATE_EOI_EXITMAP(v, e) { \ - if (test_and_clear_bit(e, &v->arch.hvm_vmx.eoi_exitmap_changed)) { \ - __vmwrite(EOI_EXIT_BITMAP##e, v->arch.hvm_vmx.eoi_exit_bitmap[e]);}} - UPDATE_EOI_EXITMAP(v, 0); - UPDATE_EOI_EXITMAP(v, 1); - UPDATE_EOI_EXITMAP(v, 2); - UPDATE_EOI_EXITMAP(v, 3); + + n = ARRAY_SIZE(v->arch.hvm_vmx.eoi_exit_bitmap); + while ( (i = find_first_bit(&v->arch.hvm_vmx.eoi_exitmap_changed, + n)) < n ) + { + clear_bit(i, &v->arch.hvm_vmx.eoi_exitmap_changed); + __vmwrite(EOI_EXIT_BITMAP(i), v->arch.hvm_vmx.eoi_exit_bitmap[i]); } pt_intr_post(v, intack); diff --git a/xen/arch/x86/hvm/vmx/vmcs.c b/xen/arch/x86/hvm/vmx/vmcs.c index 79efb3f639..31347bb014 100644 --- a/xen/arch/x86/hvm/vmx/vmcs.c +++ b/xen/arch/x86/hvm/vmx/vmcs.c @@ -915,15 +915,12 @@ static int construct_vmcs(struct vcpu *v) if ( cpu_has_vmx_virtual_intr_delivery ) { + unsigned int i; + /* EOI-exit bitmap */ - v->arch.hvm_vmx.eoi_exit_bitmap[0] = (uint64_t)0; - __vmwrite(EOI_EXIT_BITMAP0, v->arch.hvm_vmx.eoi_exit_bitmap[0]); - v->arch.hvm_vmx.eoi_exit_bitmap[1] = (uint64_t)0; - __vmwrite(EOI_EXIT_BITMAP1, v->arch.hvm_vmx.eoi_exit_bitmap[1]); - v->arch.hvm_vmx.eoi_exit_bitmap[2] = (uint64_t)0; - __vmwrite(EOI_EXIT_BITMAP2, v->arch.hvm_vmx.eoi_exit_bitmap[2]); - v->arch.hvm_vmx.eoi_exit_bitmap[3] = (uint64_t)0; - __vmwrite(EOI_EXIT_BITMAP3, v->arch.hvm_vmx.eoi_exit_bitmap[3]); + bitmap_zero(v->arch.hvm_vmx.eoi_exit_bitmap, NR_VECTORS); + for ( i = 0; i < ARRAY_SIZE(v->arch.hvm_vmx.eoi_exit_bitmap); ++i ) + __vmwrite(EOI_EXIT_BITMAP(i), 0); /* Initialise Guest Interrupt Status (RVI and SVI) to 0 */ __vmwrite(GUEST_INTR_STATUS, 0); @@ -1171,26 +1168,16 @@ int vmx_add_host_load_msr(u32 msr) void vmx_set_eoi_exit_bitmap(struct vcpu *v, u8 vector) { - int index, offset, changed; - - index = vector >> 6; - offset = vector & 63; - changed = !test_and_set_bit(offset, - (uint64_t *)&v->arch.hvm_vmx.eoi_exit_bitmap[index]); - if (changed) - set_bit(index, &v->arch.hvm_vmx.eoi_exitmap_changed); + if ( !test_and_set_bit(vector, v->arch.hvm_vmx.eoi_exit_bitmap) ) + set_bit(vector / BITS_PER_LONG, + &v->arch.hvm_vmx.eoi_exitmap_changed); } void vmx_clear_eoi_exit_bitmap(struct vcpu *v, u8 vector) { - int index, offset, changed; - - index = vector >> 6; - offset = vector & 63; - changed = test_and_clear_bit(offset, - (uint64_t *)&v->arch.hvm_vmx.eoi_exit_bitmap[index]); - if (changed) - set_bit(index, &v->arch.hvm_vmx.eoi_exitmap_changed); + if ( test_and_clear_bit(vector, v->arch.hvm_vmx.eoi_exit_bitmap) ) + set_bit(vector / BITS_PER_LONG, + &v->arch.hvm_vmx.eoi_exitmap_changed); } int vmx_create_vmcs(struct vcpu *v) |