On Wed, Feb 26, 2020 at 5:29 AM Sumit Semwal <[email protected]> wrote: > > Hello Andrew, > > > On Wed, 26 Feb 2020 at 07:25, Andrew Morton <[email protected]> wrote: > > > > > > The patch titled > > Subject: dma-buf: free dmabuf->name in dma_buf_release() > > has been added to the -mm tree. Its filename is > > dma-buf-free-dmabuf-name-in-dma_buf_release.patch > > Thanks for taking this patch via -mm during my absence (I'm just > returning from a bit of an illness). If there are other dma-buf > patches on your radar that you'd like to take via the mm tree, please > let me know and I can provide the necessary Acks. > Else I will take them in via drm-misc as usual.
I thought at least that for cases like these -mm is the last resort tree, so proper thing to do here is apply this fix to drm-misc-fixes and get it out there. -mm rebases, so will fall out again. Also maybe we should have had some tests for this, plus some integration with the gem object name instead of duplicating buffer object names everywhere for different reasons. -Daniel > > > > > This patch should soon appear at > > > > http://ozlabs.org/~akpm/mmots/broken-out/dma-buf-free-dmabuf-name-in-dma_buf_release.patch > > and later at > > > > http://ozlabs.org/~akpm/mmotm/broken-out/dma-buf-free-dmabuf-name-in-dma_buf_release.patch > > > > Before you just go and hit "reply", please: > > a) Consider who else should be cc'ed > > b) Prefer to cc a suitable mailing list as well > > c) Ideally: find the original patch on the mailing list and do a > > reply-to-all to that, adding suitable additional cc's > > > > *** Remember to use Documentation/process/submit-checklist.rst when testing > > your code *** > > > > The -mm tree is included into linux-next and is updated > > there every 3-4 working days > > > > ------------------------------------------------------ > > From: Cong Wang <[email protected]> > > Subject: dma-buf: free dmabuf->name in dma_buf_release() > > > > dma-buff name can be set via DMA_BUF_SET_NAME ioctl, but once set > > it never gets freed. > > > > Free it in dma_buf_release(). > > > > Link: > > http://lkml.kernel.org/r/[email protected] > > Fixes: bb2bb9030425 ("dma-buf: add DMA_BUF_SET_NAME ioctls") > > Reported-by: [email protected] > > Signed-off-by: Cong Wang <[email protected]> > > Reviewed-by: Andrew Morton <[email protected]> > > Acked-by: Chenbo Feng <[email protected]> > > Cc: Sumit Semwal <[email protected]> > > Cc: Greg Hackmann <[email protected]> > > Signed-off-by: Andrew Morton <[email protected]> > > --- > > > > drivers/dma-buf/dma-buf.c | 1 + > > 1 file changed, 1 insertion(+) > > > > --- a/drivers/dma-buf/dma-buf.c~dma-buf-free-dmabuf-name-in-dma_buf_release > > +++ a/drivers/dma-buf/dma-buf.c > > @@ -108,6 +108,7 @@ static int dma_buf_release(struct inode > > dma_resv_fini(dmabuf->resv); > > > > module_put(dmabuf->owner); > > + kfree(dmabuf->name); > > kfree(dmabuf); > > return 0; > > } > > _ > > > > Patches currently in -mm which might be from [email protected] are > > > > dma-buf-free-dmabuf-name-in-dma_buf_release.patch > > > Thanks and best regards, > Sumit. > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
