diff options
| author | Izik Eidus <izike@qumranet.com> | 2007-11-19 11:16:57 +0200 |
|---|---|---|
| committer | Avi Kivity <avi@qumranet.com> | 2008-01-30 17:53:06 +0200 |
| commit | 3e021bf5053bc45448e43cf1d7d020ddf1409714 (patch) | |
| tree | 77068aa73bcb6db4af655fce0a48b9c375c2d758 | |
| parent | ec8d4eaefc06636eb8fc83610c6605168796b4f7 (diff) | |
KVM: Simplify kvm_clear_guest_page()
Use kvm_write_guest_page() with empty_zero_page, instead of doing
kmap and memset.
Signed-off-by: Izik Eidus <izike@qumranet.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
| -rw-r--r-- | drivers/kvm/kvm_main.c | 18 |
1 files changed, 2 insertions, 16 deletions
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c index 595ed323fc53..e83a178dc80f 100644 --- a/drivers/kvm/kvm_main.c +++ b/drivers/kvm/kvm_main.c @@ -46,6 +46,7 @@ #include <asm/io.h> #include <asm/uaccess.h> #include <asm/desc.h> +#include <asm/pgtable.h> MODULE_AUTHOR("Qumranet"); MODULE_LICENSE("GPL"); @@ -633,22 +634,7 @@ int kvm_write_guest(struct kvm *kvm, gpa_t gpa, const void *data, int kvm_clear_guest_page(struct kvm *kvm, gfn_t gfn, int offset, int len) { - void *page_virt; - struct page *page; - - page = gfn_to_page(kvm, gfn); - if (is_error_page(page)) { - kvm_release_page(page); - return -EFAULT; - } - page_virt = kmap_atomic(page, KM_USER0); - - memset(page_virt + offset, 0, len); - - kunmap_atomic(page_virt, KM_USER0); - kvm_release_page(page); - mark_page_dirty(kvm, gfn); - return 0; + return kvm_write_guest_page(kvm, gfn, empty_zero_page, offset, len); } EXPORT_SYMBOL_GPL(kvm_clear_guest_page); |