diff options
author | Jan Beulich <JBeulich@suse.com> | 2012-04-02 16:22:39 +0200 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2012-04-06 18:16:02 +0200 |
commit | f09d8432e39797abf39531f41ac8a46a3fbf442a (patch) | |
tree | ea8db5e01aaea997ccec66ebdfa84a273952fb96 /drivers/pci/xen-pcifront.c | |
parent | xen/pciback: fix XEN_PCI_OP_enable_msix result (diff) | |
download | linux-f09d8432e39797abf39531f41ac8a46a3fbf442a.tar.xz linux-f09d8432e39797abf39531f41ac8a46a3fbf442a.zip |
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>
Diffstat (limited to '')
-rw-r--r-- | drivers/pci/xen-pcifront.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pci/xen-pcifront.c b/drivers/pci/xen-pcifront.c index 7cf3d2fcf56a..c18fab289ad5 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); |