Hi Tomasz, On 8 August 2012 19:36, Daniel Vetter <daniel@xxxxxxxx> wrote: > On Wed, Aug 08, 2012 at 03:53:10PM +0200, Tomasz Stanislawski wrote: >> Hi Laurent, >> >> On 08/08/2012 03:35 PM, Laurent Pinchart wrote: >> > Hi Tomasz, >> > >> > Thanks for the patch. Thanks for the patch; may I ask you to split it into 2 patches (1 dma-buf and 1 drm) and submit? That ways, either Dave or I can take the patches for either pull request. With that, please feel free to add my Acked-by as well. >> > >> > On Wednesday 08 August 2012 12:17:41 Tomasz Stanislawski wrote: >> >> This patch adds reference counting on a module that exports dma-buf and >> >> implements its operations. This prevents the module from being unloaded >> >> while DMABUF file is in use. >> >> >> >> Signed-off-by: Tomasz Stanislawski <t.stanislaws@xxxxxxxxxxx> >> >> --- >> >> Documentation/dma-buf-sharing.txt | 3 ++- >> >> drivers/base/dma-buf.c | 10 +++++++++- >> >> drivers/gpu/drm/exynos/exynos_drm_dmabuf.c | 1 + >> >> drivers/gpu/drm/i915/i915_gem_dmabuf.c | 1 + >> >> drivers/gpu/drm/nouveau/nouveau_prime.c | 1 + >> >> drivers/gpu/drm/radeon/radeon_prime.c | 1 + >> >> drivers/staging/omapdrm/omap_gem_dmabuf.c | 1 + >> >> include/linux/dma-buf.h | 2 ++ >> >> 8 files changed, 18 insertions(+), 2 deletions(-) >> >> >> [snip] >> >> >> @@ -96,6 +98,7 @@ struct dma_buf *dma_buf_export(void *priv, const struct >> >> dma_buf_ops *ops, struct file *file; >> >> >> >> if (WARN_ON(!priv || !ops >> >> + || !ops->owner >> >> Thank you for spotting this. >> I didn'y know that try_get_module returned true is module was NULL. >> >> BTW. Is it worth to add ".owner = THIS_MODULE," to all dma_buf >> exporters in this patch? > > Yeah, I think that makes sense. Otherwise it might get lost somewhere, > i.e. you can smash my Ack on this. > -Daniel > -- > Daniel Vetter > Mail: daniel@xxxxxxxx > Mobile: +41 (0)79 365 57 48 -- Thanks and regards, Sumit Semwal. -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
