[PATCH] kvm: device-assignment: Fix kvm_get_irq_route_gsi() return check

From: Alex Williamson <alex.williamson@hp.com>
To: kvm@vger.kernel.org
Cc: sheng.yang@intel.com, alex.williamson@hp.com
Subject: [PATCH] kvm: device-assignment: Fix kvm_get_irq_route_gsi() return check
Date: Tue, 12 May 2009 16:14:01 -0600	[thread overview]
Message-ID: <20090512221144.5883.46470.stgit@dl380g6-3.ned.telco.ned.telco> (raw)
In-Reply-To: <20090507170834.26367.91907.stgit@dl380g6-3.ned.telco.ned.telco>

Cast to a signed int to test for < 0.

Signed-off-by: Alex Williamson <alex.williamson@hp.com>
---

 This supersedes "[PATCH] kvm: device-assignment: Catch GSI overflow"

 hw/device-assignment.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/hw/device-assignment.c b/hw/device-assignment.c
index a6cc9b9..5bdae24 100644
--- a/hw/device-assignment.c
+++ b/hw/device-assignment.c
@@ -798,7 +798,7 @@ static void assigned_dev_update_msi(PCIDevice *pci_dev, unsigned int ctrl_pos)
                 pci_dev->cap.start + PCI_MSI_DATA_32);
         assigned_dev->entry->type = KVM_IRQ_ROUTING_MSI;
         assigned_dev->entry->gsi = kvm_get_irq_route_gsi(kvm_context);
-        if (assigned_dev->entry->gsi < 0) {
+        if ((int)(assigned_dev->entry->gsi) < 0) {
             perror("assigned_dev_update_msi: kvm_get_irq_route_gsi");
             return;
         }

