* FAILED: patch "[PATCH] spi: dw: explicitly free IRQ handler in dw_spi_remove_host()" failed to apply to 4.3-stable tree
@ 2016-02-13 22:17 gregkh
2016-02-17 10:01 ` Andy Shevchenko
0 siblings, 1 reply; 2+ messages in thread
From: gregkh @ 2016-02-13 22:17 UTC (permalink / raw)
To: andriy.shevchenko; +Cc: stable
The patch below does not apply to the 4.3-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@vger.kernel.org>.
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
>From 02f20387e1bca550639c37b1945f20cd32ddfcce Mon Sep 17 00:00:00 2001
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: Tue, 20 Oct 2015 12:11:40 +0300
Subject: [PATCH] spi: dw: explicitly free IRQ handler in dw_spi_remove_host()
The following warning occurs when DW SPI is compiled as a module and it's a PCI
device. On the removal stage pcibios_free_irq() is called earlier than
free_irq() due to the latter is called at managed resources free strage.
------------[ cut here ]------------
WARNING: CPU: 1 PID: 1003 at /home/andy/prj/linux/fs/proc/generic.c:575 remove_proc_entry+0x118/0x150()
remove_proc_entry: removing non-empty directory 'irq/38', leaking at least 'dw_spi1'
Modules linked in: spi_dw_midpci(-) spi_dw [last unloaded: dw_dmac_core]
CPU: 1 PID: 1003 Comm: modprobe Not tainted 4.3.0-rc5-next-20151013+ #32
00000000 00000000 f5535d70 c12dc220 f5535db0 f5535da0 c104e912 c198a6bc
f5535dcc 000003eb c198a638 0000023f c11b4098 c11b4098 f54f1ec8 f54f1ea0
f642ba20 f5535db8 c104e96e 00000009 f5535db0 c198a6bc f5535dcc f5535df0
Call Trace:
[<c12dc220>] dump_stack+0x41/0x61
[<c104e912>] warn_slowpath_common+0x82/0xb0
[<c11b4098>] ? remove_proc_entry+0x118/0x150
[<c11b4098>] ? remove_proc_entry+0x118/0x150
[<c104e96e>] warn_slowpath_fmt+0x2e/0x30
[<c11b4098>] remove_proc_entry+0x118/0x150
[<c109b96a>] unregister_irq_proc+0xaa/0xc0
[<c109575e>] free_desc+0x1e/0x60
[<c10957d2>] irq_free_descs+0x32/0x70
[<c109b1a0>] irq_domain_free_irqs+0x120/0x150
[<c1039e8c>] mp_unmap_irq+0x5c/0x60
[<c16277b0>] intel_mid_pci_irq_disable+0x20/0x40
[<c1627c7f>] pcibios_free_irq+0xf/0x20
[<c13189f2>] pci_device_remove+0x52/0xb0
[<c13f6367>] __device_release_driver+0x77/0x100
[<c13f6da7>] driver_detach+0x87/0x90
[<c13f5eaa>] bus_remove_driver+0x4a/0xc0
[<c128bf0d>] ? selinux_capable+0xd/0x10
[<c13f7483>] driver_unregister+0x23/0x60
[<c10bad8a>] ? find_module_all+0x5a/0x80
[<c1317413>] pci_unregister_driver+0x13/0x60
[<f80ac654>] dw_spi_driver_exit+0xd/0xf [spi_dw_midpci]
[<c10bce9a>] SyS_delete_module+0x17a/0x210
Explicitly call free_irq() at removal stage of the DW SPI driver.
Fixes: 04f421e7b0b1 (spi: dw: use managed resources)
Cc: stable@vger.kernel.org
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c
index a730c353d3bf..221ff97557a3 100644
--- a/drivers/spi/spi-dw.c
+++ b/drivers/spi/spi-dw.c
@@ -486,8 +486,7 @@ int dw_spi_add_host(struct device *dev, struct dw_spi *dws)
dws->dma_addr = (dma_addr_t)(dws->paddr + 0x60);
snprintf(dws->name, sizeof(dws->name), "dw_spi%d", dws->bus_num);
- ret = devm_request_irq(dev, dws->irq, dw_spi_irq, IRQF_SHARED,
- dws->name, master);
+ ret = request_irq(dws->irq, dw_spi_irq, IRQF_SHARED, dws->name, master);
if (ret < 0) {
dev_err(dev, "can not get IRQ\n");
goto err_free_master;
@@ -532,6 +531,7 @@ err_dma_exit:
if (dws->dma_ops && dws->dma_ops->dma_exit)
dws->dma_ops->dma_exit(dws);
spi_enable_chip(dws, 0);
+ free_irq(dws->irq, master);
err_free_master:
spi_master_put(master);
return ret;
@@ -548,6 +548,8 @@ void dw_spi_remove_host(struct dw_spi *dws)
dws->dma_ops->dma_exit(dws);
spi_shutdown_chip(dws);
+
+ free_irq(dws->irq, dws->master);
}
EXPORT_SYMBOL_GPL(dw_spi_remove_host);
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: FAILED: patch "[PATCH] spi: dw: explicitly free IRQ handler in dw_spi_remove_host()" failed to apply to 4.3-stable tree
2016-02-13 22:17 FAILED: patch "[PATCH] spi: dw: explicitly free IRQ handler in dw_spi_remove_host()" failed to apply to 4.3-stable tree gregkh
@ 2016-02-17 10:01 ` Andy Shevchenko
0 siblings, 0 replies; 2+ messages in thread
From: Andy Shevchenko @ 2016-02-17 10:01 UTC (permalink / raw)
To: gregkh; +Cc: stable
On Sat, 2016-02-13 at 14:17 -0800, gregkh@linuxfoundation.org wrote:
> The patch below does not apply to the 4.3-stable tree.
> If someone wants it applied there, or to any other stable or longterm
> tree, then please email the backport, including the original git
> commit
> id to <stable@vger.kernel.org>.
Yes, because the next patch in the series had been applied before this
one. Thus, to apply this cleanly we have to apply the commit
1cc3f141f0cb5a822cdef30fb1d92ae6f4176bfa ("spi: dw: introduce
spi_shutdown_chip()") first.
Sorry for this.
>
> thanks,
>
> greg k-h
>
> ------------------ original commit in Linus's tree ------------------
>
> From 02f20387e1bca550639c37b1945f20cd32ddfcce Mon Sep 17 00:00:00
> 2001
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Date: Tue, 20 Oct 2015 12:11:40 +0300
> Subject: [PATCH] spi: dw: explicitly free IRQ handler in
> dw_spi_remove_host()
>
> The following warning occurs when DW SPI is compiled as a module and
> it's a PCI
> device. On the removal stage pcibios_free_irq() is called earlier
> than
> free_irq() due to the latter is called at managed resources free
> strage.
>
> ------------[ cut here ]------------
> WARNING: CPU: 1 PID: 1003 at
> /home/andy/prj/linux/fs/proc/generic.c:575
> remove_proc_entry+0x118/0x150()
> remove_proc_entry: removing non-empty directory 'irq/38', leaking at
> least 'dw_spi1'
> Modules linked in: spi_dw_midpci(-) spi_dw [last unloaded:
> dw_dmac_core]
> CPU: 1 PID: 1003 Comm: modprobe Not tainted 4.3.0-rc5-next-20151013+
> #32
> 00000000 00000000 f5535d70 c12dc220 f5535db0 f5535da0 c104e912
> c198a6bc
> f5535dcc 000003eb c198a638 0000023f c11b4098 c11b4098 f54f1ec8
> f54f1ea0
> f642ba20 f5535db8 c104e96e 00000009 f5535db0 c198a6bc f5535dcc
> f5535df0
> Call Trace:
> [<c12dc220>] dump_stack+0x41/0x61
> [<c104e912>] warn_slowpath_common+0x82/0xb0
> [<c11b4098>] ? remove_proc_entry+0x118/0x150
> [<c11b4098>] ? remove_proc_entry+0x118/0x150
> [<c104e96e>] warn_slowpath_fmt+0x2e/0x30
> [<c11b4098>] remove_proc_entry+0x118/0x150
> [<c109b96a>] unregister_irq_proc+0xaa/0xc0
> [<c109575e>] free_desc+0x1e/0x60
> [<c10957d2>] irq_free_descs+0x32/0x70
> [<c109b1a0>] irq_domain_free_irqs+0x120/0x150
> [<c1039e8c>] mp_unmap_irq+0x5c/0x60
> [<c16277b0>] intel_mid_pci_irq_disable+0x20/0x40
> [<c1627c7f>] pcibios_free_irq+0xf/0x20
> [<c13189f2>] pci_device_remove+0x52/0xb0
> [<c13f6367>] __device_release_driver+0x77/0x100
> [<c13f6da7>] driver_detach+0x87/0x90
> [<c13f5eaa>] bus_remove_driver+0x4a/0xc0
> [<c128bf0d>] ? selinux_capable+0xd/0x10
> [<c13f7483>] driver_unregister+0x23/0x60
> [<c10bad8a>] ? find_module_all+0x5a/0x80
> [<c1317413>] pci_unregister_driver+0x13/0x60
> [<f80ac654>] dw_spi_driver_exit+0xd/0xf [spi_dw_midpci]
> [<c10bce9a>] SyS_delete_module+0x17a/0x210
>
> Explicitly call free_irq() at removal stage of the DW SPI driver.
>
> Fixes: 04f421e7b0b1 (spi: dw: use managed resources)
> Cc: stable@vger.kernel.org
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Signed-off-by: Mark Brown <broonie@kernel.org>
>
> diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c
> index a730c353d3bf..221ff97557a3 100644
> --- a/drivers/spi/spi-dw.c
> +++ b/drivers/spi/spi-dw.c
> @@ -486,8 +486,7 @@ int dw_spi_add_host(struct device *dev, struct
> dw_spi *dws)
> dws->dma_addr = (dma_addr_t)(dws->paddr + 0x60);
> snprintf(dws->name, sizeof(dws->name), "dw_spi%d", dws-
> >bus_num);
>
> - ret = devm_request_irq(dev, dws->irq, dw_spi_irq,
> IRQF_SHARED,
> - dws->name, master);
> + ret = request_irq(dws->irq, dw_spi_irq, IRQF_SHARED, dws-
> >name, master);
> if (ret < 0) {
> dev_err(dev, "can not get IRQ\n");
> goto err_free_master;
> @@ -532,6 +531,7 @@ err_dma_exit:
> if (dws->dma_ops && dws->dma_ops->dma_exit)
> dws->dma_ops->dma_exit(dws);
> spi_enable_chip(dws, 0);
> + free_irq(dws->irq, master);
> err_free_master:
> spi_master_put(master);
> return ret;
> @@ -548,6 +548,8 @@ void dw_spi_remove_host(struct dw_spi *dws)
> dws->dma_ops->dma_exit(dws);
>
> spi_shutdown_chip(dws);
> +
> + free_irq(dws->irq, dws->master);
> }
> EXPORT_SYMBOL_GPL(dw_spi_remove_host);
>
>
--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-02-17 10:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-13 22:17 FAILED: patch "[PATCH] spi: dw: explicitly free IRQ handler in dw_spi_remove_host()" failed to apply to 4.3-stable tree gregkh
2016-02-17 10:01 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).