Messages in this thread | | | Date | Wed, 13 Dec 2017 09:47:55 +0100 | From | Cornelia Huck <> | Subject | Re: [PATCH] KVM: introduce kvm_arch_vcpu_async_ioctl |
| |
On Tue, 12 Dec 2017 18:31:55 +0100 Paolo Bonzini <pbonzini@redhat.com> wrote:
> After the vcpu_load/vcpu_put pushdown, the handling of asynchronous VCPU > ioctl is already much clearer in that it is obvious that they bypass > vcpu_load and vcpu_put. > > However, it is still not perfect in that the different state of the VCPU > mutex is still hidden in the caller. Separate those ioctls into a new > function kvm_arch_vcpu_async_ioctl that returns -ENOIOCTLCMD for more > "traditional" synchronous ioctls. > > Cc: James Hogan <jhogan@kernel.org> > Cc: Paul Mackerras <paulus@ozlabs.org> > Cc: Christian Borntraeger <borntraeger@de.ibm.com> > Suggested-by: Cornelia Huck <cohuck@redhat.com> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> > --- > arch/mips/kvm/mips.c | 15 ++++++++++++--- > arch/powerpc/kvm/powerpc.c | 14 +++++++++++--- > arch/s390/kvm/kvm-s390.c | 16 ++++++++++++---- > include/linux/kvm_host.h | 2 ++ > virt/kvm/kvm_main.c | 8 ++++---- > 5 files changed, 41 insertions(+), 14 deletions(-)
Another vote for a dummy function here, but otherwise
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
| |