github/acrn-hypervisor
1
0
Fork 0
mirror of https://github.com/projectacrn/acrn-hypervisor.git synced 2025-06-28 00:06:55 +00:00
Code Issues Projects Releases Wiki Activity

HV: remove vcpu arch lock, not needed.

Browse Source 
the pcpu just write its own vmcs, not need spinlock.
and the arch.lock not used other places, remove it too.

Tracked-On: #3130

Signed-off-by: Minggui Cao <minggui.cao@intel.com>
Acked-by: Eddie Dong <eddie.dong@intel.com>
This commit is contained in:
Minggui Cao 2019-05-15 18:52:33 +08:00 committed by ACRN System Integration
parent 9876138b69
commit fc1cbebe31
4 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
hypervisor/arch/x86/cpu_caps.c
View File

@ -9,7 +9,6 @@
#include <page.h> #include <page.h>
#include <cpufeatures.h> #include <cpufeatures.h>
#include <cpuid.h> #include <cpuid.h>
#include <spinlock.h>
#include <cpu.h> #include <cpu.h>
#include <per_cpu.h> #include <per_cpu.h>
#include <vmx.h> #include <vmx.h>

6
hypervisor/arch/x86/guest/vcpu.c
View File

@ -135,19 +135,16 @@ void vcpu_set_vmcs_eoi_exit(struct acrn_vcpu *vcpu)
{ {
pr_dbg("%s", __func__); pr_dbg("%s", __func__);
spinlock_obtain(&(vcpu->arch.lock));
if (is_apicv_advanced_feature_supported()) { if (is_apicv_advanced_feature_supported()) {
exec_vmwrite64(VMX_EOI_EXIT0_FULL, vcpu->arch.eoi_exit_bitmap[0]); exec_vmwrite64(VMX_EOI_EXIT0_FULL, vcpu->arch.eoi_exit_bitmap[0]);
exec_vmwrite64(VMX_EOI_EXIT1_FULL, vcpu->arch.eoi_exit_bitmap[1]); exec_vmwrite64(VMX_EOI_EXIT1_FULL, vcpu->arch.eoi_exit_bitmap[1]);
exec_vmwrite64(VMX_EOI_EXIT2_FULL, vcpu->arch.eoi_exit_bitmap[2]); exec_vmwrite64(VMX_EOI_EXIT2_FULL, vcpu->arch.eoi_exit_bitmap[2]);
exec_vmwrite64(VMX_EOI_EXIT3_FULL, vcpu->arch.eoi_exit_bitmap[3]); exec_vmwrite64(VMX_EOI_EXIT3_FULL, vcpu->arch.eoi_exit_bitmap[3]);
} }
spinlock_release(&(vcpu->arch.lock));
} }
/* /*
* Set the eoi_exit_bitmap bit for specific vector * Set the eoi_exit_bitmap bit for specific vector
* called with vcpu->arch.lock held
* @pre vcpu != NULL && vector <= 255U * @pre vcpu != NULL && vector <= 255U
*/ */
void vcpu_set_eoi_exit_bitmap(struct acrn_vcpu *vcpu, uint32_t vector) void vcpu_set_eoi_exit_bitmap(struct acrn_vcpu *vcpu, uint32_t vector)
@ -172,7 +169,6 @@ void vcpu_clear_eoi_exit_bitmap(struct acrn_vcpu *vcpu, uint32_t vector)
/* /*
* Reset all eoi_exit_bitmaps * Reset all eoi_exit_bitmaps
* called with vcpu->arch.lock held
*/ */
void vcpu_reset_eoi_exit_bitmaps(struct acrn_vcpu *vcpu) void vcpu_reset_eoi_exit_bitmaps(struct acrn_vcpu *vcpu)
{ {
@ -402,8 +398,6 @@ int32_t create_vcpu(uint16_t pcpu_id, struct acrn_vm *vm, struct acrn_vcpu **rtn
init_vmtrr(vcpu); init_vmtrr(vcpu);
} }
spinlock_init(&(vcpu->arch.lock));
/* Populate the return handle */ /* Populate the return handle */
*rtn_vcpu_handle = vcpu; *rtn_vcpu_handle = vcpu;

1
hypervisor/arch/x86/security.c
View File

@ -7,7 +7,6 @@
#include <types.h> #include <types.h>
#include <msr.h> #include <msr.h>
#include <cpufeatures.h> #include <cpufeatures.h>
#include <spinlock.h>
#include <cpu.h> #include <cpu.h>
#include <per_cpu.h> #include <per_cpu.h>
#include <cpu_caps.h> #include <cpu_caps.h>

1
hypervisor/include/arch/x86/guest/vcpu.h
View File

@ -298,7 +298,6 @@ struct acrn_vcpu_arch {
/* EOI_EXIT_BITMAP buffer, for the bitmap update */ /* EOI_EXIT_BITMAP buffer, for the bitmap update */
uint64_t eoi_exit_bitmap[EOI_EXIT_BITMAP_SIZE >> 6U]; uint64_t eoi_exit_bitmap[EOI_EXIT_BITMAP_SIZE >> 6U];
spinlock_t lock;
} __aligned(PAGE_SIZE); } __aligned(PAGE_SIZE);
struct acrn_vm; struct acrn_vm;