Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Paolo Bonzini
Il 18/06/2013 14:39, Zhanghaoyu (A) ha scritto: >> Applied to master, but check your setup because I had to do so manually. >> Your patch has 2 tabs + 1 space for each indentation level (??), the file >> only has one tab. >> >> This time the patch was pretty simple and may even be worth having in

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Zhanghaoyu (A)
>> __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr >> is called in two flows, one is invoked by guest, call stack shown as >> below, handle_xsetbv(or xsetbv_interception) >> kvm_set_xcr >> __kvm_set_xcr >> the other one is invoked by host(QEMU), call stack shown as bel

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-18 Thread Paolo Bonzini
Il 14/06/2013 09:36, Zhanghaoyu (A) ha scritto: > __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr is > called in two flows, > one is invoked by guest, call stack shown as below, > handle_xsetbv(or xsetbv_interception) > kvm_set_xcr > __kvm_set_xcr > the other one is inv

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-17 Thread Zhanghaoyu (A)
>> __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr >> is called in two flows, one is invoked by guest, call stack shown as >> below, handle_xsetbv(or xsetbv_interception) >> kvm_set_xcr >> __kvm_set_xcr >> the other one is invoked by host(QEMU), call stack shown as bel

Re: [PATCH] KVM: x86: remove vcpu's CPL check in host invoked vcpu's xcr set process

2013-06-17 Thread Paolo Bonzini
Il 14/06/2013 09:36, Zhanghaoyu (A) ha scritto: > __kvm_set_xcr function does the CPL check when set xcr. __kvm_set_xcr is > called in two flows, > one is invoked by guest, call stack shown as below, > handle_xsetbv(or xsetbv_interception) > kvm_set_xcr > __kvm_set_xcr > the other one is inv