summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Beulich <JBeulich@suse.com>2012-04-02 15:22:39 +0100
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>2012-04-06 12:16:02 -0400
commitf09d8432e39797abf39531f41ac8a46a3fbf442a (patch)
treeea8db5e01aaea997ccec66ebdfa84a273952fb96
parent0ee46eca0476faf0e93c1387b1597b861b79711f (diff)
downloadop-kernel-dev-f09d8432e39797abf39531f41ac8a46a3fbf442a.zip
op-kernel-dev-f09d8432e39797abf39531f41ac8a46a3fbf442a.tar.gz
xen/pcifront: avoid pci_frontend_enable_msix() falsely returning success
The original XenoLinux code has always had things this way, and for compatibility reasons (in particular with a subsequent pciback adjustment) upstream Linux should behave the same way (allowing for two distinct error indications to be returned by the backend). Signed-off-by: Jan Beulich <jbeulich@suse.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
-rw-r--r--drivers/pci/xen-pcifront.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pci/xen-pcifront.c b/drivers/pci/xen-pcifront.c
index 7cf3d2f..c18fab2 100644
--- a/drivers/pci/xen-pcifront.c
+++ b/drivers/pci/xen-pcifront.c
@@ -290,6 +290,7 @@ static int pci_frontend_enable_msix(struct pci_dev *dev,
} else {
printk(KERN_DEBUG "enable msix get value %x\n",
op.value);
+ err = op.value;
}
} else {
dev_err(&dev->dev, "enable msix get err %x\n", err);
OpenPOWER on IntegriCloud