On Wed, Feb 27, 2013 at 11:33:25AM +0800, Chen Gang wrote: > > irp->irq is __u32 which is never < 0. > > Signed-off-by: Chen Gang <gang.chen@xxxxxxxxxxx> Applied, thanks. > --- > arch/x86/kvm/x86.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index f71500a..811c5c9 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -2700,7 +2700,7 @@ static int kvm_vcpu_ioctl_set_lapic(struct kvm_vcpu *vcpu, > static int kvm_vcpu_ioctl_interrupt(struct kvm_vcpu *vcpu, > struct kvm_interrupt *irq) > { > - if (irq->irq < 0 || irq->irq >= KVM_NR_INTERRUPTS) > + if (irq->irq >= KVM_NR_INTERRUPTS) > return -EINVAL; > if (irqchip_in_kernel(vcpu->kvm)) > return -ENXIO; > -- > 1.7.7.6 -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html