summaryrefslogtreecommitdiffstats
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
parente2358851efbcdc34583ee11971a6e4d587ea8bf9 (diff)
downloadop-kernel-dev-b93a35532767a2cf78bdbc88730d5c28aa66b941.zip
op-kernel-dev-b93a35532767a2cf78bdbc88730d5c28aa66b941.tar.gz
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>
-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 758e3b3..ece8061 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;
}
OpenPOWER on IntegriCloud