Commit d27d4aca authored by Avi Kivity's avatar Avi Kivity

KVM: Cosmetics

Signed-off-by: default avatarAvi Kivity <avi@qumranet.com>
parent 43934a38
...@@ -126,10 +126,8 @@ static inline int valid_vcpu(int n) ...@@ -126,10 +126,8 @@ static inline int valid_vcpu(int n)
return likely(n >= 0 && n < KVM_MAX_VCPUS); return likely(n >= 0 && n < KVM_MAX_VCPUS);
} }
int kvm_read_guest(struct kvm_vcpu *vcpu, int kvm_read_guest(struct kvm_vcpu *vcpu, gva_t addr, unsigned long size,
gva_t addr, void *dest)
unsigned long size,
void *dest)
{ {
unsigned char *host_buf = dest; unsigned char *host_buf = dest;
unsigned long req_size = size; unsigned long req_size = size;
...@@ -161,10 +159,8 @@ int kvm_read_guest(struct kvm_vcpu *vcpu, ...@@ -161,10 +159,8 @@ int kvm_read_guest(struct kvm_vcpu *vcpu,
} }
EXPORT_SYMBOL_GPL(kvm_read_guest); EXPORT_SYMBOL_GPL(kvm_read_guest);
int kvm_write_guest(struct kvm_vcpu *vcpu, int kvm_write_guest(struct kvm_vcpu *vcpu, gva_t addr, unsigned long size,
gva_t addr, void *data)
unsigned long size,
void *data)
{ {
unsigned char *host_buf = data; unsigned char *host_buf = data;
unsigned long req_size = size; unsigned long req_size = size;
...@@ -457,7 +453,7 @@ EXPORT_SYMBOL_GPL(set_cr4); ...@@ -457,7 +453,7 @@ EXPORT_SYMBOL_GPL(set_cr4);
void set_cr3(struct kvm_vcpu *vcpu, unsigned long cr3) void set_cr3(struct kvm_vcpu *vcpu, unsigned long cr3)
{ {
if (is_long_mode(vcpu)) { if (is_long_mode(vcpu)) {
if ( cr3 & CR3_L_MODE_RESEVED_BITS) { if (cr3 & CR3_L_MODE_RESEVED_BITS) {
printk(KERN_DEBUG "set_cr3: #GP, reserved bits\n"); printk(KERN_DEBUG "set_cr3: #GP, reserved bits\n");
inject_gp(vcpu); inject_gp(vcpu);
return; return;
...@@ -774,7 +770,6 @@ static int kvm_dev_ioctl_get_dirty_log(struct kvm *kvm, ...@@ -774,7 +770,6 @@ static int kvm_dev_ioctl_get_dirty_log(struct kvm *kvm,
if (copy_to_user(log->dirty_bitmap, memslot->dirty_bitmap, n)) if (copy_to_user(log->dirty_bitmap, memslot->dirty_bitmap, n))
goto out; goto out;
if (any) { if (any) {
cleared = 0; cleared = 0;
for (i = 0; i < KVM_MAX_VCPUS; ++i) { for (i = 0; i < KVM_MAX_VCPUS; ++i) {
...@@ -903,8 +898,9 @@ static int emulator_read_emulated(unsigned long addr, ...@@ -903,8 +898,9 @@ static int emulator_read_emulated(unsigned long addr,
return X86EMUL_CONTINUE; return X86EMUL_CONTINUE;
else { else {
gpa_t gpa = vcpu->mmu.gva_to_gpa(vcpu, addr); gpa_t gpa = vcpu->mmu.gva_to_gpa(vcpu, addr);
if (gpa == UNMAPPED_GVA) if (gpa == UNMAPPED_GVA)
return vcpu_printf(vcpu, "not present\n"), X86EMUL_PROPAGATE_FAULT; return X86EMUL_PROPAGATE_FAULT;
vcpu->mmio_needed = 1; vcpu->mmio_needed = 1;
vcpu->mmio_phys_addr = gpa; vcpu->mmio_phys_addr = gpa;
vcpu->mmio_size = bytes; vcpu->mmio_size = bytes;
...@@ -1801,12 +1797,11 @@ static long kvm_dev_ioctl(struct file *filp, ...@@ -1801,12 +1797,11 @@ static long kvm_dev_ioctl(struct file *filp,
case KVM_GET_API_VERSION: case KVM_GET_API_VERSION:
r = KVM_API_VERSION; r = KVM_API_VERSION;
break; break;
case KVM_CREATE_VCPU: { case KVM_CREATE_VCPU:
r = kvm_dev_ioctl_create_vcpu(kvm, arg); r = kvm_dev_ioctl_create_vcpu(kvm, arg);
if (r) if (r)
goto out; goto out;
break; break;
}
case KVM_RUN: { case KVM_RUN: {
struct kvm_run kvm_run; struct kvm_run kvm_run;
......
...@@ -441,9 +441,8 @@ static int FNAME(page_fault)(struct kvm_vcpu *vcpu, gva_t addr, ...@@ -441,9 +441,8 @@ static int FNAME(page_fault)(struct kvm_vcpu *vcpu, gva_t addr,
/* /*
* mmio: emulate if accessible, otherwise its a guest fault. * mmio: emulate if accessible, otherwise its a guest fault.
*/ */
if (is_io_pte(*shadow_pte)) { if (is_io_pte(*shadow_pte))
return 1; return 1;
}
++kvm_stat.pf_fixed; ++kvm_stat.pf_fixed;
kvm_mmu_audit(vcpu, "post page fault (fixed)"); kvm_mmu_audit(vcpu, "post page fault (fixed)");
......
...@@ -1042,22 +1042,22 @@ static int io_interception(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run) ...@@ -1042,22 +1042,22 @@ static int io_interception(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run)
addr_mask = io_adress(vcpu, _in, &kvm_run->io.address); addr_mask = io_adress(vcpu, _in, &kvm_run->io.address);
if (!addr_mask) { if (!addr_mask) {
printk(KERN_DEBUG "%s: get io address failed\n", __FUNCTION__); printk(KERN_DEBUG "%s: get io address failed\n",
__FUNCTION__);
return 1; return 1;
} }
if (kvm_run->io.rep) { if (kvm_run->io.rep) {
kvm_run->io.count = vcpu->regs[VCPU_REGS_RCX] & addr_mask; kvm_run->io.count
= vcpu->regs[VCPU_REGS_RCX] & addr_mask;
kvm_run->io.string_down = (vcpu->svm->vmcb->save.rflags kvm_run->io.string_down = (vcpu->svm->vmcb->save.rflags
& X86_EFLAGS_DF) != 0; & X86_EFLAGS_DF) != 0;
} }
} else { } else
kvm_run->io.value = vcpu->svm->vmcb->save.rax; kvm_run->io.value = vcpu->svm->vmcb->save.rax;
}
return 0; return 0;
} }
static int nop_on_interception(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run) static int nop_on_interception(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run)
{ {
return 1; return 1;
......
...@@ -27,7 +27,6 @@ ...@@ -27,7 +27,6 @@
#include "segment_descriptor.h" #include "segment_descriptor.h"
MODULE_AUTHOR("Qumranet"); MODULE_AUTHOR("Qumranet");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
...@@ -418,10 +417,9 @@ static int vmx_set_msr(struct kvm_vcpu *vcpu, u32 msr_index, u64 data) ...@@ -418,10 +417,9 @@ static int vmx_set_msr(struct kvm_vcpu *vcpu, u32 msr_index, u64 data)
case MSR_IA32_SYSENTER_ESP: case MSR_IA32_SYSENTER_ESP:
vmcs_write32(GUEST_SYSENTER_ESP, data); vmcs_write32(GUEST_SYSENTER_ESP, data);
break; break;
case MSR_IA32_TIME_STAMP_COUNTER: { case MSR_IA32_TIME_STAMP_COUNTER:
guest_write_tsc(data); guest_write_tsc(data);
break; break;
}
default: default:
msr = find_msr_entry(vcpu, msr_index); msr = find_msr_entry(vcpu, msr_index);
if (msr) { if (msr) {
......
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