* [PATCH] spi: dw-mid: don't leak memory on exit
@ 2014-08-28 8:53 Andy Shevchenko
2014-08-28 18:19 ` Mark Brown
0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2014-08-28 8:53 UTC (permalink / raw)
To: Feng Tang, Mark Brown, linux-spi-u79uwXL29TY76Z2rM5mHXA
Cc: Andy Shevchenko, stable-u79uwXL29TY76Z2rM5mHXA
When built with DMA support the memory for an internal structure is allocated
but not freed. Converting to devm_kzalloc solves the issue.
Fixes: 7063c0d942a1 (spi/dw_spi: add DMA support)
Signed-off-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Cc: <stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> # v2.6.38+
---
drivers/spi/spi-dw-mid.c | 4 ++--
drivers/spi/spi-dw-pci.c | 5 ++---
drivers/spi/spi-dw.h | 3 ++-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/spi/spi-dw-mid.c b/drivers/spi/spi-dw-mid.c
index c79bf96..ceb95b4 100644
--- a/drivers/spi/spi-dw-mid.c
+++ b/drivers/spi/spi-dw-mid.c
@@ -202,7 +202,7 @@ static struct dw_spi_dma_ops mid_dma_ops = {
#define CLK_SPI_CDIV_MASK 0x00000e00
#define CLK_SPI_DISABLE_OFFSET 8
-int dw_spi_mid_init(struct dw_spi *dws)
+int dw_spi_mid_init(struct device *dev, struct dw_spi *dws)
{
void __iomem *clk_reg;
u32 clk_cdiv;
@@ -220,7 +220,7 @@ int dw_spi_mid_init(struct dw_spi *dws)
dws->fifo_len = 40; /* FIFO has 40 words buffer */
#ifdef CONFIG_SPI_DW_MID_DMA
- dws->dma_priv = kzalloc(sizeof(struct mid_dma), GFP_KERNEL);
+ dws->dma_priv = devm_kzalloc(dev, sizeof(struct mid_dma), GFP_KERNEL);
if (!dws->dma_priv)
return -ENOMEM;
dws->dma_ops = &mid_dma_ops;
diff --git a/drivers/spi/spi-dw-pci.c b/drivers/spi/spi-dw-pci.c
index b8b02d7..dd06c07 100644
--- a/drivers/spi/spi-dw-pci.c
+++ b/drivers/spi/spi-dw-pci.c
@@ -28,8 +28,7 @@ struct dw_spi_pci {
struct dw_spi dws;
};
-static int spi_pci_probe(struct pci_dev *pdev,
- const struct pci_device_id *ent)
+static int spi_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
{
struct dw_spi_pci *dwpci;
struct dw_spi *dws;
@@ -66,7 +65,7 @@ static int spi_pci_probe(struct pci_dev *pdev,
* clock rate, FIFO depth.
*/
if (pdev->device == 0x0800) {
- ret = dw_spi_mid_init(dws);
+ ret = dw_spi_mid_init(&pdev->dev, dws);
if (ret)
return ret;
}
diff --git a/drivers/spi/spi-dw.h b/drivers/spi/spi-dw.h
index 0f8cece..b3ff9eb 100644
--- a/drivers/spi/spi-dw.h
+++ b/drivers/spi/spi-dw.h
@@ -232,5 +232,6 @@ extern int dw_spi_resume_host(struct dw_spi *dws);
extern void dw_spi_xfer_done(struct dw_spi *dws);
/* platform related setup */
-extern int dw_spi_mid_init(struct dw_spi *dws); /* Intel MID platforms */
+/* Intel MID platforms */
+extern int dw_spi_mid_init(struct device *dev, struct dw_spi *dws);
#endif /* DW_SPI_HEADER_H */
--
2.1.0
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] spi: dw-mid: don't leak memory on exit
2014-08-28 8:53 [PATCH] spi: dw-mid: don't leak memory on exit Andy Shevchenko
@ 2014-08-28 18:19 ` Mark Brown
2014-08-29 7:35 ` Andy Shevchenko
0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2014-08-28 18:19 UTC (permalink / raw)
To: Andy Shevchenko; +Cc: Feng Tang, linux-spi, stable
[-- Attachment #1: Type: text/plain, Size: 562 bytes --]
On Thu, Aug 28, 2014 at 11:53:17AM +0300, Andy Shevchenko wrote:
> When built with DMA support the memory for an internal structure is allocated
> but not freed. Converting to devm_kzalloc solves the issue.
>
> Fixes: 7063c0d942a1 (spi/dw_spi: add DMA support)
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Cc: <stable@vger.kernel.org> # v2.6.38+
You've sent two versions of this with different changelogs and diffs -
what's the story there?
This doesn't seem like an obvious stable candidate - it's a leak fix on
probe.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] spi: dw-mid: don't leak memory on exit
2014-08-28 18:19 ` Mark Brown
@ 2014-08-29 7:35 ` Andy Shevchenko
0 siblings, 0 replies; 3+ messages in thread
From: Andy Shevchenko @ 2014-08-29 7:35 UTC (permalink / raw)
To: Mark Brown; +Cc: Feng Tang, linux-spi, stable
On Thu, 2014-08-28 at 19:19 +0100, Mark Brown wrote:
> On Thu, Aug 28, 2014 at 11:53:17AM +0300, Andy Shevchenko wrote:
> > When built with DMA support the memory for an internal structure is allocated
> > but not freed. Converting to devm_kzalloc solves the issue.
> >
> > Fixes: 7063c0d942a1 (spi/dw_spi: add DMA support)
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > Cc: <stable@vger.kernel.org> # v2.6.38+
>
> You've sent two versions of this with different changelogs and diffs -
> what's the story there?
Sorry for confusion.
There are two different fixes against MID support.
One of them to protect against crash on removal, another is this one.
> This doesn't seem like an obvious stable candidate - it's a leak fix on
> probe.
Yes, I may agree with you. Then, please, do not consider this one. It
would be a part of a patches against spi-dw-mid.c
--
Andy Shevchenko <andriy.shevchenko@intel.com>
Intel Finland Oy
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-29 7:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-28 8:53 [PATCH] spi: dw-mid: don't leak memory on exit Andy Shevchenko
2014-08-28 18:19 ` Mark Brown
2014-08-29 7:35 ` Andy Shevchenko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.