summaryrefslogtreecommitdiffstats
path: root/virt
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2012-01-18 20:07:09 +0200
committerAvi Kivity <avi@redhat.com>2012-03-05 14:52:43 +0200
commitb93a35532767a2cf78bdbc88730d5c28aa66b941 (patch)
tree604c5c62ed765b1059735056a01c18020b0e4c26 /virt
parente2358851efbcdc34583ee11971a6e4d587ea8bf9 (diff)
downloadlinux-b93a35532767a2cf78bdbc88730d5c28aa66b941.tar.gz
linux-b93a35532767a2cf78bdbc88730d5c28aa66b941.tar.bz2
linux-b93a35532767a2cf78bdbc88730d5c28aa66b941.zip
KVM: fix error handling for out of range irq
find_index_from_host_irq returns 0 on error but callers assume < 0 on error. This should not matter much: an out of range irq should never happen since irq handler was registered with this irq #, and even if it does we get a spurious msix irq in guest and typically nothing terrible happens. Still, better to make it consistent. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com> Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'virt')
-rw-r--r--virt/kvm/assigned-dev.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/virt/kvm/assigned-dev.c b/virt/kvm/assigned-dev.c
index 758e3b36d4cf..ece80612b594 100644
--- a/virt/kvm/assigned-dev.c
+++ b/virt/kvm/assigned-dev.c
@@ -49,10 +49,8 @@ static int find_index_from_host_irq(struct kvm_assigned_dev_kernel
index = i;
break;
}
- if (index < 0) {
+ if (index < 0)
printk(KERN_WARNING "Fail to find correlated MSI-X entry!\n");
- return 0;
- }
return index;
}