lkml.org 
[lkml]   [2013]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC][PATCH 0/2] dma-buf: add importer private data for reimporting
    Hello Daniel,

    Thanks for your comment.

    On 2013년 05월 31일 18:14, Daniel Vetter wrote:
    > On Fri, May 31, 2013 at 10:54 AM, Seung-Woo Kim <sw0312.kim@samsung.com> wrote:
    >> importer private data in dma-buf attachment can be used by importer to
    >> reimport same dma-buf.
    >>
    >> Seung-Woo Kim (2):
    >> dma-buf: add importer private data to attachment
    >> drm/prime: find gem object from the reimported dma-buf
    >
    > Self-import should already work (at least with the latest refcount
    > fixes merged). At least the tests to check both re-import on the same
    > drm fd and on a different all work as expected now.

    Currently, prime works well for all case including self-importing,
    importing, and reimporting as you describe. Just, importing dma-buf from
    other driver twice with different drm_fd, each import create its own gem
    object even two import is done for same buffer because prime_priv is in
    struct drm_file. This means mapping to the device is done also twice.
    IMHO, these duplicated creations and maps are not necessary if drm can
    find previous import in different prime_priv.

    >
    > Second, the dma_buf_attachment is _definitely_ the wrong place to do
    > this. If you need iommu mapping caching, that should happen at a lower
    > level (i.e. in the map_attachment callback somewhere of the exporter,
    > that's what the priv field in the attachment is for). Snatching away
    > the attachement from some random other import is certainly not the way
    > to go - attachements are _not_ refcounted!

    Yes, attachments do not have refcount, so importer should handle and drm
    case in my patch, importer private data is gem object and it has, of
    course, refcount.

    And at current, exporter can not classify map_dma_buf requests of same
    importer to same buffer with different attachment because dma_buf_attach
    always makes new attachments. To resolve this exporter should search all
    different attachment from same importer of dma-buf and it seems more
    complex than importer private data to me.

    If I misunderstood something, please let me know.

    Best Regards,
    - Seung-Woo Kim

    > -Daniel
    >
    >>
    >> drivers/base/dma-buf.c | 31 ++++++++++++++++++++++++++++
    >> drivers/gpu/drm/drm_prime.c | 19 ++++++++++++----
    >> drivers/gpu/drm/exynos/exynos_drm_dmabuf.c | 1 +
    >> drivers/gpu/drm/i915/i915_gem_dmabuf.c | 1 +
    >> drivers/gpu/drm/udl/udl_gem.c | 1 +
    >> include/linux/dma-buf.h | 4 +++
    >> 6 files changed, 52 insertions(+), 5 deletions(-)
    >>
    >> --
    >> 1.7.4.1
    >>
    >
    >
    >
    > --
    > Daniel Vetter
    > Software Engineer, Intel Corporation
    > +41 (0) 79 365 57 48 - http://blog.ffwll.ch
    >

    --
    Seung-Woo Kim
    Samsung Software R&D Center
    --

    --
    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/

    \
     
     \ /
      Last update: 2013-05-31 13:01    [W:6.888 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site