Commit 1e7a4830 authored by Sean Christopherson's avatar Sean Christopherson Committed by Paolo Bonzini

KVM: VMX: Rename "__find_msr_index" to "__vmx_find_uret_msr"

Rename "__find_msr_index" to scope it to VMX, associate it with
guest_uret_msrs, and to avoid conflating "MSR's ECX index" with "MSR's
array index".  Similarly, don't use "slot" in the name so as to avoid
colliding the common x86's half of "user_return_msrs" (the slot in
kvm_user_return_msrs is not the same slot in guest_uret_msrs).

No functional change intended.
Signed-off-by: default avatarSean Christopherson <sean.j.christopherson@intel.com>
Message-Id: <20200923180409.32255-9-sean.j.christopherson@intel.com>
Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
parent 658ece84
...@@ -611,7 +611,7 @@ static inline bool report_flexpriority(void) ...@@ -611,7 +611,7 @@ static inline bool report_flexpriority(void)
return flexpriority_enabled; return flexpriority_enabled;
} }
static inline int __find_msr_index(struct vcpu_vmx *vmx, u32 msr) static inline int __vmx_find_uret_msr(struct vcpu_vmx *vmx, u32 msr)
{ {
int i; int i;
...@@ -625,7 +625,7 @@ struct vmx_uret_msr *find_msr_entry(struct vcpu_vmx *vmx, u32 msr) ...@@ -625,7 +625,7 @@ struct vmx_uret_msr *find_msr_entry(struct vcpu_vmx *vmx, u32 msr)
{ {
int i; int i;
i = __find_msr_index(vmx, msr); i = __vmx_find_uret_msr(vmx, msr);
if (i >= 0) if (i >= 0)
return &vmx->guest_uret_msrs[i]; return &vmx->guest_uret_msrs[i];
return NULL; return NULL;
...@@ -1637,24 +1637,24 @@ static void setup_msrs(struct vcpu_vmx *vmx) ...@@ -1637,24 +1637,24 @@ static void setup_msrs(struct vcpu_vmx *vmx)
* when EFER.SCE is set. * when EFER.SCE is set.
*/ */
if (is_long_mode(&vmx->vcpu) && (vmx->vcpu.arch.efer & EFER_SCE)) { if (is_long_mode(&vmx->vcpu) && (vmx->vcpu.arch.efer & EFER_SCE)) {
index = __find_msr_index(vmx, MSR_STAR); index = __vmx_find_uret_msr(vmx, MSR_STAR);
if (index >= 0) if (index >= 0)
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
index = __find_msr_index(vmx, MSR_LSTAR); index = __vmx_find_uret_msr(vmx, MSR_LSTAR);
if (index >= 0) if (index >= 0)
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
index = __find_msr_index(vmx, MSR_SYSCALL_MASK); index = __vmx_find_uret_msr(vmx, MSR_SYSCALL_MASK);
if (index >= 0) if (index >= 0)
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
} }
#endif #endif
index = __find_msr_index(vmx, MSR_EFER); index = __vmx_find_uret_msr(vmx, MSR_EFER);
if (index >= 0 && update_transition_efer(vmx, index)) if (index >= 0 && update_transition_efer(vmx, index))
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
index = __find_msr_index(vmx, MSR_TSC_AUX); index = __vmx_find_uret_msr(vmx, MSR_TSC_AUX);
if (index >= 0 && guest_cpuid_has(&vmx->vcpu, X86_FEATURE_RDTSCP)) if (index >= 0 && guest_cpuid_has(&vmx->vcpu, X86_FEATURE_RDTSCP))
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
index = __find_msr_index(vmx, MSR_IA32_TSX_CTRL); index = __vmx_find_uret_msr(vmx, MSR_IA32_TSX_CTRL);
if (index >= 0) if (index >= 0)
move_msr_up(vmx, index, nr_active_uret_msrs++); move_msr_up(vmx, index, nr_active_uret_msrs++);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment