diff options
| author | Jan Beulich <JBeulich@suse.com> | 2012-04-02 15:22:39 +0100 |
|---|---|---|
| committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2012-04-06 12:16:02 -0400 |
| commit | f09d8432e39797abf39531f41ac8a46a3fbf442a (patch) | |
| tree | ea8db5e01aaea997ccec66ebdfa84a273952fb96 /drivers | |
| parent | 0ee46eca0476faf0e93c1387b1597b861b79711f (diff) | |
| download | linux-f09d8432e39797abf39531f41ac8a46a3fbf442a.tar.bz2 | |
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 'drivers')
| -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); |