* [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages @ 2014-09-03 9:28 Roman Gushchin 2014-09-19 12:04 ` Roman Gushchin 2014-09-24 4:57 ` Vinod Koul 0 siblings, 2 replies; 7+ messages in thread From: Roman Gushchin @ 2014-09-03 9:28 UTC (permalink / raw) To: dan.j.williams, vinod.koul, dmaengine, linux-kernel dma_pin_iovec_pages() calls get_user_pages() for each iovec. If get_user_pages() returns a number smaller than the requested number, dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases previously allocated iovecs, but pages pinned by last get_user_pages() call remain unreleased. Fix this by calling put_page() for each such page. Signed-off-by: Roman Gushchin <klamm@yandex-team.ru> --- drivers/dma/iovlock.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/dma/iovlock.c b/drivers/dma/iovlock.c index bb48a57..c393cf9 100644 --- a/drivers/dma/iovlock.c +++ b/drivers/dma/iovlock.c @@ -107,8 +107,11 @@ struct dma_pinned_list *dma_pin_iovec_pages(struct iovec *iov, size_t len) NULL); up_read(¤t->mm->mmap_sem); - if (ret != page_list->nr_pages) + if (ret != page_list->nr_pages) { + for (i = 0; i < ret; i++) + put_page(page_list->pages[i]); goto unpin; + } local_list->nr_iovecs = i + 1; } -- 1.9.3 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-03 9:28 [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages Roman Gushchin @ 2014-09-19 12:04 ` Roman Gushchin 2014-09-24 4:57 ` Vinod Koul 1 sibling, 0 replies; 7+ messages in thread From: Roman Gushchin @ 2014-09-19 12:04 UTC (permalink / raw) To: dan.j.williams@intel.com, vinod.koul@intel.com, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org Ping? 03.09.2014, 13:38, "Roman Gushchin" <klamm@yandex-team.ru>: > dma_pin_iovec_pages() calls get_user_pages() for each iovec. If > get_user_pages() returns a number smaller than the requested number, > dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases > previously allocated iovecs, but pages pinned by last get_user_pages() > call remain unreleased. > Fix this by calling put_page() for each such page. > > Signed-off-by: Roman Gushchin <klamm@yandex-team.ru> > --- > drivers/dma/iovlock.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/dma/iovlock.c b/drivers/dma/iovlock.c > index bb48a57..c393cf9 100644 > --- a/drivers/dma/iovlock.c > +++ b/drivers/dma/iovlock.c > @@ -107,8 +107,11 @@ struct dma_pinned_list *dma_pin_iovec_pages(struct iovec *iov, size_t len) > NULL); > up_read(¤t->mm->mmap_sem); > > - if (ret != page_list->nr_pages) > + if (ret != page_list->nr_pages) { > + for (i = 0; i < ret; i++) > + put_page(page_list->pages[i]); > goto unpin; > + } > > local_list->nr_iovecs = i + 1; > } > -- > 1.9.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-03 9:28 [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages Roman Gushchin 2014-09-19 12:04 ` Roman Gushchin @ 2014-09-24 4:57 ` Vinod Koul 2014-09-24 6:27 ` Dan Williams 1 sibling, 1 reply; 7+ messages in thread From: Vinod Koul @ 2014-09-24 4:57 UTC (permalink / raw) To: Roman Gushchin; +Cc: dan.j.williams, dmaengine, linux-kernel On Wed, Sep 03, 2014 at 01:28:59PM +0400, Roman Gushchin wrote: > dma_pin_iovec_pages() calls get_user_pages() for each iovec. If > get_user_pages() returns a number smaller than the requested number, > dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases > previously allocated iovecs, but pages pinned by last get_user_pages() > call remain unreleased. > Fix this by calling put_page() for each such page. > Applied, thanks -- ~Vinod ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-24 4:57 ` Vinod Koul @ 2014-09-24 6:27 ` Dan Williams 2014-09-25 16:16 ` Vinod Koul 0 siblings, 1 reply; 7+ messages in thread From: Dan Williams @ 2014-09-24 6:27 UTC (permalink / raw) To: Vinod Koul Cc: Roman Gushchin, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org On Tue, Sep 23, 2014 at 9:57 PM, Vinod Koul <vinod.koul@intel.com> wrote: > On Wed, Sep 03, 2014 at 01:28:59PM +0400, Roman Gushchin wrote: >> dma_pin_iovec_pages() calls get_user_pages() for each iovec. If >> get_user_pages() returns a number smaller than the requested number, >> dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases >> previously allocated iovecs, but pages pinned by last get_user_pages() >> call remain unreleased. >> Fix this by calling put_page() for each such page. >> > Applied, thanks Vinod, I have a patch in my queue that completely removes the remnants of NET_DMA. It's been deprecated for more than a few cycles now without any reports to reinstate it. Time to push the final removal patch. I'll rebase it on top of this so it doesn't collide. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-24 6:27 ` Dan Williams @ 2014-09-25 16:16 ` Vinod Koul 2014-09-28 15:36 ` Dan Williams 0 siblings, 1 reply; 7+ messages in thread From: Vinod Koul @ 2014-09-25 16:16 UTC (permalink / raw) To: Dan Williams Cc: Roman Gushchin, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org On Tue, Sep 23, 2014 at 11:27:06PM -0700, Dan Williams wrote: > On Tue, Sep 23, 2014 at 9:57 PM, Vinod Koul <vinod.koul@intel.com> wrote: > > On Wed, Sep 03, 2014 at 01:28:59PM +0400, Roman Gushchin wrote: > >> dma_pin_iovec_pages() calls get_user_pages() for each iovec. If > >> get_user_pages() returns a number smaller than the requested number, > >> dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases > >> previously allocated iovecs, but pages pinned by last get_user_pages() > >> call remain unreleased. > >> Fix this by calling put_page() for each such page. > >> > > Applied, thanks > > Vinod, I have a patch in my queue that completely removes the remnants > of NET_DMA. It's been deprecated for more than a few cycles now > without any reports to reinstate it. Time to push the final removal > patch. I'll rebase it on top of this so it doesn't collide. Okay, thanks for update. If you wnat I can remove it, so you don't need to rebase. -- ~Vinod ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-25 16:16 ` Vinod Koul @ 2014-09-28 15:36 ` Dan Williams 2014-09-28 15:25 ` Vinod Koul 0 siblings, 1 reply; 7+ messages in thread From: Dan Williams @ 2014-09-28 15:36 UTC (permalink / raw) To: Vinod Koul Cc: Roman Gushchin, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org On Thu, Sep 25, 2014 at 9:16 AM, Vinod Koul <vinod.koul@intel.com> wrote: > On Tue, Sep 23, 2014 at 11:27:06PM -0700, Dan Williams wrote: >> On Tue, Sep 23, 2014 at 9:57 PM, Vinod Koul <vinod.koul@intel.com> wrote: >> > On Wed, Sep 03, 2014 at 01:28:59PM +0400, Roman Gushchin wrote: >> >> dma_pin_iovec_pages() calls get_user_pages() for each iovec. If >> >> get_user_pages() returns a number smaller than the requested number, >> >> dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases >> >> previously allocated iovecs, but pages pinned by last get_user_pages() >> >> call remain unreleased. >> >> Fix this by calling put_page() for each such page. >> >> >> > Applied, thanks >> >> Vinod, I have a patch in my queue that completely removes the remnants >> of NET_DMA. It's been deprecated for more than a few cycles now >> without any reports to reinstate it. Time to push the final removal >> patch. I'll rebase it on top of this so it doesn't collide. > > Okay, thanks for update. If you wnat I can remove it, so you don't need to > rebase. > Yes, please drop it. I'm sending the full removal to Linus. I added Roman to the Reported-by and cc'd stable. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages 2014-09-28 15:36 ` Dan Williams @ 2014-09-28 15:25 ` Vinod Koul 0 siblings, 0 replies; 7+ messages in thread From: Vinod Koul @ 2014-09-28 15:25 UTC (permalink / raw) To: Dan Williams Cc: Roman Gushchin, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org On Sun, Sep 28, 2014 at 08:36:44AM -0700, Dan Williams wrote: > On Thu, Sep 25, 2014 at 9:16 AM, Vinod Koul <vinod.koul@intel.com> wrote: > > On Tue, Sep 23, 2014 at 11:27:06PM -0700, Dan Williams wrote: > >> On Tue, Sep 23, 2014 at 9:57 PM, Vinod Koul <vinod.koul@intel.com> wrote: > >> > On Wed, Sep 03, 2014 at 01:28:59PM +0400, Roman Gushchin wrote: > >> >> dma_pin_iovec_pages() calls get_user_pages() for each iovec. If > >> >> get_user_pages() returns a number smaller than the requested number, > >> >> dma_pin_iovec_pages() calls dma_unpin_iovec_pages(). It releases > >> >> previously allocated iovecs, but pages pinned by last get_user_pages() > >> >> call remain unreleased. > >> >> Fix this by calling put_page() for each such page. > >> >> > >> > Applied, thanks > >> > >> Vinod, I have a patch in my queue that completely removes the remnants > >> of NET_DMA. It's been deprecated for more than a few cycles now > >> without any reports to reinstate it. Time to push the final removal > >> patch. I'll rebase it on top of this so it doesn't collide. > > > > Okay, thanks for update. If you wnat I can remove it, so you don't need to > > rebase. > > > > Yes, please drop it. I'm sending the full removal to Linus. I added > Roman to the Reported-by and cc'd stable. Done, i have pushed the updated tree. -- ~Vinod ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-09-28 15:55 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-09-03 9:28 [PATCH] net_dma: fix memory leak in dma_pin_iocvec_pages Roman Gushchin 2014-09-19 12:04 ` Roman Gushchin 2014-09-24 4:57 ` Vinod Koul 2014-09-24 6:27 ` Dan Williams 2014-09-25 16:16 ` Vinod Koul 2014-09-28 15:36 ` Dan Williams 2014-09-28 15:25 ` Vinod Koul
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).