diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2012-01-18 13:07:09 -0500 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2012-03-05 07:52:43 -0500 |
commit | b93a35532767a2cf78bdbc88730d5c28aa66b941 (patch) | |
tree | 604c5c62ed765b1059735056a01c18020b0e4c26 /virt/kvm | |
parent | e2358851efbcdc34583ee11971a6e4d587ea8bf9 (diff) |
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/kvm')
-rw-r--r-- | virt/kvm/assigned-dev.c | 4 |
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 | |||
49 | index = i; | 49 | index = i; |
50 | break; | 50 | break; |
51 | } | 51 | } |
52 | if (index < 0) { | 52 | if (index < 0) |
53 | printk(KERN_WARNING "Fail to find correlated MSI-X entry!\n"); | 53 | printk(KERN_WARNING "Fail to find correlated MSI-X entry!\n"); |
54 | return 0; | ||
55 | } | ||
56 | 54 | ||
57 | return index; | 55 | return index; |
58 | } | 56 | } |