Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-13 Thread Paolo Bonzini
On 12/08/2015 22:03, Christian Borntraeger wrote: >> > #if defined(CONFIG_S390) || defined(CONFIG_PPC) || defined(CONFIG_MIPS) >> > - if (ioctl == KVM_S390_INTERRUPT || ioctl == KVM_S390_IRQ || ioctl == >> > KVM_INTERRUPT) >> > - return kvm_arch_vcpu_ioctl(filp, ioctl, arg); >> > +

Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-13 Thread Radim Krčmář
2015-08-12 22:03+0200, Christian Borntraeger: > Am 05.08.2015 um 18:33 schrieb Radim Krčmář: >> diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c >> @@ -2252,12 +2252,15 @@ static long kvm_vcpu_ioctl(struct file *filp, >> * Special cases: vcpu ioctls that are asynchronous to vcpu

Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-13 Thread Radim Krčmář
2015-08-12 22:03+0200, Christian Borntraeger: Am 05.08.2015 um 18:33 schrieb Radim Krčmář: diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c @@ -2252,12 +2252,15 @@ static long kvm_vcpu_ioctl(struct file *filp, * Special cases: vcpu ioctls that are asynchronous to vcpu execution,

Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-13 Thread Paolo Bonzini
On 12/08/2015 22:03, Christian Borntraeger wrote: #if defined(CONFIG_S390) || defined(CONFIG_PPC) || defined(CONFIG_MIPS) - if (ioctl == KVM_S390_INTERRUPT || ioctl == KVM_S390_IRQ || ioctl == KVM_INTERRUPT) - return kvm_arch_vcpu_ioctl(filp, ioctl, arg); + case

Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-12 Thread Christian Borntraeger
Am 05.08.2015 um 18:33 schrieb Radim Krčmář: > I find the switch easier to read and modify. yes. > Signed-off-by: Radim Krčmář > --- > v2: new > > virt/kvm/kvm_main.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c

Re: [PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-12 Thread Christian Borntraeger
Am 05.08.2015 um 18:33 schrieb Radim Krčmář: I find the switch easier to read and modify. yes. Signed-off-by: Radim Krčmář rkrc...@redhat.com --- v2: new virt/kvm/kvm_main.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/virt/kvm/kvm_main.c

[PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-05 Thread Radim Krčmář
I find the switch easier to read and modify. Signed-off-by: Radim Krčmář --- v2: new virt/kvm/kvm_main.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index d7ffe6090520..71598554deed 100644 --- a/virt/kvm/kvm_main.c +++

[PATCH v2 5/5] KVM: refactor asynchronous vcpu ioctl dispatch

2015-08-05 Thread Radim Krčmář
I find the switch easier to read and modify. Signed-off-by: Radim Krčmář rkrc...@redhat.com --- v2: new virt/kvm/kvm_main.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index d7ffe6090520..71598554deed 100644 ---