Messages in this thread Patch in this message | | | From | Glauber Costa <> | Subject | [PATCH] always assign userspace_addr | Date | Tue, 18 Nov 2008 01:04:22 -0200 |
| |
Currently, kvm only sets new.userspace_addr in slots that were just allocated. This is not the intended behaviour, and actually breaks when we try to use the slots to implement aliases, for example.
Cirrus VGA aliases maps and address to a userspace address, and then keep mapping this same address to different locations until the whole screen is filled.
The solution is to assign new.userspace_addr no matter what, so we can be sure that whenever the guest changes this field, it sees the change being reflected in the code.
Signed-off-by: Glauber Costa <glommer@redhat.com> --- virt/kvm/kvm_main.c | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index a87f45e..fc3abf0 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -762,15 +762,6 @@ int __kvm_set_memory_region(struct kvm *kvm, memset(new.rmap, 0, npages * sizeof(*new.rmap)); new.user_alloc = user_alloc; - /* - * hva_to_rmmap() serialzies with the mmu_lock and to be - * safe it has to ignore memslots with !user_alloc && - * !userspace_addr. - */ - if (user_alloc) - new.userspace_addr = mem->userspace_addr; - else - new.userspace_addr = 0; } if (npages && !new.lpage_info) { int largepages = npages / KVM_PAGES_PER_HPAGE; @@ -791,6 +782,15 @@ int __kvm_set_memory_region(struct kvm *kvm, if ((base_gfn+npages) % KVM_PAGES_PER_HPAGE) new.lpage_info[largepages-1].write_count = 1; } + /* + * hva_to_rmmap() serialzies with the mmu_lock and to be + * safe it has to ignore memslots with !user_alloc && + * !userspace_addr. + */ + if (npages && user_alloc) + new.userspace_addr = mem->userspace_addr; + else + new.userspace_addr = 0; /* Allocate page dirty bitmap if needed */ if ((new.flags & KVM_MEM_LOG_DIRTY_PAGES) && !new.dirty_bitmap) { -- 1.5.6.5
| |