diff options
author | ruanjinjie <ruanjinjie@huawei.com> | 2022-11-14 12:21:24 +0100 |
---|---|---|
committer | Juergen Gross <jgross@suse.com> | 2022-11-14 13:29:14 +0100 |
commit | c53717e1e3f0d0f9129b2e0dbc6dcc5e0a8132e9 (patch) | |
tree | 039d575858fd740ea33fd4f972b4f6821ccca97a /drivers/xen/platform-pci.c | |
parent | xen-pciback: Allow setting PCI_MSIX_FLAGS_MASKALL too (diff) | |
download | linux-c53717e1e3f0d0f9129b2e0dbc6dcc5e0a8132e9.tar.xz linux-c53717e1e3f0d0f9129b2e0dbc6dcc5e0a8132e9.zip |
xen/platform-pci: add missing free_irq() in error path
free_irq() is missing in case of error in platform_pci_probe(), fix that.
Signed-off-by: ruanjinjie <ruanjinjie@huawei.com>
Reviewed-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>
Link: https://lore.kernel.org/r/20221114112124.1965611-1-ruanjinjie@huawei.com
Signed-off-by: Juergen Gross <jgross@suse.com>
Diffstat (limited to 'drivers/xen/platform-pci.c')
-rw-r--r-- | drivers/xen/platform-pci.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/xen/platform-pci.c b/drivers/xen/platform-pci.c index 18f0ed8b1f93..6ebd819338ec 100644 --- a/drivers/xen/platform-pci.c +++ b/drivers/xen/platform-pci.c @@ -144,7 +144,7 @@ static int platform_pci_probe(struct pci_dev *pdev, if (ret) { dev_warn(&pdev->dev, "Unable to set the evtchn callback " "err=%d\n", ret); - goto out; + goto irq_out; } } @@ -152,13 +152,16 @@ static int platform_pci_probe(struct pci_dev *pdev, grant_frames = alloc_xen_mmio(PAGE_SIZE * max_nr_gframes); ret = gnttab_setup_auto_xlat_frames(grant_frames); if (ret) - goto out; + goto irq_out; ret = gnttab_init(); if (ret) goto grant_out; return 0; grant_out: gnttab_free_auto_xlat_frames(); +irq_out: + if (!xen_have_vector_callback) + free_irq(pdev->irq, pdev); out: pci_release_region(pdev, 0); mem_out: |